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

Commit 1546c9a1 authored by Saurav Kumar's avatar Saurav Kumar Committed by Gerrit - the friendly Code Review server
Browse files

dsp: add change to fix use-after-free issue 

Add change to properly handle the pointers by setting them to
NULL after free and adding some null checks before dereferencing.

Change-Id: Ic2f3ef5631d0f4e4191bf772eb0b3f0b825044a6
parent dec796e3

dsp/msm_audio_ion_vm.c

+8 −1
Original line number Diff line number Diff line
@@ -93,7 +93,7 @@ static int msm_audio_dma_buf_map(struct dma_buf *dma_buf, 
				 bool cma_mem)

{



	struct msm_audio_alloc_data *alloc_data;

	struct msm_audio_alloc_data *alloc_data = NULL;

	struct device *cb_dev;

	unsigned long ionflag = 0;

	int rc = 0;
@@ -162,6 +162,7 @@ static int msm_audio_dma_buf_map(struct dma_buf *dma_buf, 
		       alloc_data->attach);

free_alloc_data:

	kfree(alloc_data);

	alloc_data = NULL;



	return rc;

}
@@ -204,6 +205,7 @@ static int msm_audio_dma_buf_unmap(struct dma_buf *dma_buf, bool cma_mem) 


			list_del(&(alloc_data->list));

			kfree(alloc_data);

			alloc_data = NULL;

			break;

		}

	}
@@ -507,6 +509,11 @@ static int msm_audio_ion_map_buf(struct dma_buf *dma_buf, dma_addr_t *paddr, 
{

	int rc = 0;



	if (!dma_buf || !paddr || !vaddr || !plen) {

		pr_err("%s: Invalid params\n", __func__);

		return -EINVAL;

	}



	rc = msm_audio_ion_get_phys(dma_buf, paddr, plen);

	if (rc) {

		pr_err("%s: ION Get Physical for AUDIO failed, rc = %d\n",