]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
ALSA: trident: Suppress gcc string warning
authorTakashi Iwai <tiwai@suse.de>
Thu, 26 Jul 2018 12:58:03 +0000 (14:58 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 13 Dec 2018 08:22:29 +0000 (09:22 +0100)
commit d6b340d7cb33c816ef4abe8143764ec5ab14a5cc upstream.

The meddlesome gcc warns about the possible shortname string in
trident driver code:
  sound/pci/trident/trident.c: In function ‘snd_trident_probe’:
  sound/pci/trident/trident.c:126:2: warning: ‘strcat’ accessing 17 or more bytes at offsets 36 and 20 may overlap 1 byte at offset 36 [-Wrestrict]
  strcat(card->shortname, card->driver);

It happens since gcc calculates the possible string size from
card->driver, but this can't be true since we did set the string just
before that, and they are much shorter.

For shutting it up, use the exactly same string set to card->driver
for strcat() to card->shortname, too.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
sound/pci/trident/trident.c

index a54cd6879b310106414e1886e566eb72b3ce4c52..89ceda5c69795d8a1d1edd4e1cdaa3879cafd224 100644 (file)
@@ -123,7 +123,7 @@ static int snd_trident_probe(struct pci_dev *pci,
        } else {
                strcpy(card->shortname, "Trident ");
        }
-       strcat(card->shortname, card->driver);
+       strcat(card->shortname, str);
        sprintf(card->longname, "%s PCI Audio at 0x%lx, irq %d",
                card->shortname, trident->port, trident->irq);