Commit dadd30fc authored Dec 06, 2022 by Jiasheng Jiang Committed by Greg Kroah-Hartman Mar 11, 2023

drm/msm/dpu: Add check for pstates



[ Upstream commit 93340e10b9c5fc86730d149636e0aa8b47bb5a34 ]

As kzalloc may fail and return NULL pointer,
it should be better to check pstates
in order to avoid the NULL pointer dereference.

Fixes: 25fdd593 ("drm/msm: Add SDM845 DPU support")
Signed-off-by: Jiasheng Jiang <jiasheng@iscas.ac.cn>
Reviewed-by: Abhinav Kumar <quic_abhinavk@quicinc.com>
Patchwork: https://patchwork.freedesktop.org/patch/514160/
Link: https://lore.kernel.org/r/20221206080236.43687-1-jiasheng@iscas.ac.cn


Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>

parent a6afb829

drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c

+2 −0

Original line number	Diff line number	Diff line
		@@ -836,6 +836,8 @@ static int dpu_crtc_atomic_check(struct drm_crtc *crtc,
		}

		pstates = kzalloc(sizeof(pstates) DPU_STAGE_MAX * 4, GFP_KERNEL);
		if (!pstates)
		return -ENOMEM;

		dpu_crtc = to_dpu_crtc(crtc);
		cstate = to_dpu_crtc_state(state);