From ef3a66389ba05b7063395021b893b3a8a1069fd4 Mon Sep 17 00:00:00 2001 From: Jayaprakash Date: Mon, 22 Feb 2021 13:53:45 +0530 Subject: [PATCH] disp: msm: sde: fix null dereference in drm_atomic_get_property Add changes to fix the null dereference in drm_atomic_get_property caused by connector->state being NULL. This change allows the drm_mode_config_reset operation to happen before drm_dev_register to avoid this. In current scenario, connector->state->crtc is being accessed due to call to drm_mode_getconnector ioctl with the drm_mode_config_reset operation pending. Change-Id: I374d9485819fad85100d1837f4ae22fc2a3ccc40 Signed-off-by: Jayaprakash --- msm/msm_drv.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/msm/msm_drv.c b/msm/msm_drv.c index 7d0b1e07ea..0334a98960 100644 --- a/msm/msm_drv.c +++ b/msm/msm_drv.c @@ -846,13 +846,13 @@ static int msm_drm_component_init(struct device *dev) } } + drm_mode_config_reset(ddev); + ret = drm_dev_register(ddev, 0); if (ret) goto fail; priv->registered = true; - drm_mode_config_reset(ddev); - if (kms && kms->funcs && kms->funcs->cont_splash_config) { ret = kms->funcs->cont_splash_config(kms, NULL); if (ret) {