diff options
| author | Gustavo A. R. Silva | 2017-05-24 04:26:23 -0500 |
|---|---|---|
| committer | Lee Jones | 2017-07-06 08:29:11 +0100 |
| commit | 800e54558b0f1444e480ad4eb8d4a774899875d8 (patch) | |
| tree | c1b6ee236059d97f42132d214a3229eee2f4ffef /drivers/mfd | |
| parent | 87d284443d071dc70344dda4b2fb43723686acdb (diff) | |
mfd: wm831x-i2c: Add NULL check before pointer dereference
Add NULL check before dereferencing pointer of_id in order to avoid
a potential NULL pointer dereference.
Addresses-Coverity-ID: 1408829
Signed-off-by: Gustavo A. R. Silva <garsilva@embeddedor.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
Diffstat (limited to 'drivers/mfd')
| -rw-r--r-- | drivers/mfd/wm831x-i2c.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/mfd/wm831x-i2c.c b/drivers/mfd/wm831x-i2c.c index 781af060f32d..22f7054d1b28 100644 --- a/drivers/mfd/wm831x-i2c.c +++ b/drivers/mfd/wm831x-i2c.c @@ -37,6 +37,10 @@ static int wm831x_i2c_probe(struct i2c_client *i2c, if (i2c->dev.of_node) { of_id = of_match_device(wm831x_of_match, &i2c->dev); + if (!of_id) { + dev_err(&i2c->dev, "Failed to match device\n"); + return -ENODEV; + } type = (enum wm831x_parent)of_id->data; } else { type = (enum wm831x_parent)id->driver_data; |