Require base-10 for strtol() calls
authorBryce Harrington <bryce@osg.samsung.com>
Tue, 12 Jul 2016 23:51:27 +0000 (16:51 -0700)
committerBryce Harrington <bryce@osg.samsung.com>
Wed, 13 Jul 2016 01:46:13 +0000 (18:46 -0700)
commit375759e63602a4fd580bb7c9dcfe2770c3c25971
treeeea55fb5d12722dddf4f9a6940145a5bedf1ac38
parent1dbdc0bd8a1b9cedda12571c44c5068cecaa5d26
Require base-10 for strtol() calls

The third arg to strtol() specifies the base to assume for the number.
When 0 is passed, as is currently done in option-parser.c, hexadecimal
and octal numbers are permitted and automatically detected and
converted.

This change is an expansion of f6051cbab84c0e577473b67f0585c0f329eb80fe
to cover the remaining strtol() calls in Weston, where the routine is
being used to read fds and pids - which are always expressed in base-10.
It also changes the calls in config-parser, used by
weston_config_section_get_int(), which in turn is being used to read
scales, sizes, times, rates, and delays; these are all expressed in
base-10 numbers only.

The benefit of limiting this to base-10 is to eliminate surprises when
parsing numbers from the command line.  Also, by making the code
consistent with other usages of strtol, it may make it possible to
factor out the common code in the future.

Signed-off-by: Bryce Harrington <bryce@osg.samsung.com>
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
compositor/main.c
libweston/compositor.c
shared/config-parser.c
xwayland/launcher.c