diff options
author | Takashi Iwai <tiwai@suse.de> | 2007-11-15 10:17:24 -0500 |
---|---|---|
committer | Jaroslav Kysela <perex@perex.cz> | 2008-01-31 11:29:25 -0500 |
commit | 3b892467786410f26dffc2c7bccd3ea445604037 (patch) | |
tree | 4a70f86b1545e9be95d987fc248cac3cebbbd9a8 /sound/i2c/other | |
parent | 498ade1a133dffd0f3ee90952737045d56e6689a (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/i2c/other')
-rw-r--r-- | sound/i2c/other/pt2258.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/sound/i2c/other/pt2258.c b/sound/i2c/other/pt2258.c index 00c83d8b32b1..987d2c9a7a64 100644 --- a/sound/i2c/other/pt2258.c +++ b/sound/i2c/other/pt2258.c | |||
@@ -113,6 +113,8 @@ static int pt2258_stereo_volume_put(struct snd_kcontrol *kcontrol, | |||
113 | 113 | ||
114 | val0 = 79 - ucontrol->value.integer.value[0]; | 114 | val0 = 79 - ucontrol->value.integer.value[0]; |
115 | val1 = 79 - ucontrol->value.integer.value[1]; | 115 | val1 = 79 - ucontrol->value.integer.value[1]; |
116 | if (val0 < 0 || val0 > 79 || val1 < 0 || val1 > 79) | ||
117 | return -EINVAL; | ||
116 | if (val0 == pt->volume[base] && val1 == pt->volume[base + 1]) | 118 | if (val0 == pt->volume[base] && val1 == pt->volume[base + 1]) |
117 | return 0; | 119 | return 0; |
118 | 120 | ||