summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--stack/mcap/mca_cact.c20
1 files changed, 17 insertions, 3 deletions
diff --git a/stack/mcap/mca_cact.c b/stack/mcap/mca_cact.c
index 6be5b4f4b..2541f4898 100644
--- a/stack/mcap/mca_cact.c
+++ b/stack/mcap/mca_cact.c
@@ -480,13 +480,27 @@ void mca_ccb_hdl_rsp(tMCA_CCB *p_ccb, tMCA_CCB_EVT *p_data)
tMCA_RESULT result = MCA_BAD_HANDLE;
tMCA_TC_TBL *p_tbl;
- if (p_ccb->p_tx_req)
+ if (p_pkt->len < sizeof(evt_data.hdr.op_code) +
+ sizeof(evt_data.rsp.rsp_code) +
+ sizeof(evt_data.hdr.mdl_id))
+ {
+ android_errorWriteLog(0x534e4554, "116319076");
+ MCA_TRACE_ERROR("%s: Response packet is too short", __func__);
+ }
+ else if (p_ccb->p_tx_req)
{
/* verify that the received response matches the sent request */
p = (UINT8 *)(p_pkt + 1) + p_pkt->offset;
evt_data.hdr.op_code = *p++;
- if ((evt_data.hdr.op_code == 0) ||
- ((p_ccb->p_tx_req->op_code + 1) == evt_data.hdr.op_code))
+ if ((evt_data.hdr.op_code == MCA_OP_MDL_CREATE_RSP) &&
+ (p_pkt->len <
+ sizeof(evt_data.hdr.op_code) + sizeof(evt_data.rsp.rsp_code) +
+ sizeof(evt_data.hdr.mdl_id) + sizeof(evt_data.create_cfm.cfg)))
+ {
+ android_errorWriteLog(0x534e4554, "116319076");
+ MCA_TRACE_ERROR("%s: MDL Create Response packet is too short", __func__);
+ } else if ((evt_data.hdr.op_code == 0) ||
+ ((p_ccb->p_tx_req->op_code + 1) == evt_data.hdr.op_code))
{
evt_data.rsp.rsp_code = *p++;
mca_stop_timer(p_ccb);