diff options
author | ngie <ngie@FreeBSD.org> | 2016-05-14 23:22:38 +0000 |
---|---|---|
committer | ngie <ngie@FreeBSD.org> | 2016-05-14 23:22:38 +0000 |
commit | e6c895156edf27b8b59126dea4281044f707e6b9 (patch) | |
tree | a7a699ea5da03eb90486ab01daf604a36cba5b0e /LOCKS | |
parent | 1552c80595ca0d207cd88f655e84c2287d382432 (diff) | |
download | FreeBSD-src-e6c895156edf27b8b59126dea4281044f707e6b9.zip FreeBSD-src-e6c895156edf27b8b59126dea4281044f707e6b9.tar.gz |
Use a consistent errno save/restore pattern before running strtoul
- Save errno
- Set errno to 0
- Call strtoul
- Test errno (optional, but many calls to strtoul did this afterwards)
Some of the code was setting errno = 0 after calling strtoul, not setting
errno = 0, or setting errno to saved_errno after the call, but before the
test. These all have unwanted behavioral side-effects, depending on the
initial value of errno and whether or not the input to strtoul was correct
or incorrect.
MFC after: 3 weeks
Sponsored by: EMC / Isilon Storage Division
Diffstat (limited to 'LOCKS')
0 files changed, 0 insertions, 0 deletions