fixeria has submitted this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/42405?usp=email )

Change subject: stats: osmo_stats_reporter_udp_open(): fix unconditional jump
......................................................................

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(-)

Approvals:
  laforge: Looks good to me, but someone else must approve
  Jenkins Builder: Verified
  pespin: Looks good to me, approved




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 https://gerrit.osmocom.org/c/libosmocore/+/42405?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: merged
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ifb4842ea11e4df0cae8733438b3812a086396b99
Gerrit-Change-Number: 42405
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>

Reply via email to