[PATCH 4/6] imx-drm: parallel-display: Check 'dmfc' pointer first

2014-02-26 Thread Fabio Estevam
From: Fabio Estevam fabio.este...@freescale.com

Fix the following static checker warnings:

drivers/staging/imx-drm/parallel-display.c:64 imx_pd_connector_get_modes() 
error: potential null dereference 'mode'.  (drm_mode_create returns null)
drivers/staging/imx-drm/parallel-display.c:73 imx_pd_connector_get_modes() 
error: potential null dereference 'mode'.  (drm_mode_create returns null)

Reported-by: Dan Carpenter dan.carpen...@oracle.com
Signed-off-by: Fabio Estevam fabio.este...@freescale.com
---
 drivers/staging/imx-drm/parallel-display.c | 4 
 1 file changed, 4 insertions(+)

diff --git a/drivers/staging/imx-drm/parallel-display.c 
b/drivers/staging/imx-drm/parallel-display.c
index d610f07..08e47aa 100644
--- a/drivers/staging/imx-drm/parallel-display.c
+++ b/drivers/staging/imx-drm/parallel-display.c
@@ -61,6 +61,8 @@ static int imx_pd_connector_get_modes(struct drm_connector 
*connector)
 
if (imxpd-mode_valid) {
struct drm_display_mode *mode = drm_mode_create(connector-dev);
+   if (!mode)
+   return -EINVAL;
drm_mode_copy(mode, imxpd-mode);
mode-type |= DRM_MODE_TYPE_DRIVER | DRM_MODE_TYPE_PREFERRED,
drm_mode_probed_add(connector, mode);
@@ -69,6 +71,8 @@ static int imx_pd_connector_get_modes(struct drm_connector 
*connector)
 
if (np) {
struct drm_display_mode *mode = drm_mode_create(connector-dev);
+   if (!mode)
+   return -EINVAL;
of_get_drm_display_mode(np, imxpd-mode, OF_USE_NATIVE_MODE);
drm_mode_copy(mode, imxpd-mode);
mode-type |= DRM_MODE_TYPE_DRIVER | DRM_MODE_TYPE_PREFERRED,
-- 
1.8.1.2

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


Re: [PATCH 4/6] imx-drm: parallel-display: Check 'dmfc' pointer first

2014-02-26 Thread Dan Carpenter
On Wed, Feb 26, 2014 at 06:44:37PM -0300, Fabio Estevam wrote:
 From: Fabio Estevam fabio.este...@freescale.com
 
 Fix the following static checker warnings:
 
 drivers/staging/imx-drm/parallel-display.c:64 imx_pd_connector_get_modes() 
 error: potential null dereference 'mode'.  (drm_mode_create returns null)
 drivers/staging/imx-drm/parallel-display.c:73 imx_pd_connector_get_modes() 
 error: potential null dereference 'mode'.  (drm_mode_create returns null)
 

Cut and paste error in the subject.

regards,
dan carpenter


___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel