leds: gpio: Fix error handling for led name null pointer case

Make sure that already registered LEDs will be torn down properly
if the name of the next LED to create is unavailable.

Signed-off-by: Jacek Anaszewski <j.anaszewski@samsung.com>
Cc: Richard Purdie <rpurdie@rpsys.net>
Signed-off-by: Bryan Wu <cooloney@gmail.com>

1 files changed, 4 insertions, 2 deletions

diff --git a/drivers/leds/leds-gpio.c b/drivers/leds/leds-gpio.c
index 25df4a240972..c3db74a1aba7 100644
--- a/drivers/leds/leds-gpio.c
+++ b/drivers/leds/leds-gpio.c
@@ -198,8 +198,10 @@ static struct gpio_leds_priv *gpio_leds_create(struct platform_device *pdev)
                 } else {
                         if (IS_ENABLED(CONFIG_OF) && !led.name && np)
                                 led.name = np->name;
-                        if (!led.name)
-                                return ERR_PTR(-EINVAL);
+                        if (!led.name) {
+                                ret = -EINVAL;
+                                goto err;
+                        }
                 }
                 fwnode_property_read_string(child, "linux,default-trigger",
                                             &led.default_trigger);