diff options
author | Rich Felker <dalias@aerifal.cx> | 2014-09-16 16:08:53 -0400 |
---|---|---|
committer | Rich Felker <dalias@aerifal.cx> | 2014-09-16 16:08:53 -0400 |
commit | e2e1bb81485a37321d928a8d8b63f40b9d8fa228 (patch) | |
tree | 9b62eab14fa49e9a1f7cf9bc350a53d79eca60f0 /src | |
parent | ec1aed0a144b3e00e16eeb142c9d13362d6048e7 (diff) | |
download | musl-e2e1bb81485a37321d928a8d8b63f40b9d8fa228.tar.gz musl-e2e1bb81485a37321d928a8d8b63f40b9d8fa228.tar.xz musl-e2e1bb81485a37321d928a8d8b63f40b9d8fa228.zip |
fix overflow corner case in strtoul-family functions
incorrect behavior occurred only in cases where the input overflows unsigned long long, not just the (possibly lower) range limit for the result type. in this case, processing of the '-' sign character was not suppressed, and the function returned a value of 1 despite setting errno to ERANGE.
Diffstat (limited to 'src')
-rw-r--r-- | src/internal/intscan.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/internal/intscan.c b/src/internal/intscan.c index 69350efa..65d497ec 100644 --- a/src/internal/intscan.c +++ b/src/internal/intscan.c @@ -83,6 +83,7 @@ unsigned long long __intscan(FILE *f, unsigned base, int pok, unsigned long long for (; val[c]<base; c=shgetc(f)); errno = ERANGE; y = lim; + if (lim&1) neg = 0; } done: shunget(f); |