laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-e1d/+/33785 ) Change subject: e1d-ts-pipe: Assert that osmo_fd_register() succeeds ...................................................................... e1d-ts-pipe: Assert that osmo_fd_register() succeeds Fixes: CID#307497: Unchecked return value (CHECKED_RETURN) Change-Id: Ie2abd580e37d7d58c507d12105ec2130fb533ed9 --- M src/e1d-ts-pipe.c 1 file changed, 11 insertions(+), 1 deletion(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve tnt: Looks good to me, but someone else must approve laforge: Looks good to me, approved diff --git a/src/e1d-ts-pipe.c b/src/e1d-ts-pipe.c index 99b7c55..45cafb8 100644 --- a/src/e1d-ts-pipe.c +++ b/src/e1d-ts-pipe.c @@ -219,7 +219,7 @@ exit(2); osmo_fd_setup(&ts_ofd, tsfd, OSMO_FD_READ|OSMO_FD_WRITE, ts_fd_cb, NULL, 0); - osmo_fd_register(&ts_ofd); + OSMO_ASSERT(osmo_fd_register(&ts_ofd) == 0); set_nonblock(infd); set_nonblock(outfd); -- To view, visit https://gerrit.osmocom.org/c/osmo-e1d/+/33785 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-e1d Gerrit-Branch: master Gerrit-Change-Id: Ie2abd580e37d7d58c507d12105ec2130fb533ed9 Gerrit-Change-Number: 33785 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge@osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge@osmocom.org> Gerrit-Reviewer: pespin <pespin@sysmocom.de> Gerrit-Reviewer: tnt <tnt@246tNt.com> Gerrit-MessageType: merged