diff options
| author | Andy Shevchenko <[email protected]> | 2024-10-30 17:36:51 +0000 |
|---|---|---|
| committer | Bartosz Golaszewski <[email protected]> | 2024-10-31 12:47:36 +0000 |
| commit | 5e3eedf55f13ef20ff475eacac30d1e6f91641da (patch) | |
| tree | a24c774854e1761897c93c186a6f6853ad328eaa /drivers/gpio/gpio-mpsse.c | |
| parent | gpiolib: avoid format string weakness in workqueue interface (diff) | |
| download | kernel-5e3eedf55f13ef20ff475eacac30d1e6f91641da.tar.gz kernel-5e3eedf55f13ef20ff475eacac30d1e6f91641da.zip | |
gpio: mpsse: Check for error code from devm_mutex_init() call
Even if it's not critical, the avoidance of checking the error code
from devm_mutex_init() call today diminishes the point of using devm
variant of it. Tomorrow it may even leak something. Add the missed
check.
Fixes: c46a74ff05c0 ("gpio: add support for FTDI's MPSSE as GPIO")
Reviewed-by: Mary Strodl <[email protected]>
Signed-off-by: Andy Shevchenko <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Bartosz Golaszewski <[email protected]>
Diffstat (limited to 'drivers/gpio/gpio-mpsse.c')
| -rw-r--r-- | drivers/gpio/gpio-mpsse.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/drivers/gpio/gpio-mpsse.c b/drivers/gpio/gpio-mpsse.c index 3ab6651d2226..9ef24449126a 100644 --- a/drivers/gpio/gpio-mpsse.c +++ b/drivers/gpio/gpio-mpsse.c @@ -430,8 +430,13 @@ static int gpio_mpsse_probe(struct usb_interface *interface, if (err) return err; - devm_mutex_init(dev, &priv->io_mutex); - devm_mutex_init(dev, &priv->irq_mutex); + err = devm_mutex_init(dev, &priv->io_mutex); + if (err) + return err; + + err = devm_mutex_init(dev, &priv->irq_mutex); + if (err) + return err; priv->gpio.label = devm_kasprintf(dev, GFP_KERNEL, "gpio-mpsse.%d.%d", |