msuraev submitted this change.

View Change

Approvals: Jenkins Builder: Verified fixeria: Looks good to me, but someone else must approve laforge: Looks good to me, approved
telnet_init_dynif: propagate error

Return actual error from osmo_sock_init_ofd() to the caller.

Change-Id: Ib09f4315b3868ec031ecbced8a63069e7c2de7da
---
M src/vty/telnet_interface.c
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/vty/telnet_interface.c b/src/vty/telnet_interface.c
index 132cd2e..1758860 100644
--- a/src/vty/telnet_interface.c
+++ b/src/vty/telnet_interface.c
@@ -94,7 +94,7 @@
if (rc < 0) {
LOGP(DLGLOBAL, LOGL_ERROR, "Cannot bind telnet at %s %d\n",
ip, port);
- return -1;
+ return rc;
}

LOGP(DLGLOBAL, LOGL_NOTICE, "Available via telnet %s %d\n", ip, port);

3 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one.

To view, visit change 30442. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ib09f4315b3868ec031ecbced8a63069e7c2de7da
Gerrit-Change-Number: 30442
Gerrit-PatchSet: 4
Gerrit-Owner: msuraev <msuraev@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: msuraev <msuraev@sysmocom.de>
Gerrit-MessageType: merged