laforge has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/32649 ) Change subject: core: Check return value of osmo_fd_register() ...................................................................... core: Check return value of osmo_fd_register() Change-Id: I15bb456c42d5033c9cf4c0937df22fa02b6a6086 --- M src/core/tun.c 1 file changed, 15 insertions(+), 1 deletion(-) Approvals: laforge: Looks good to me, approved fixeria: Looks good to me, but someone else must approve Jenkins Builder: Verified diff --git a/src/core/tun.c b/src/core/tun.c index 4c9dec1..33c62d5 100644 --- a/src/core/tun.c +++ b/src/core/tun.c @@ -399,10 +399,15 @@ if (rc < 0) goto err_close_tun; - osmo_fd_register(&tundev->wqueue.bfd); + rc = osmo_fd_register(&tundev->wqueue.bfd); + if (rc < 0) + goto err_unregister_netdev; + tundev->opened = true; return 0; +err_unregister_netdev: + osmo_netdev_unregister(tundev->netdev); err_close_tun: close(tundev->wqueue.bfd.fd); tundev->wqueue.bfd.fd = -1; -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/32649 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: I15bb456c42d5033c9cf4c0937df22fa02b6a6086 Gerrit-Change-Number: 32649 Gerrit-PatchSet: 3 Gerrit-Owner: arehbein &lt;arehbein(a)sysmocom.de&gt; Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria &lt;vyanitskiy(a)sysmocom.de&gt; Gerrit-Reviewer: laforge &lt;laforge(a)osmocom.org&gt; Gerrit-MessageType: merged