]> Pileus Git - ~andy/linux/commit
ALSA: ak4114: Fix wrong register array size
authorTakashi Iwai <tiwai@suse.de>
Tue, 29 Oct 2013 15:37:11 +0000 (16:37 +0100)
committerTakashi Iwai <tiwai@suse.de>
Tue, 29 Oct 2013 15:42:36 +0000 (16:42 +0100)
commite12483e0f3dbc32dad8fa1dc97efac22b6aee94f
tree53207314a9b65a25832427e0d835032f132f1ecc
parent2026d24ef2ea8caad5e87662a58075e930ccab63
ALSA: ak4114: Fix wrong register array size

The size of the register cache array is actually 6 instead of 7,
as it caches up to AK4114_REG_INT1_MASK.  This resulted in unexpected
access out of array range, although most of them aren't so serious
(just reading one more byte on the stack at snd_ak4114_create()).

Also, the check of cache size was wrongly done by checking with
sizeof() instead of ARRAY_SIZE().  Fixed this together.

(And yes, hardcoded numbers are bad, but I keep the coding style as is
 for making it clear what this patch actually does.)

Spotted by coverity among several CIDs, e.g. 711621.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
include/sound/ak4114.h
sound/i2c/other/ak4114.c