On Apr 07, Mauro Carvalho Chehab wrote: > strncpy() doesn't warrant a NUL terminated string. Use > strlcpy() instead. > > Fixes Coverity bug CID#1195195. > > Signed-off-by: Mauro Carvalho Chehab <m.che...@samsung.com> > --- > drivers/media/usb/stk1160/stk1160-ac97.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/media/usb/stk1160/stk1160-ac97.c > b/drivers/media/usb/stk1160/stk1160-ac97.c > index c46c8be89602..2dd308f9541f 100644 > --- a/drivers/media/usb/stk1160/stk1160-ac97.c > +++ b/drivers/media/usb/stk1160/stk1160-ac97.c > @@ -108,7 +108,7 @@ int stk1160_ac97_register(struct stk1160 *dev) > "stk1160-mixer"); > snprintf(card->longname, sizeof(card->longname), > "stk1160 ac97 codec mixer control"); > - strncpy(card->driver, dev->dev->driver->name, sizeof(card->driver)); > + strlcpy(card->driver, dev->dev->driver->name, sizeof(card->driver)); > > rc = snd_ac97_bus(card, 0, &stk1160_ac97_ops, NULL, &ac97_bus); > if (rc)
Shame on me. Acked-by: Ezequiel Garcia <ezequiel.gar...@free-electrons.com> -- Ezequiel García, Free Electrons Embedded Linux, Kernel and Android Engineering http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
