pespin submitted this change.

View Change

Approvals: osmith: Looks good to me, but someone else must approve Jenkins Builder: Verified laforge: Looks good to me, approved
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(-)

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 change 38890. To unsubscribe, or for help writing mail filters, visit settings.

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@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>