pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/libosmo-gprs/+/32568 )

Change subject: llc: Log received wrong FCS field
......................................................................

llc: Log received wrong FCS field

Change-Id: I448680ee82fca226df4d33650537167b2e496e24
---
M src/llc/llc.c
1 file changed, 11 insertions(+), 1 deletion(-)

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




diff --git a/src/llc/llc.c b/src/llc/llc.c
index 3831f8a..ef52ec3 100644
--- a/src/llc/llc.c
+++ b/src/llc/llc.c
@@ -754,7 +754,8 @@
        if (~pdu_dec->flags & OSMO_GPRS_LLC_PDU_F_PROT_MODE)
                crc_length = OSMO_MIN(crc_length, UI_HDR_LEN + N202);
        if (pdu_dec->fcs != gprs_llc_fcs(ll_pdu, crc_length)) {
-               LOGLLE(lle, LOGL_INFO, "Dropping frame with invalid FCS\n");
+               LOGLLE(lle, LOGL_NOTICE, "Dropping frame with invalid FCS 
0x%06x vs exp 0x%06x\n",
+                      pdu_dec->fcs, gprs_llc_fcs(ll_pdu, crc_length));
                return -EIO;
        }


--
To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/32568
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: I448680ee82fca226df4d33650537167b2e496e24
Gerrit-Change-Number: 32568
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged

Reply via email to