diff options
| author | Axel Lin <axel.lin@ingics.com> | 2016-03-21 13:03:50 +0100 |
|---|---|---|
| committer | Linus Walleij <linus.walleij@linaro.org> | 2016-03-30 10:39:39 +0200 |
| commit | 8d8ee18cc707f498b6dad18915576f23b00bcff8 (patch) | |
| tree | bb2862066c6a7494e3e1b22d56961db3cf740f0b /drivers/gpio | |
| parent | gpio: menz127: Drop lock field from struct men_z127_gpio (diff) | |
| download | linux-8d8ee18cc707f498b6dad18915576f23b00bcff8.tar.xz linux-8d8ee18cc707f498b6dad18915576f23b00bcff8.zip | |
gpio: xgene: Prevent NULL pointer dereference
platform_get_resource() can return NULL, thus add NULL test to prevent NULL
pointer dereference.
Signed-off-by: Axel Lin <axel.lin@ingics.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'drivers/gpio')
| -rw-r--r-- | drivers/gpio/gpio-xgene.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/gpio/gpio-xgene.c b/drivers/gpio/gpio-xgene.c index c0aa387664bf..0dc916191689 100644 --- a/drivers/gpio/gpio-xgene.c +++ b/drivers/gpio/gpio-xgene.c @@ -173,6 +173,11 @@ static int xgene_gpio_probe(struct platform_device *pdev) } res = platform_get_resource(pdev, IORESOURCE_MEM, 0); + if (!res) { + err = -EINVAL; + goto err; + } + gpio->base = devm_ioremap_nocache(&pdev->dev, res->start, resource_size(res)); if (!gpio->base) { |