vallist: more thoroughly check for trailing garbage after numeric string

When converting numeric strings into numbers, ensure that only optional
trailing whitespace follows and no other characters.

Fixes: #231
Signed-off-by: Jo-Philipp Wich <jo@mein.io>

1 files changed, 8 insertions, 2 deletions

diff --git a/vallist.c b/vallist.c
index 886ede0..61a4a59 100644
--- a/vallist.c
+++ b/vallist.c
@@ -106,7 +106,10 @@ uc_number_parse_common(const char *buf, bool octal, char **end)
 	if (base >= 10 && (**end == '.' || (**end|32) == 'e')) {
 		d = strtod(p, end);
 
-		if (!isspace(**end) && **end != 0)
+		while (isspace(**end))
+			(*end)++;
+
+		if (**end != 0)
 			return NULL;
 
 		if (neg)
@@ -115,7 +118,10 @@ uc_number_parse_common(const char *buf, bool octal, char **end)
 		return ucv_double_new(d);
 	}
 
-	if (!isspace(**end) && **end != 0)
+	while (isspace(**end))
+		(*end)++;
+
+	if (**end != 0)
 		return NULL;
 
 	if (neg) {