Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 9c429506 authored by kunleiz's avatar kunleiz
Browse files

rtac: add size check when reading cal data kvaddr buffer 



Add size check to ensure cal data bytes size fits inside
the cal data when copying to user space buffer.

CRs-Fixed: 2110256
Change-Id: I511999984684a9db4aaf1cf2c65eb1495c36980f
Signed-off-by: default avatarkunleiz <kunleiz@codeaurora.org>
parent 53a557ad

dsp/rtac.c

+32 −0
Original line number Diff line number Diff line
@@ -909,6 +909,14 @@ int send_adm_apr(void *buf, u32 opcode) 
		bytes_returned = ((u32 *)rtac_cal[ADM_RTAC_CAL].cal_data.

			kvaddr)[2] + 3 * sizeof(u32);



		if (bytes_returned > rtac_cal[ADM_RTAC_CAL].

			map_data.map_size) {

			pr_err("%s: Invalid data size = %d\n",

				__func__, bytes_returned);

			result = -EINVAL;

			goto err;

		}



		if (bytes_returned > user_buf_size) {

			pr_err("%s: User buf not big enough, size = 0x%x, returned size = 0x%x\n",

				__func__, user_buf_size, bytes_returned);
@@ -1132,6 +1140,14 @@ int send_rtac_asm_apr(void *buf, u32 opcode) 
		bytes_returned = ((u32 *)rtac_cal[ASM_RTAC_CAL].cal_data.

			kvaddr)[2] + 3 * sizeof(u32);



		if (bytes_returned > rtac_cal[ASM_RTAC_CAL].

			map_data.map_size) {

			pr_err("%s: Invalid data size = %d\n",

				__func__, bytes_returned);

			result = -EINVAL;

			goto err;

		}



		if (bytes_returned > user_buf_size) {

			pr_err("%s: User buf not big enough, size = 0x%x, returned size = 0x%x\n",

				__func__, user_buf_size, bytes_returned);
@@ -1392,6 +1408,14 @@ static int send_rtac_afe_apr(void *buf, uint32_t opcode) 
		bytes_returned = get_resp->param_size +

				sizeof(struct afe_port_param_data_v2);



		if (bytes_returned > rtac_cal[AFE_RTAC_CAL].

			map_data.map_size) {

			pr_err("%s: Invalid data size = %d\n",

				__func__, bytes_returned);

			result = -EINVAL;

			goto err;

		}



		if (bytes_returned > user_afe_buf.buf_size) {

			pr_err("%s: user size = 0x%x, returned size = 0x%x\n",

				__func__, user_afe_buf.buf_size,
@@ -1617,6 +1641,14 @@ int send_voice_apr(u32 mode, void *buf, u32 opcode) 
		bytes_returned = ((u32 *)rtac_cal[VOICE_RTAC_CAL].cal_data.

			kvaddr)[2] + 3 * sizeof(u32);



		if (bytes_returned > rtac_cal[VOICE_RTAC_CAL].

			map_data.map_size) {

			pr_err("%s: Invalid data size = %d\n",

				__func__, bytes_returned);

			result = -EINVAL;

			goto err;

		}



		if (bytes_returned > user_buf_size) {

			pr_err("%s: User buf not big enough, size = 0x%x, returned size = 0x%x\n",

				__func__, user_buf_size, bytes_returned);