Change in osmo-pcu[master]: pdch.cpp: Fix logging line format in rcv_block wrong length

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

laforge gerrit-no-reply at lists.osmocom.org
Wed Jul 1 15:43:39 UTC 2020


laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/19065 )

Change subject: pdch.cpp: Fix logging line format in rcv_block wrong length
......................................................................

pdch.cpp: Fix logging line format in rcv_block wrong length

Change-Id: I2f818021cef41ab6f4569cd87026072811853352
---
M src/pdch.cpp
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, approved



diff --git a/src/pdch.cpp b/src/pdch.cpp
index b97f4c7..6bddbfd 100644
--- a/src/pdch.cpp
+++ b/src/pdch.cpp
@@ -754,8 +754,8 @@
 	enum CodingScheme cs = mcs_get_by_size_ul(len);
 	if (!cs) {
 		bts()->do_rate_ctr_inc(CTR_DECODE_ERRORS);
-		LOGP(DRLCMACUL, LOGL_ERROR, "Dropping data block with invalid"
-			"length: %d)\n", len);
+		LOGP(DRLCMACUL, LOGL_ERROR, "Dropping data block with invalid "
+		     "length %d: %s\n", len, osmo_hexdump(data, len));
 		return -EINVAL;
 	}
 

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/19065
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I2f818021cef41ab6f4569cd87026072811853352
Gerrit-Change-Number: 19065
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20200701/c3d99ed1/attachment.htm>


More information about the gerrit-log mailing list