diff options
author | Gustavo A. R. Silva <garsilva@embeddedor.com> | 2017-05-24 05:18:52 -0400 |
---|---|---|
committer | Lee Jones <lee.jones@linaro.org> | 2017-07-06 03:29:11 -0400 |
commit | 7b55033fbed07bad695ba744db3070c1cc351dc4 (patch) | |
tree | 686775423c95b19f1b7ee2e0f608ddc1cdbbe037 | |
parent | 800e54558b0f1444e480ad4eb8d4a774899875d8 (diff) |
mfd: wm831x-spi: 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: 1408830
Signed-off-by: Gustavo A. R. Silva <garsilva@embeddedor.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
-rw-r--r-- | drivers/mfd/wm831x-spi.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/mfd/wm831x-spi.c b/drivers/mfd/wm831x-spi.c index c332e2885b26..018ce652ae57 100644 --- a/drivers/mfd/wm831x-spi.c +++ b/drivers/mfd/wm831x-spi.c | |||
@@ -34,6 +34,10 @@ static int wm831x_spi_probe(struct spi_device *spi) | |||
34 | 34 | ||
35 | if (spi->dev.of_node) { | 35 | if (spi->dev.of_node) { |
36 | of_id = of_match_device(wm831x_of_match, &spi->dev); | 36 | of_id = of_match_device(wm831x_of_match, &spi->dev); |
37 | if (!of_id) { | ||
38 | dev_err(&spi->dev, "Failed to match device\n"); | ||
39 | return -ENODEV; | ||
40 | } | ||
37 | type = (enum wm831x_parent)of_id->data; | 41 | type = (enum wm831x_parent)of_id->data; |
38 | } else { | 42 | } else { |
39 | type = (enum wm831x_parent)id->driver_data; | 43 | type = (enum wm831x_parent)id->driver_data; |