Commit 6b808223 authored by 蒋家盛's avatar 蒋家盛 Committed by Hans Verkuil
Browse files

media: coda: Add check for dcoda_iram_alloc 



As the coda_iram_alloc may return NULL pointer,
it should be better to check the return value
in order to avoid NULL poineter dereference,
same as the others.

Fixes: b313bcc9 ("[media] coda: simplify IRAM setup")
Signed-off-by: default avatarJiasheng Jiang <jiasheng@iscas.ac.cn>
Signed-off-by: default avatarHans Verkuil <hverkuil-cisco@xs4all.nl>
parent a8a0bc81

drivers/media/platform/chips-media/coda-bit.c

+2 −2
Original line number Diff line number Diff line
@@ -854,7 +854,7 @@ static void coda_setup_iram(struct coda_ctx *ctx) 
		/* Only H.264BP and H.263P3 are considered */

		iram_info->buf_dbk_y_use = coda_iram_alloc(iram_info, w64);

		iram_info->buf_dbk_c_use = coda_iram_alloc(iram_info, w64);

		if (!iram_info->buf_dbk_c_use)

		if (!iram_info->buf_dbk_y_use || !iram_info->buf_dbk_c_use)

			goto out;

		iram_info->axi_sram_use |= dbk_bits;
@@ -878,7 +878,7 @@ static void coda_setup_iram(struct coda_ctx *ctx) 


		iram_info->buf_dbk_y_use = coda_iram_alloc(iram_info, w128);

		iram_info->buf_dbk_c_use = coda_iram_alloc(iram_info, w128);

		if (!iram_info->buf_dbk_c_use)

		if (!iram_info->buf_dbk_y_use || !iram_info->buf_dbk_c_use)

			goto out;

		iram_info->axi_sram_use |= dbk_bits;