diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-06-26 03:52:20 -0400 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2013-06-26 04:35:55 -0400 |
commit | bd5fe738e388ceaa32e5171481e0d3ec59f0ccfe (patch) | |
tree | e5195fbd95d2d77b6fc5eb7dba404ec82fe32649 /sound/i2c/other | |
parent | f02fe86199d90cd3c7c0c51e223840c3891e5d18 (diff) |
ALSA: ak4xx-adda: info leak in ak4xxx_capture_source_info()
"idx" is controled by the user and can be a negative offset into the
input_names[] array.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/i2c/other')
-rw-r--r-- | sound/i2c/other/ak4xxx-adda.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/i2c/other/ak4xxx-adda.c b/sound/i2c/other/ak4xxx-adda.c index cef813d23641..ed726d1569e8 100644 --- a/sound/i2c/other/ak4xxx-adda.c +++ b/sound/i2c/other/ak4xxx-adda.c | |||
@@ -571,7 +571,7 @@ static int ak4xxx_capture_source_info(struct snd_kcontrol *kcontrol, | |||
571 | struct snd_akm4xxx *ak = snd_kcontrol_chip(kcontrol); | 571 | struct snd_akm4xxx *ak = snd_kcontrol_chip(kcontrol); |
572 | int mixer_ch = AK_GET_SHIFT(kcontrol->private_value); | 572 | int mixer_ch = AK_GET_SHIFT(kcontrol->private_value); |
573 | const char **input_names; | 573 | const char **input_names; |
574 | int num_names, idx; | 574 | unsigned int num_names, idx; |
575 | 575 | ||
576 | num_names = ak4xxx_capture_num_inputs(ak, mixer_ch); | 576 | num_names = ak4xxx_capture_num_inputs(ak, mixer_ch); |
577 | if (!num_names) | 577 | if (!num_names) |