diff options
| author | Bartosz Golaszewski <brgl@bgdev.pl> | 2018-03-04 13:45:51 +0100 |
|---|---|---|
| committer | Linus Walleij <linus.walleij@linaro.org> | 2018-03-27 15:18:20 +0200 |
| commit | 6cb9215baeb9c1ed336a5e8905f7ad7c4698acc9 (patch) | |
| tree | 2326844d2fc8c9504fc49fdd27d4f0ad2234af1d /drivers/gpio | |
| parent | 3a711e0dd4e68f6b202db3f9e2c0086a8780da25 (diff) | |
| download | op-kernel-dev-6cb9215baeb9c1ed336a5e8905f7ad7c4698acc9.zip op-kernel-dev-6cb9215baeb9c1ed336a5e8905f7ad7c4698acc9.tar.gz | |
gpio: mockup: fix a potential crash when creating debugfs entries
If we failed to create the top debugfs directory, we must not try to
create the child nodes. We currently only check if gpio_mockup_dbg_dir
is not NULL, but it can also contain an errno if debugfs is disabled
in build options. Use IS_ERR_OR_NULL() instead.
Signed-off-by: Bartosz Golaszewski <brgl@bgdev.pl>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'drivers/gpio')
| -rw-r--r-- | drivers/gpio/gpio-mockup.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpio/gpio-mockup.c b/drivers/gpio/gpio-mockup.c index 21422b8..76c2fe9 100644 --- a/drivers/gpio/gpio-mockup.c +++ b/drivers/gpio/gpio-mockup.c @@ -314,7 +314,7 @@ static int gpio_mockup_probe(struct platform_device *pdev) if (rv) return rv; - if (gpio_mockup_dbg_dir) + if (!IS_ERR_OR_NULL(gpio_mockup_dbg_dir)) gpio_mockup_debugfs_setup(dev, chip); return 0; |
