tjh.dev / kernel
Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux
fork atom

ALSA: sc6000: Use safer strscpy() instead of strcpy()

Use a safer function strscpy() instead of strcpy() for copying to
arrays.

Only idiomatic code replacement, and no functional changes.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
Link: https://patch.msgid.link/20250710100727.22653-36-tiwai@suse.de

Takashi Iwai c8eef317 b4a82a97

+10 -10
+10 -10
sound/isa/sc6000.c
··· 469 469 id1.iface = SNDRV_CTL_ELEM_IFACE_MIXER; 470 470 id2.iface = SNDRV_CTL_ELEM_IFACE_MIXER; 471 471 /* reassign AUX0 to FM */ 472 - strcpy(id1.name, "Aux Playback Switch"); 473 - strcpy(id2.name, "FM Playback Switch"); 472 + strscpy(id1.name, "Aux Playback Switch"); 473 + strscpy(id2.name, "FM Playback Switch"); 474 474 err = snd_ctl_rename_id(card, &id1, &id2); 475 475 if (err < 0) 476 476 return err; 477 - strcpy(id1.name, "Aux Playback Volume"); 478 - strcpy(id2.name, "FM Playback Volume"); 477 + strscpy(id1.name, "Aux Playback Volume"); 478 + strscpy(id2.name, "FM Playback Volume"); 479 479 err = snd_ctl_rename_id(card, &id1, &id2); 480 480 if (err < 0) 481 481 return err; 482 482 /* reassign AUX1 to CD */ 483 - strcpy(id1.name, "Aux Playback Switch"); id1.index = 1; 484 - strcpy(id2.name, "CD Playback Switch"); 483 + strscpy(id1.name, "Aux Playback Switch"); id1.index = 1; 484 + strscpy(id2.name, "CD Playback Switch"); 485 485 err = snd_ctl_rename_id(card, &id1, &id2); 486 486 if (err < 0) 487 487 return err; 488 - strcpy(id1.name, "Aux Playback Volume"); 489 - strcpy(id2.name, "CD Playback Volume"); 488 + strscpy(id1.name, "Aux Playback Volume"); 489 + strscpy(id2.name, "CD Playback Volume"); 490 490 err = snd_ctl_rename_id(card, &id1, &id2); 491 491 if (err < 0) 492 492 return err; ··· 652 652 mpu_port[dev]); 653 653 } 654 654 655 - strcpy(card->driver, DRV_NAME); 656 - strcpy(card->shortname, "SC-6000"); 655 + strscpy(card->driver, DRV_NAME); 656 + strscpy(card->shortname, "SC-6000"); 657 657 sprintf(card->longname, "Gallant SC-6000 at 0x%lx, irq %d, dma %d", 658 658 mss_port[dev], xirq, xdma); 659 659