From: Bryce Harrington Date: Sat, 9 Jul 2016 02:00:20 +0000 (-0700) Subject: tests: Require base 10 for the string specifying the number of open fd's X-Git-Tag: 1.11.91~22 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=1cda73f3f85514ef18f180d350e1e390ff388f10;p=platform%2Fupstream%2Fwayland.git tests: Require base 10 for the string specifying the number of open fd's The third arg to strtol() specifies the base to assume for the number. When 0 is passed, as is currently done in wayland-client.c, hexadecimal and octal numbers are permitted and automatically detected and converted. exec-fd-leak-checker's single argument is the count of file descriptors it should expect to be open. We should expect this to be specified only as a decimal number, there's no reason why one would want to use octal or hexadecimal for that. Suggested by Yong Bakos. Signed-off-by: Bryce Harrington Reviewed-by: Yong Bakos --- diff --git a/tests/exec-fd-leak-checker.c b/tests/exec-fd-leak-checker.c index 0c69da3..5f3b395 100644 --- a/tests/exec-fd-leak-checker.c +++ b/tests/exec-fd-leak-checker.c @@ -37,7 +37,7 @@ parse_count(const char *str, int *value) long v; errno = 0; - v = strtol(str, &end, 0); + v = strtol(str, &end, 10); if ((errno == ERANGE && (v == LONG_MAX || v == LONG_MIN)) || (errno != 0 && v == 0) || (end == str) ||