Max has submitted this change and it was merged. (
https://gerrit.osmocom.org/12484 )
Change subject: msgb: fix debug print
......................................................................
msgb: fix debug print
Since osmo_hexdump() use static buffers we can't re-use pointers to it
after subsequent osmo_hexdump() calls. Let's print data used for
comparison directly instead.
Change-Id: I24dc3fad6f64ef788da9b7d790f9d5f689190c42
---
M src/msgb.c
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Max: Looks good to me, approved
Pau Espin Pedrol: Looks good to me, but someone else must approve
osmith: Looks good to me, but someone else must approve
Jenkins Builder: Verified
diff --git a/src/msgb.c b/src/msgb.c
index 9cd59c6..3902f6e 100644
--- a/src/msgb.c
+++ b/src/msgb.c
@@ -260,7 +260,7 @@
} else
LOGPC(DLGLOBAL, LOGL_FATAL, ".. ");
- LOGPC(DLGLOBAL, LOGL_FATAL, " msgb %s\n", m_dump);
+ LOGPC(DLGLOBAL, LOGL_FATAL, " msgb %s\n", osmo_hexdump(m_data, len));
return false;
}
--
To view, visit https://gerrit.osmocom.org/12484
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I24dc3fad6f64ef788da9b7d790f9d5f689190c42
Gerrit-Change-Number: 12484
Gerrit-PatchSet: 2
Gerrit-Owner: Max <[email protected]>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Max <[email protected]>
Gerrit-Reviewer: Pau Espin Pedrol <[email protected]>
Gerrit-Reviewer: Stefan Sperling <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
