strerror_r: fix OpenBSD behavior on 0
authorEric Blake <eblake@redhat.com>
Tue, 21 Jun 2011 14:50:51 +0000 (08:50 -0600)
committerEric Blake <eblake@redhat.com>
Tue, 21 Jun 2011 16:38:53 +0000 (10:38 -0600)
commitc74873191b6e585cdd5a3b2ce5e07edae1427cef
treedaf257a4d02a59fc10242a7e735e3a4ab0ac3636
parent8681e553d88b3ea891c98792fcc6675dc05e4b2c
strerror_r: fix OpenBSD behavior on 0

OpenBSD treats strerror_r(0,,) as a success, but with a message
"Undefined error: 0"; while this is distinct from strerror_r(-1,,)
returning "Unknown error: -1", it does not imply success.  Meanwhile,
if buf is short enough for ERANGE, then we can't use strstr to look
for "Unknown" or "Undefined" in the resulting message, like we had
been doing for strerror().  Fix this by shifting the burden - now
the strerror-override code guarantees that 0 will have an
override when needed.

* lib/strerror-override.c (strerror_override): Also override 0
when needed.
* lib/strerror-override.h (strerror_override): Likewise.
* lib/strerror.c (strerror): Simplify, now that 0 override is done
earlier.
* lib/strerror_r.c (strerror_r): Likewise.
* m4/strerror.m4 (gl_FUNC_STRERROR): Split detection of 0
behavior...
(gl_FUNC_STRERROR_0): ...into new macro.
* m4/strerror_r.m4 (gl_FUNC_STRERROR_R): Replace strerror_r if 0
is overridden.
(gl_FUNC_STRERROR_R_WORKS): Avoid extra tests if 0 is broken.
* modules/strerror-override (Files): Add strerror.m4.
(configure.ac): Also provide override for 0 when needed.
* doc/posix-functions/strerror.texi (strerror): Document this.
* doc/posix-functions/perror.texi (perror): Likewise.

Signed-off-by: Eric Blake <eblake@redhat.com>
ChangeLog
doc/posix-functions/perror.texi
doc/posix-functions/strerror.texi
lib/strerror-override.c
lib/strerror-override.h
lib/strerror.c
lib/strerror_r.c
m4/strerror.m4
m4/strerror_r.m4
modules/strerror-override