[PATCH 1/1] drm/msm/a6xx: Fix null pointer access in a6xx_get_indexed_registers

quic_ddhamara at quicinc.com quic_ddhamara at quicinc.com
Wed Jul 13 06:15:28 UTC 2022


From: Akhil P Oommen <quic_akhilpo at quicinc.com>

Fix a null pointer access when memory allocation fails in
a6xx_get_indexed_registers().

Signed-off-by: Akhil P Oommen <quic_akhilpo at quicinc.com>
Signed-off-by: ddhamara <ddhamara at quicinc.com>
---
 drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c b/drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c
index 55f443328d8e..507074f6222c 100644
--- a/drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c
+++ b/drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c
@@ -952,6 +952,12 @@ static void a6xx_get_indexed_registers(struct msm_gpu *gpu,
 	a6xx_get_indexed_regs(gpu, a6xx_state, &a6xx_cp_mempool_indexed,
 		&a6xx_state->indexed_regs[i]);
 
+	if (!a6xx_state->indexed_regs[i].data) {
+		gpu_write(gpu, REG_A6XX_CP_MEM_POOL_SIZE, mempool_size);
+		a6xx_state->nr_indexed_regs = count - 1;
+		return;
+	}
+
 	/*
 	 * Offset 0x2000 in the mempool is the size - copy the saved size over
 	 * so the data is consistent
-- 
2.37.0



More information about the dri-devel mailing list