pespin has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/43246?usp=email )
Change subject: iuup: Fix formatting of log line printing payload checksum error ......................................................................
iuup: Fix formatting of log line printing payload checksum error
The payload CRC is 10bits long, hence 2 bytes are used. Printing 3 digits would be enough, but keep it as 4 digits since anyway we are storing it in uint16_t.
Change-Id: Ife1de59a8257d6c857f724460b546f511db59ebd --- M src/gsm/iuup.c 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved osmith: Looks good to me, approved
diff --git a/src/gsm/iuup.c b/src/gsm/iuup.c index 622302a..b7aa5f9 100644 --- a/src/gsm/iuup.c +++ b/src/gsm/iuup.c @@ -923,7 +923,7 @@ return 0;
payload_crc_err: - LOGP(DLIUUP, LOGL_NOTICE, "Payload Checksum error (pdu type %u): rx 0x%02x vs exp 0x%02x\n", + LOGP(DLIUUP, LOGL_NOTICE, "Payload Checksum error (pdu type %u): rx 0x%04x vs exp 0x%04x\n", pdu_type, payload_crc, payload_crc_computed); return -EIO; }