diff options
| -rw-r--r-- | drivers/atmodem/sms.c | 19 |
1 files changed, 15 insertions, 4 deletions
diff --git a/drivers/atmodem/sms.c b/drivers/atmodem/sms.c index 439beeaf..74e80764 100644 --- a/drivers/atmodem/sms.c +++ b/drivers/atmodem/sms.c @@ -283,16 +283,27 @@ static void at_cds_notify(GAtResult *result, gpointer user_data) { struct ofono_sms *sms = user_data; struct sms_data *data = ofono_sms_get_data(sms); - int pdulen; - const char *pdu; + long pdu_len; + int tpdu_len; + const char *hexpdu; + unsigned char pdu[176]; char buf[256]; - if (!at_parse_pdu_common(result, "+CDS:", &pdu, &pdulen)) { + if (!at_parse_pdu_common(result, "+CDS:", &hexpdu, &tpdu_len)) { ofono_error("Unable to parse CDS notification"); return; } - DBG("Got new Status-Report PDU via CDS: %s, %d", pdu, pdulen); + if (strlen(hexpdu) > sizeof(pdu) * 2) { + ofono_error("Bad PDU length in CDS notification"); + return; + } + + DBG("Got new Status-Report PDU via CDS: %s, %d", hexpdu, tpdu_len); + + /*Decode pdu and notify about new SMS status report*/ + decode_hex_own_buf(hexpdu, -1, &pdu_len, 0, pdu); + ofono_sms_status_notify(sms, pdu, pdu_len, tpdu_len); /* We must acknowledge the PDU using CNMA */ if (data->cnma_ack_pdu) |