From dce78f7a2bd56c6582c123eb3b070255b5781e2d Mon Sep 17 00:00:00 2001 From: Brandon Bayer Date: Mon, 15 Feb 2016 14:50:21 -0600 Subject: fix: CE910 PDU decode truncation - It turns out the CE910 PDU data lengh is # of septets when sending, but number of octets when receiving. Wonder why I didn't guess that?! --- src/pdu_decode.c | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) (limited to 'src') diff --git a/src/pdu_decode.c b/src/pdu_decode.c index 7c36385..5acb07b 100644 --- a/src/pdu_decode.c +++ b/src/pdu_decode.c @@ -227,16 +227,17 @@ int pdu_decode_user_data(const char *pdu_str, struct pdu_info *pdu, int *nr_octe // CDMA 7-BIT ENCODING } else if (pdu->data_coding.general.alphabet == PDU_ALPHABET_CDMA_DEFAULT) { - if (!strcmp(Global.core.model, "DE910-DUAL")) { - // DE910-DUAL stores data length as # of octets - log_debug("counting PDU length byte as number of octets (DE910-DUAL)"); - *nr_octets = pdu->user_data_len; - pdu->user_data_len = septets_from_octets(pdu->user_data_len); + if (!strcmp(Global.core.model, "LE910-SVG")) { + // LE910-SVG stores data length as # of septets + log_debug("counting PDU length byte as number of septets (LE910-SVG)"); + *nr_octets = octets_from_septets(pdu->user_data_len); } else { - // Other CDMA radios store data length as # of septets - log_debug("counting PDU length byte as number of septets"); - *nr_octets = octets_from_septets(pdu->user_data_len); + // Other CDMA radios store data length as # of octets + // NOTE: CE910 send PDU needs # of septets but receive PDU is in # of octets + log_debug("counting PDU length byte as number of octets"); + *nr_octets = pdu->user_data_len; + pdu->user_data_len = septets_from_octets(pdu->user_data_len); } -- cgit v1.2.3