fixeria submitted this change.

View Change

Approvals: laforge: Looks good to me, but someone else must approve Jenkins Builder: Verified pespin: Looks good to me, approved
stats: osmo_stats_reporter_udp_open(): fix unconditional jump

Change-Id: Ifb4842ea11e4df0cae8733438b3812a086396b99
Fixes: cc3694b65 ("Fix build for OSX")
---
M src/core/stats.c
1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/core/stats.c b/src/core/stats.c
index 16e6f62..617c1a5 100644
--- a/src/core/stats.c
+++ b/src/core/stats.c
@@ -471,7 +471,8 @@
static int val = 1;

rc = setsockopt(sock, SOL_SOCKET, SO_NOSIGPIPE, (void*)&val, sizeof(val));
- goto failed;
+ if (rc == -1)
+ goto failed;
}
#endif
if (srep->bind_addr_len > 0) {

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

Gerrit-MessageType: merged
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ifb4842ea11e4df0cae8733438b3812a086396b99
Gerrit-Change-Number: 42405
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>