Commit 1cda73f3 authored by Bryce Harrington's avatar Bryce Harrington
Browse files

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

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: default avatarBryce Harrington <>
Reviewed-by: default avatarYong Bakos <>
parent d6d7dab4
......@@ -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) ||
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment