strerror_r: fix Solaris test failures

Solaris 10 populates buf on EINVAL, but not on ERANGE.

* lib/strerror_r.c (strerror_r): Partially populate buf on ERANGE
failures.
* doc/posix-functions/strerror_r.texi (strerror_r): Document this.

Signed-off-by: Eric Blake <eblake@redhat.com>

diff --git a/ChangeLog b/ChangeLog
index bc838bdacb7cf763b4aeaedb109e9cb50175ae5f..80e9369be9c4b4b216ce51d895dca1e459b20676 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,10 @@
 2011-05-24  Eric Blake  <eblake@redhat.com>
 
+       strerror_r: fix Solaris test failures
+       * lib/strerror_r.c (strerror_r): Partially populate buf on ERANGE
+       failures.
+       * doc/posix-functions/strerror_r.texi (strerror_r): Document this.
+
        strerror_r: enforce POSIX recommendations
        * lib/strerror_r.c (safe_copy): New helper method.
        (strerror_r): Guarantee a non-empty string.

diff --git a/doc/posix-functions/strerror_r.texi b/doc/posix-functions/strerror_r.texi
index 91026ef912cf176c42f0e12549ff5100f7655bc3..e0f19c01ef8bb65ff2b5c66e2fe45c5a43761df1 100644 (file)
--- a/doc/posix-functions/strerror_r.texi
+++ b/doc/posix-functions/strerror_r.texi
@@ -59,7 +59,7 @@ AIX 6.1, OSF/1 5.1.
 When the value is not in range or the buffer is too small, this
 function fails to leave a NUL-terminated string in the buffer on some
 platforms:
-glibc 2.13, FreeBSD 8.2.
+glibc 2.13, FreeBSD 8.2, Solaris 10.
 @end itemize
 
 Portability problems not fixed by Gnulib:

diff --git a/lib/strerror_r.c b/lib/strerror_r.c
index 40ebc59814f8d11d8a4a019d29c394c445ff9334..f6ce8a324811cdf19697896de940015d5b8e29a7 100644 (file)
--- a/lib/strerror_r.c
+++ b/lib/strerror_r.c
@@ -27,12 +27,16 @@
 
 #include <errno.h>
 
-# if GNULIB_defined_ESOCK /* native Windows platforms */
-#  if HAVE_WINSOCK2_H
-#   include <winsock2.h>
-#  endif
+#if GNULIB_defined_ESOCK /* native Windows platforms */
+# if HAVE_WINSOCK2_H
+#  include <winsock2.h>
 # endif
+#endif
 
+/* Reasonable buffer size that should never trigger ERANGE; if this
+   proves too small, we intentionally abort(), to remind us to fix
+   this value as well as strerror-impl.h.  */
+#define STACKBUF_LEN 256
 
 #if (__GLIBC__ >= 2 || defined __UCLIBC__ || defined __CYGWIN__) && HAVE___XPG_STRERROR_R /* glibc >= 2.3.4, cygwin >= 1.7.9 */
 
@@ -483,7 +487,18 @@ strerror_r (int errnum, char *buf, size_t buflen)
         ret = strerror_r (errnum, buf, buflen);
     }
 # else
+    /* Solaris 10 does not populate buf on ERANGE.  */
     ret = strerror_r (errnum, buf, buflen);
+    if (ret == ERANGE && !*buf)
+      {
+        char stackbuf[STACKBUF_LEN];
+
+        /* strerror-impl.h is also affected if our choice of stackbuf
+           size is not large enough.  */
+        if (strerror_r (errnum, stackbuf, sizeof stackbuf) == ERANGE)
+          abort ();
+        safe_copy (buf, buflen, stackbuf);
+      }
 # endif
 
     /* Some old implementations may return (-1, EINVAL) instead of EINVAL.  */