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

Commit b13a605f authored by Xiaojun Sang's avatar Xiaojun Sang
Browse files

dsp: ion: unmap DMA buffer after kernel ion map failure 



DMA buffer does not unmap after kernel ion map failure. It leads
to unexpected vaddr and memory leak.

Change-Id: I363dd9178a0bcbca601180bfd0a7bd7ef2430d3e
Signed-off-by: default avatarXiaojun Sang <xsang@codeaurora.org>
parent 923243ad

dsp/msm_audio_ion.c

+4 −6
Original line number Diff line number Diff line
@@ -331,6 +331,7 @@ static int msm_audio_ion_map_buf(struct dma_buf *dma_buf, dma_addr_t *paddr, 
	if (rc) {

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

				__func__, rc);

		dma_buf_put(dma_buf);

		goto err;

	}
@@ -338,6 +339,7 @@ static int msm_audio_ion_map_buf(struct dma_buf *dma_buf, dma_addr_t *paddr, 
	if (IS_ERR_OR_NULL(*vaddr)) {

		pr_err("%s: ION memory mapping for AUDIO failed\n", __func__);

		rc = -ENOMEM;

		msm_audio_dma_buf_unmap(dma_buf);

		goto err;

	}
@@ -399,17 +401,13 @@ int msm_audio_ion_alloc(struct dma_buf **dma_buf, size_t bufsz, 
	rc = msm_audio_ion_map_buf(*dma_buf, paddr, plen, vaddr);

	if (rc) {

		pr_err("%s: failed to map ION buf, rc = %d\n", __func__, rc);

		goto err_dma_buf;

		goto err;

	}

	pr_debug("%s: mapped address = %pK, size=%zd\n", __func__,

		*vaddr, bufsz);



	memset(*vaddr, 0, bufsz);



	return rc;



err_dma_buf:

	dma_buf_put(*dma_buf);

err:

	return rc;

}
@@ -503,7 +501,7 @@ int msm_audio_ion_import(struct dma_buf **dma_buf, int fd, 
	rc = msm_audio_ion_map_buf(*dma_buf, paddr, plen, vaddr);

	if (rc) {

		pr_err("%s: failed to map ION buf, rc = %d\n", __func__, rc);

		goto err_ion_flag;

		goto err;

	}

	pr_debug("%s: mapped address = %pK, size=%zd\n", __func__,

		*vaddr, bufsz);