This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.
Harald Welte gerrit-no-reply at lists.osmocom.orgHarald Welte has submitted this change and it was merged.
Change subject: stream.c: Fix SCTP_NODELAY setting
......................................................................
stream.c: Fix SCTP_NODELAY setting
We were using the wrong variable when setting the SCTP_NODELAY,
resulting in the setsoctkopt() fialing.
Change-Id: Ic04cb8bb5ff41f177f7f5b7f7e2a2ecc686dd4c0
---
M src/stream.c
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Harald Welte: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/src/stream.c b/src/stream.c
index 653f26b..ed73a62 100644
--- a/src/stream.c
+++ b/src/stream.c
@@ -551,7 +551,7 @@
}
if (link->flags & OSMO_STREAM_SRV_F_NODELAY) {
- ret = setsockopt_nodelay(ret, link->proto, 1);
+ ret = setsockopt_nodelay(sock_fd, link->proto, 1);
if (ret < 0)
goto error_close_socket;
}
--
To view, visit https://gerrit.osmocom.org/2380
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic04cb8bb5ff41f177f7f5b7f7e2a2ecc686dd4c0
Gerrit-PatchSet: 1
Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Owner: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder