diff options
| author | Takashi Iwai <tiwai@suse.de> | 2007-11-15 16:14:12 +0100 |
|---|---|---|
| committer | Jaroslav Kysela <perex@perex.cz> | 2008-01-31 17:29:25 +0100 |
| commit | d4079ac49a08e36d6839a9ceb26aec8c24c9ed82 (patch) | |
| tree | 91ebc493375fd4e7fa84c3a6ab67a885121b1c81 /sound/ppc/beep.c | |
| parent | d05ab185b770de96399766be6bcb5769ab99bc09 (diff) | |
| download | op-kernel-dev-d4079ac49a08e36d6839a9ceb26aec8c24c9ed82.zip op-kernel-dev-d4079ac49a08e36d6839a9ceb26aec8c24c9ed82.tar.gz | |
[ALSA] powermac - Check value range in ctl callbacks
Check the value ranges in ctl put callbacks properly in snd-powermac
driver.
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Jaroslav Kysela <perex@perex.cz>
Diffstat (limited to 'sound/ppc/beep.c')
| -rw-r--r-- | sound/ppc/beep.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/sound/ppc/beep.c b/sound/ppc/beep.c index 566b5ab..465dd04 100644 --- a/sound/ppc/beep.c +++ b/sound/ppc/beep.c @@ -195,10 +195,13 @@ static int snd_pmac_put_beep(struct snd_kcontrol *kcontrol, struct snd_ctl_elem_value *ucontrol) { struct snd_pmac *chip = snd_kcontrol_chip(kcontrol); - int oval; + unsigned int oval, nval; snd_assert(chip->beep, return -ENXIO); oval = chip->beep->volume; - chip->beep->volume = ucontrol->value.integer.value[0]; + nval = ucontrol->value.integer.value[0]; + if (nval > 100) + return -EINVAL; + chip->beep->volume = nval; return oval != chip->beep->volume; } |
