pespin submitted this change.

View Change

Approvals: osmith: Looks good to me, approved Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve
stream: Improve error logging sctp_recvmsg without sctp_sndrcvinfo

Related: OS#6775
Change-Id: I4ef03a2505ca7409d50fd34371ae4ec40bebb40f
---
M src/stream.c
1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/stream.c b/src/stream.c
index fb15f93..f0902a4 100644
--- a/src/stream.c
+++ b/src/stream.c
@@ -266,8 +266,11 @@
return ret;
}

- if (OSMO_UNLIKELY(ret > 0 && !sinfo))
- LOGP(DLINP, LOGL_ERROR, "%s sctp_recvmsg without SNDRCV cmsg?!?\n", log_pfx);
+ if (OSMO_UNLIKELY(ret > 0 && !sinfo)) {
+ char buf[512];
+ LOGP(DLINP, LOGL_ERROR, "%s sctp_recvmsg without SNDRCV cmsg?!? ret=%d, flags=0x%x, %s\n",
+ log_pfx, ret, flags, msgb_hexdump_buf(buf, sizeof(buf), msg));
+ }

return ret;
}

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

Gerrit-MessageType: merged
Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: I4ef03a2505ca7409d50fd34371ae4ec40bebb40f
Gerrit-Change-Number: 40122
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>