File 0015-amixer-Parse-the-value-more-strictly.patch of Package alsa-utils

Overview Repositories Revisions Requests Users Attributes Meta

File 0015-amixer-Parse-the-value-more-strictly.patch of Package alsa-utils (Revision b4b1ef2d007acd759c3b2638787a5fdc)

Currently displaying revision b4b1ef2d007acd759c3b2638787a5fdc , Show latest

From 088593c03980209c44a9e9cde19723361d341c0a Mon Sep 17 00:00:00 2001
From: Takashi Iwai <tiwai@suse.de>
Date: Sun, 23 Nov 2014 10:04:24 +0100
Subject: [PATCH 15/15] amixer: Parse the value more strictly

So far amixer allows some unexpected suffix and assumes as a raw
absolute value without returning an error.  This is rather dangerous,
e.g. user might not notice that a completely wrong value was set when
the command line included a typo.

This patch makes the parser a bit more strict: it doesn't allow any
longer invalid suffixes, instead either returns an error or skips the
invalid value, depending on the operation mode.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
---
 amixer/amixer.c | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/amixer/amixer.c b/amixer/amixer.c
index 6a2fdb96c62f..ed60e7c3a960 100644
--- a/amixer/amixer.c
+++ b/amixer/amixer.c
@@ -325,7 +325,7 @@ static int set_volume_simple(snd_mixer_elem_t *elem,
 	long val, orig, pmin, pmax;
 	char *p = *ptr, *s;
 	int invalid = 0, percent = 0, err = 0;
-	int vol_type = std_vol_type;
+	int vol_type;
 	double scale = 1.0;
 	int correct = 0;
 
@@ -344,14 +344,19 @@ static int set_volume_simple(snd_mixer_elem_t *elem,
 		strtol(p, &p, 10);
 	}
 	if (*p == '%') {
+		vol_type = std_vol_type;
 		percent = 1;
 		p++;
 	} else if (toupper(p[0]) == 'D' && toupper(p[1]) == 'B') {
 		vol_type = VOL_DB;
 		p += 2;
 		scale = 100;
-	} else
+	} else {
 		vol_type = VOL_RAW;
+	}
+
+	if (*p && !strchr(",:+-", *p))
+		invalid = 1;
 
 	val = (long)(strtod(s, NULL) * scale);
 	if (vol_ops[dir].v[vol_type].get_range(elem, &pmin, &pmax) < 0)
@@ -372,6 +377,10 @@ static int set_volume_simple(snd_mixer_elem_t *elem,
 		}
 		p++;
 	}
+
+	if (*p && !strchr(",:", *p))
+		invalid = 1;
+
 	if (! invalid) {
 		val = check_range(val, pmin, pmax);
 		err = vol_ops[dir].v[vol_type].set(elem, chn, val, correct);
-- 
2.2.0

Places

File 0015-amixer-Parse-the-value-more-strictly.patch of Package alsa-utils (Revision b4b1ef2d007acd759c3b2638787a5fdc)

Places