ALSA: bebob: sizeof() vs ARRAY_SIZE() typo
authorDan Carpenter <dan.carpenter@oracle.com>
Wed, 28 May 2014 16:43:30 +0000 (19:43 +0300)
committerTakashi Iwai <tiwai@suse.de>
Sun, 1 Jun 2014 16:16:04 +0000 (18:16 +0200)
ARRAY_SIZE() was intended here instead of sizeof().  The
"bridgeco_freq_table" array holds integers so the original condition is
never true.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewd-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Tested-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/firewire/bebob/bebob_stream.c

index 514c7c9..bc4f827 100644 (file)
@@ -749,7 +749,7 @@ parse_stream_formation(u8 *buf, unsigned int len,
                if (buf[2] == bridgeco_freq_table[i])
                        break;
        }
-       if (i == sizeof(bridgeco_freq_table))
+       if (i == ARRAY_SIZE(bridgeco_freq_table))
                return -ENOSYS;
 
        /* Avoid double count by different entries for the same rate. */