ALSA: usb-audio: Fix a limit check in proc_dump_substream_formats()
authorDan Carpenter <dan.carpenter@oracle.com>
Wed, 22 Apr 2020 09:22:55 +0000 (12:22 +0300)
committerTakashi Iwai <tiwai@suse.de>
Wed, 22 Apr 2020 09:29:15 +0000 (11:29 +0200)
This should be ARRAY_SIZE() instead of sizeof().  The sizeof() limit is
too high so it doesn't work.

Fixes: 093b8494f299 ("ALSA: usb-audio: Print more information in stream proc files")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Link: https://lore.kernel.org/r/20200422092255.GB195357@mwanda
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/usb/proc.c

index 5a36e19..889c550 100644 (file)
@@ -140,7 +140,7 @@ static void proc_dump_substream_formats(struct snd_usb_substream *subs, struct s
 
                        snd_iprintf(buffer, "    Channel map:");
                        for (c = 0; c < map->channels; c++) {
-                               if (map->map[c] >= sizeof(channel_labels) ||
+                               if (map->map[c] >= ARRAY_SIZE(channel_labels) ||
                                    !channel_labels[map->map[c]])
                                        snd_iprintf(buffer, " --");
                                else