From: Shailendra Verma <shailendra.v@samsung.com>
To: Liviu Dudau <liviu.dudau@arm.com>,
        Brian Starkey <brian.starkey@arm.com>,
        Mali DP Maintainers <malidp@foss.arm.com>,
        David Airlie <airlied@linux.ie>, dri-devel@lists.freedesktop.org,
        linux-kernel@vger.kernel.org,
        Shailendra Verma <shailendra.v@samsung.com>,
        Shailendra Verma <shailendra.capricorn@gmail.com>
Cc: vidushi.koul@samsung.com
Subject: [PATCH] Gpu: drm: arm: - Fix possible dereference of NULL
Date: Fri, 11 Nov 2016 17:13:29 +0530
Message-id: <1478864609-32512-1-git-send-email-shailendra.v@samsung.com>
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 856
Lines: 26

From: "Shailendra Verma" <shailendra.v@samsung.com>

There is possible dereference of NULL pointer if kmalloc fails,
When function returns.So return NULL if kmalloc fails.

Signed-off-by: Shailendra Verma <Shailendra.v@samsung.com>
---
 drivers/gpu/drm/arm/malidp_planes.c |    3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/gpu/drm/arm/malidp_planes.c b/drivers/gpu/drm/arm/malidp_planes.c
index 82c193e..f769398 100644
--- a/drivers/gpu/drm/arm/malidp_planes.c
+++ b/drivers/gpu/drm/arm/malidp_planes.c
@@ -54,6 +54,9 @@ struct drm_plane_state *malidp_duplicate_plane_state(struct drm_plane *plane)
 		return NULL;
 
 	state = kmalloc(sizeof(*state), GFP_KERNEL);
+	if (!state)
+		return NULL;
+
 	if (state) {
 		m_state = to_malidp_plane_state(plane->state);
 		__drm_atomic_helper_plane_duplicate_state(plane, &state->base);
-- 
1.7.9.5