diff options
author | Rich Felker <dalias@aerifal.cx> | 2011-09-26 00:59:28 -0400 |
---|---|---|
committer | Rich Felker <dalias@aerifal.cx> | 2011-09-26 00:59:28 -0400 |
commit | fd142e5ec44aaafffbb8bb4ea41c4288d3fa937a (patch) | |
tree | 23fac00d2f8a9d7851d336d9bbb7d3417edc6980 /src/internal | |
parent | 729d6368bdf9faa33299cdfa68efc7422af33bd7 (diff) | |
download | musl-fd142e5ec44aaafffbb8bb4ea41c4288d3fa937a.tar.gz musl-fd142e5ec44aaafffbb8bb4ea41c4288d3fa937a.tar.xz musl-fd142e5ec44aaafffbb8bb4ea41c4288d3fa937a.zip |
cleanup various minor issues reported by nsz
the changes to syscall_ret are mostly no-ops in the generated code, just cleanup of type issues and removal of some implementation-defined behavior. the one exception is the change in the comparison value, which is fixed so that 0xf...f000 (which in principle could be a valid return value for mmap, although probably never in reality) is not treated as an error return.
Diffstat (limited to 'src/internal')
-rw-r--r-- | src/internal/syscall_ret.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/internal/syscall_ret.c b/src/internal/syscall_ret.c index 4f159e0b..e4a1bdbe 100644 --- a/src/internal/syscall_ret.c +++ b/src/internal/syscall_ret.c @@ -3,9 +3,9 @@ long __syscall_ret(unsigned long r) { - if (r >= (unsigned long)-1 - 4096) { - errno = -(long)r; + if (r > -4096UL) { + errno = -r; return -1; } - return (long)r; + return r; } |