mfd: Check for twl4030-madc NULL pointer
authorKyle Manna <kyle@kylemanna.com>
Fri, 12 Aug 2011 03:33:14 +0000 (22:33 -0500)
committerSamuel Ortiz <sameo@linux.intel.com>
Tue, 6 Sep 2011 14:37:59 +0000 (16:37 +0200)
If the twl4030-madc device wasn't registered, and another device, such
as twl4030-madc-hwmon, calls twl4030_madc_conversion() a NULL pointer is
dereferenced.

Signed-off-by: Kyle Manna <kyle@kylemanna.com>
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
drivers/mfd/twl4030-madc.c

index cb44b53..7cbf2aa 100644 (file)
@@ -510,8 +510,9 @@ int twl4030_madc_conversion(struct twl4030_madc_request *req)
        u8 ch_msb, ch_lsb;
        int ret;
 
-       if (!req)
+       if (!req || !twl4030_madc)
                return -EINVAL;
+
        mutex_lock(&twl4030_madc->lock);
        if (req->method < TWL4030_MADC_RT || req->method > TWL4030_MADC_SW2) {
                ret = -EINVAL;