net: dsa: hellcreek: fix missing error handling in LED registration

The LED setup routine registered both led_sync_good
and led_is_gm devices without checking the return
values of led_classdev_register(). If either registration
failed, the function continued silently, leaving the
driver in a partially-initialized state and leaking
a registered LED classdev.

Add proper error handling

Fixes: 7d9ee2e8ff15 ("net: dsa: hellcreek: Add PTP status LEDs")
Signed-off-by: Pavel Zhigulin <Pavel.Zhigulin@kaspersky.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Acked-by: Kurt Kanzenbach <kurt@linutronix.de>
Link: https://patch.msgid.link/20251113135745.92375-1-Pavel.Zhigulin@kaspersky.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>

diff --git a/drivers/net/dsa/hirschmann/hellcreek_ptp.c b/drivers/net/dsa/hirschmann/hellcreek_ptp.c
index bfe21f9f7dcd36e649d17a514302d13434696217..cb23bea9c21b8172e8b1c528927be8a443559764 100644 (file)
--- a/drivers/net/dsa/hirschmann/hellcreek_ptp.c
+++ b/drivers/net/dsa/hirschmann/hellcreek_ptp.c
@@ -376,8 +376,18 @@ static int hellcreek_led_setup(struct hellcreek *hellcreek)
                hellcreek_set_brightness(hellcreek, STATUS_OUT_IS_GM, 1);
 
        /* Register both leds */
-       led_classdev_register(hellcreek->dev, &hellcreek->led_sync_good);
-       led_classdev_register(hellcreek->dev, &hellcreek->led_is_gm);
+       ret = led_classdev_register(hellcreek->dev, &hellcreek->led_sync_good);
+       if (ret) {
+               dev_err(hellcreek->dev, "Failed to register sync_good LED\n");
+               goto out;
+       }
+
+       ret = led_classdev_register(hellcreek->dev, &hellcreek->led_is_gm);
+       if (ret) {
+               dev_err(hellcreek->dev, "Failed to register is_gm LED\n");
+               led_classdev_unregister(&hellcreek->led_sync_good);
+               goto out;
+       }
 
        ret = 0;