aboutsummaryrefslogtreecommitdiffstats
path: root/sound/sh
diff options
context:
space:
mode:
authorTakashi Iwai <tiwai@suse.de>2007-11-15 10:17:24 -0500
committerJaroslav Kysela <perex@perex.cz>2008-01-31 11:29:25 -0500
commit3b892467786410f26dffc2c7bccd3ea445604037 (patch)
tree4a70f86b1545e9be95d987fc248cac3cebbbd9a8 /sound/sh
parent498ade1a133dffd0f3ee90952737045d56e6689a (diff)
[ALSA] Check value range in ctl callbacks
Check the value ranges in ctl put callbacks properly (in the rest drivers). Signed-off-by: Takashi Iwai <tiwai@suse.de> Signed-off-by: Jaroslav Kysela <perex@perex.cz>
Diffstat (limited to 'sound/sh')
-rw-r--r--sound/sh/aica.c7
1 files changed, 5 insertions, 2 deletions
diff --git a/sound/sh/aica.c b/sound/sh/aica.c
index 8861d2f7796e..12c41df255a1 100644
--- a/sound/sh/aica.c
+++ b/sound/sh/aica.c
@@ -523,11 +523,14 @@ static int aica_pcmvolume_put(struct snd_kcontrol *kcontrol,
523 struct snd_ctl_elem_value *ucontrol) 523 struct snd_ctl_elem_value *ucontrol)
524{ 524{
525 struct snd_card_aica *dreamcastcard; 525 struct snd_card_aica *dreamcastcard;
526 unsigned int vol;
526 dreamcastcard = kcontrol->private_data; 527 dreamcastcard = kcontrol->private_data;
527 if (unlikely(!dreamcastcard->channel)) 528 if (unlikely(!dreamcastcard->channel))
528 return -ETXTBSY; 529 return -ETXTBSY;
529 if (unlikely(dreamcastcard->channel->vol == 530 vol = ucontrol->value.integer.value[0];
530 ucontrol->value.integer.value[0])) 531 if (vol > 0xff)
532 return -EINVAL;
533 if (unlikely(dreamcastcard->channel->vol == vol))
531 return 0; 534 return 0;
532 dreamcastcard->channel->vol = ucontrol->value.integer.value[0]; 535 dreamcastcard->channel->vol = ucontrol->value.integer.value[0];
533 dreamcastcard->master_volume = ucontrol->value.integer.value[0]; 536 dreamcastcard->master_volume = ucontrol->value.integer.value[0];