ALSA: firewire: tascam: Use safer strscpy() instead of strcpy()
authorTakashi Iwai <tiwai@suse.de>
Thu, 10 Jul 2025 10:06:03 +0000 (12:06 +0200)
committerTakashi Iwai <tiwai@suse.de>
Fri, 11 Jul 2025 07:53:08 +0000 (09:53 +0200)
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>
Reviewed-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Link: https://patch.msgid.link/20250710100727.22653-22-tiwai@suse.de
sound/firewire/tascam/tascam-hwdep.c
sound/firewire/tascam/tascam.c

index 74eed95..8fc30cb 100644 (file)
@@ -265,7 +265,7 @@ int snd_tscm_create_hwdep_device(struct snd_tscm *tscm)
        if (err < 0)
                return err;
 
-       strcpy(hwdep->name, "Tascam");
+       strscpy(hwdep->name, "Tascam");
        hwdep->iface = SNDRV_HWDEP_IFACE_FW_TASCAM;
        hwdep->ops = ops;
        hwdep->private_data = tscm;
index 8688008..4f68bb4 100644 (file)
@@ -73,9 +73,9 @@ static int identify_model(struct snd_tscm *tscm)
        if (tscm->spec == NULL)
                return -ENODEV;
 
-       strcpy(tscm->card->driver, "FW-TASCAM");
-       strcpy(tscm->card->shortname, model);
-       strcpy(tscm->card->mixername, model);
+       strscpy(tscm->card->driver, "FW-TASCAM");
+       strscpy(tscm->card->shortname, model);
+       strscpy(tscm->card->mixername, model);
        snprintf(tscm->card->longname, sizeof(tscm->card->longname),
                 "TASCAM %s, GUID %08x%08x at %s, S%d", model,
                 fw_dev->config_rom[3], fw_dev->config_rom[4],