On Mon, Mar 30, 2015 at 10:55 AM, Ricardo Ribalda Delgado <ricardo.ribalda@xxxxxxxxx> wrote: > --- a/drivers/leds/led-class.c > +++ b/drivers/leds/led-class.c > @@ -223,12 +223,17 @@ static int led_classdev_next_name(const char *init_name, char *name, > size_t len) > { > int i = 0; > + int ret = 0; > > strlcpy(name, init_name, len); > > - while (class_find_device(leds_class, NULL, name, match_name)) > + while (class_find_device(leds_class, NULL, name, match_name) && > + (ret < len)) > snprintf(name, len, "%s_%d", init_name, ++i); Missing "ret = " Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe linux-leds" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html