pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/libosmo-netif/+/40122?usp=email )

Change subject: stream: Improve error logging sctp_recvmsg without 
sctp_sndrcvinfo
......................................................................

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

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




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 https://gerrit.osmocom.org/c/libosmo-netif/+/40122?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: I4ef03a2505ca7409d50fd34371ae4ec40bebb40f
Gerrit-Change-Number: 40122
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pes...@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <lafo...@osmocom.org>
Gerrit-Reviewer: osmith <osm...@sysmocom.de>
Gerrit-Reviewer: pespin <pes...@sysmocom.de>

Reply via email to