Commit ddbcaa93 authored Feb 22, 2021 by Jayaprakash

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 <jmadiset@codeaurora.org>

parent 5d955fca

msm/msm_drv.c

+3 −3

Original line number	Diff line number	Diff line
		/*
		* Copyright (c) 2016-2020, The Linux Foundation. All rights reserved.
		* Copyright (c) 2016-2021, The Linux Foundation. All rights reserved.
		* Copyright (C) 2013 Red Hat
		* Author: Rob Clark <robdclark@gmail.com>
		*
		@@ -886,13 +886,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) {