drivers/video/backlight/lm3630a_bl.c: potential NULL deref in probe()
authorDan Carpenter <dan.carpenter@oracle.com>
Tue, 12 Nov 2013 23:09:00 +0000 (15:09 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Wed, 13 Nov 2013 03:09:15 +0000 (12:09 +0900)
We dereference "pdata" later in the function so we can't leave it as NULL.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Jingoo Han <jg1.han@samsung.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/video/backlight/lm3630a_bl.c

index 200dca04383f3c3872b527e7f6228461e5635222..d719a89a4381c06535c42d4127ee8e37f8e25d42 100644 (file)
@@ -389,22 +389,21 @@ static int lm3630a_probe(struct i2c_client *client,
 
        i2c_set_clientdata(client, pchip);
        if (pdata == NULL) {
-               pchip->pdata = devm_kzalloc(pchip->dev,
-                                           sizeof(struct
-                                                  lm3630a_platform_data),
-                                           GFP_KERNEL);
-               if (pchip->pdata == NULL)
+               pdata = devm_kzalloc(pchip->dev,
+                                    sizeof(struct lm3630a_platform_data),
+                                    GFP_KERNEL);
+               if (pdata == NULL)
                        return -ENOMEM;
                /* default values */
-               pchip->pdata->leda_ctrl = LM3630A_LEDA_ENABLE;
-               pchip->pdata->ledb_ctrl = LM3630A_LEDB_ENABLE;
-               pchip->pdata->leda_max_brt = LM3630A_MAX_BRIGHTNESS;
-               pchip->pdata->ledb_max_brt = LM3630A_MAX_BRIGHTNESS;
-               pchip->pdata->leda_init_brt = LM3630A_MAX_BRIGHTNESS;
-               pchip->pdata->ledb_init_brt = LM3630A_MAX_BRIGHTNESS;
-       } else {
-               pchip->pdata = pdata;
+               pdata->leda_ctrl = LM3630A_LEDA_ENABLE;
+               pdata->ledb_ctrl = LM3630A_LEDB_ENABLE;
+               pdata->leda_max_brt = LM3630A_MAX_BRIGHTNESS;
+               pdata->ledb_max_brt = LM3630A_MAX_BRIGHTNESS;
+               pdata->leda_init_brt = LM3630A_MAX_BRIGHTNESS;
+               pdata->ledb_init_brt = LM3630A_MAX_BRIGHTNESS;
        }
+       pchip->pdata = pdata;
+
        /* chip initialize */
        rval = lm3630a_chip_init(pchip);
        if (rval < 0) {