From b778846e50b7f89ef90092f5d454bc73c651c514 Mon Sep 17 00:00:00 2001 From: Aditya Bavanari Date: Tue, 2 Jul 2019 20:08:14 +0530 Subject: [PATCH] dsp: Remove param_size from pkt_size of apr header As param_size is included in apr header pkt_size, out of bounds access occurs in glink. Remove the param size addition to fix this issue. CRs-Fixed: 2472208 Change-Id: If8b34aeacd3bc9ba67ac9276eb1a8ebf0933f9f9 Signed-off-by: Aditya Bavanari --- dsp/q6afe.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dsp/q6afe.c b/dsp/q6afe.c index 0fe38885f7c9..56b50ef00c45 100644 --- a/dsp/q6afe.c +++ b/dsp/q6afe.c @@ -1292,7 +1292,7 @@ static int q6afe_get_params_v2(u16 port_id, int index, afe_get_param.apr_hdr.hdr_field = APR_HDR_FIELD(APR_MSG_TYPE_SEQ_CMD, APR_HDR_LEN(APR_HDR_SIZE), APR_PKT_VER); - afe_get_param.apr_hdr.pkt_size = sizeof(afe_get_param) + param_size; + afe_get_param.apr_hdr.pkt_size = sizeof(afe_get_param); afe_get_param.apr_hdr.src_port = 0; afe_get_param.apr_hdr.dest_port = 0; afe_get_param.apr_hdr.token = index;