laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-e1-recorder/+/31183 ) Change subject: handle telnet_init_default() error case ...................................................................... handle telnet_init_default() error case Closes: CID#307532 Change-Id: I0210b535b844529e217fd5d6ffb01f94a262dc9c --- M src/e1_recorder.c 1 file changed, 6 insertions(+), 2 deletions(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/e1_recorder.c b/src/e1_recorder.c index 68d4543..996ede4 100644 --- a/src/e1_recorder.c +++ b/src/e1_recorder.c @@ -207,8 +207,12 @@ exit(1); } - /* start telne tafte reading config for vty_get_bind_adr() */ - telnet_init_default(rec_tall_ctx, NULL, 4444); + /* start telnet after reading config */ + rc = telnet_init_default(rec_tall_ctx, NULL, 4444); + if (rc < 0) { + fprintf(stderr, "Cannot open telnet server port: %s\n", strerror(errno)); + exit(1); + } while (1) { osmo_select_main(0); -- To view, visit https://gerrit.osmocom.org/c/osmo-e1-recorder/+/31183 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-e1-recorder Gerrit-Branch: master Gerrit-Change-Id: I0210b535b844529e217fd5d6ffb01f94a262dc9c Gerrit-Change-Number: 31183 Gerrit-PatchSet: 2 Gerrit-Owner: laforge &lt;laforge(a)osmocom.org&gt; Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge &lt;laforge(a)osmocom.org&gt; Gerrit-CC: pespin &lt;pespin(a)sysmocom.de&gt; Gerrit-MessageType: merged