diff options
author | Kangjie Lu <kjlu@umn.edu> | 2019-03-24 18:10:02 -0500 |
---|---|---|
committer | Bartosz Golaszewski <bgolaszewski@baylibre.com> | 2019-03-25 09:26:48 +0100 |
commit | 6cf4511e9729c00a7306cf94085f9cc3c52ee723 (patch) | |
tree | 5725259b33dc4479a21502412c95b2299ac06fc2 /drivers/gpio/gpio-aspeed.c | |
parent | b45a02e13ee74b6fde56df4d76786058821a3aba (diff) | |
download | lwn-6cf4511e9729c00a7306cf94085f9cc3c52ee723.tar.gz lwn-6cf4511e9729c00a7306cf94085f9cc3c52ee723.zip |
gpio: aspeed: fix a potential NULL pointer dereference
In case devm_kzalloc, the patch returns ENOMEM to avoid potential
NULL pointer dereference.
Signed-off-by: Kangjie Lu <kjlu@umn.edu>
Reviewed-by: Andrew Jeffery <andrew@aj.id.au>
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Diffstat (limited to 'drivers/gpio/gpio-aspeed.c')
-rw-r--r-- | drivers/gpio/gpio-aspeed.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/gpio/gpio-aspeed.c b/drivers/gpio/gpio-aspeed.c index 854bce4fb9e7..217507002dbc 100644 --- a/drivers/gpio/gpio-aspeed.c +++ b/drivers/gpio/gpio-aspeed.c @@ -1224,6 +1224,8 @@ static int __init aspeed_gpio_probe(struct platform_device *pdev) gpio->offset_timer = devm_kzalloc(&pdev->dev, gpio->chip.ngpio, GFP_KERNEL); + if (!gpio->offset_timer) + return -ENOMEM; return aspeed_gpio_setup_irqs(gpio, pdev); } |