Re-apply "config-parser: Catch negative numbers assigned to unsigned config values"
authorBryce Harrington <bryce@osg.samsung.com>
Fri, 15 Jul 2016 01:28:04 +0000 (18:28 -0700)
committerBryce Harrington <bryce@osg.samsung.com>
Tue, 26 Jul 2016 23:21:20 +0000 (16:21 -0700)
commitd0716f4af52e903c1a92c7fed13f7da51adb67fd
tree43146b5470424878adcd57431d9c800c49223e7d
parente776f2a4d9a335a41d9cbc8b06ba97a660051614
Re-apply "config-parser: Catch negative numbers assigned to unsigned config values"

[With hexadecimal color values now handled via their own routine,
re-introduce the negative unsigned numbers fix.]

strtoul() has a side effect that when given a string representing a
negative number, it treats it as a high value hexadecimal.  IOW,
strtoul("-42", &val) sets val to 0xffffffd6.  This could potentially
result in unintended surprise behaviors.

Catch this by using strtol() and then manually check for the negative
value.  This logic is modelled after Wayland's strtouint().

Note that this change unfortunately reduces the range of parseable
numbers from [0,UINT_MAX] to [0,INT_MAX].  The current users of
weston_config_section_get_uint() are anticipating numbers far smaller
than either of these limits, so the change is believed to have no impact
in practice.

Also add a test case for negative numbers that catches this error
condition.

Signed-off-by: Bryce Harrington <bryce@osg.samsung.com>
Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com>
shared/config-parser.c
tests/config-parser-test.c