pespin has submitted this change. (
https://gerrit.osmocom.org/c/libosmo-netif/+/38890?usp=email )
Change subject: stream: Explicitly document MSG_NOTIFICATION case in recv() API
......................................................................
stream: Explicitly document MSG_NOTIFICATION case in recv() API
Change-Id: I4fa70ae3f704d2379982153b5630009c13a6f316
---
M src/stream_cli.c
M src/stream_srv.c
2 files changed, 3 insertions(+), 1 deletion(-)
Approvals:
osmith: Looks good to me, but someone else must approve
Jenkins Builder: Verified
laforge: Looks good to me, approved
diff --git a/src/stream_cli.c b/src/stream_cli.c
index 5d8a2c2..b0dac09 100644
--- a/src/stream_cli.c
+++ b/src/stream_cli.c
@@ -1159,7 +1159,7 @@
}
if (ret < 0) {
- if (ret == -EAGAIN)
+ if (ret == -EAGAIN) /* Received MSG_NOTIFICATION from stream_sctp_recvmsg_wrapper() */
return ret;
if (errno == EPIPE || errno == ECONNRESET)
LOGSCLI(cli, LOGL_ERROR, "lost connection with srv\n");
diff --git a/src/stream_srv.c b/src/stream_srv.c
index dad6b7a..9b39bdc 100644
--- a/src/stream_srv.c
+++ b/src/stream_srv.c
@@ -1181,6 +1181,8 @@
}
if (ret < 0) {
+ if (ret == -EAGAIN) /* Received MSG_NOTIFICATION from stream_sctp_recvmsg_wrapper() */
+ return ret;
if (errno == EPIPE || errno == ECONNRESET)
LOGSSRV(conn, LOGL_ERROR, "lost connection with client\n");
return ret;
--
To view, visit
https://gerrit.osmocom.org/c/libosmo-netif/+/38890?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: I4fa70ae3f704d2379982153b5630009c13a6f316
Gerrit-Change-Number: 38890
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>