Make strerror and strerror_r use sys_errlist[0] for errnum = 0. Be

more careful about reporting truncation with ERANGE in strerror_r.
Set errno to EINVAL for "unknown" errnum in strerror as required
by P1003.1-200x Draft June 14, 2001.

More carefully document the handling of strerrbuf when errors
(ERANGE, EINVAL) are encountered in strerror_r.

Reviewed by:	bde (ongoing discussion)

1 files changed, 20 insertions, 3 deletions

diff --git a/lib/libc/string/strerror.c b/lib/libc/string/strerror.c
index e0f4cd3..55840f1 100644
--- a/lib/libc/string/strerror.c
+++ b/lib/libc/string/strerror.c
@@ -46,9 +46,9 @@ strerror_r(int errnum, char *strerrbuf, size_t buflen)
 {
 	int             len;
 
-	if ((errnum > 0) && (errnum < sys_nerr)) {
+	if ((errnum >= 0) && (errnum < sys_nerr)) {
 		len = strlcpy(strerrbuf, (char *)sys_errlist[errnum], buflen);
-		return ((len <= buflen) ? 0 : ERANGE);
+		return ((len < buflen) ? 0 : ERANGE);
 	}
 	return (EINVAL);
 }
@@ -71,10 +71,15 @@ strerror(num)
 	char            tmp[NUMLEN];	/* temporary number */
 	static char     ebuf[EBUFLEN];	/* error message */
 
-	if ((num > 0) && (num < sys_nerr))
+	if ((num >= 0) && (num < sys_nerr))
 		return ((char *)sys_errlist[num]);
 
 	/*
+	 * Set errno to EINVAL per P1003.1-200x Draft June 14, 2001.
+	 */
+	errno = EINVAL;
+
+	/*
 	 * Print unknown errno by hand so we don't link to stdio(3).
 	 * This collects the ASCII digits in reverse order.
 	 */
@@ -106,8 +111,20 @@ main()
 	char            mybuf[64];
 	int             ret;
 
+	errno = 0;
+
+	printf("strerror(0) yeilds: %s\n", strerror(0));
+	printf("strerror(1) yeilds: %s\n", strerror(1));
 	printf("strerror(47) yeilds: %s\n", strerror(47));
+	printf("strerror(sys_nerr - 1) yeilds: %s\n", strerror(sys_nerr - 1));
+	printf("errno = %d\n", errno); errno = 0;
+
+	printf("strerror(sys_nerr) yeilds: %s\n", strerror(sys_nerr));
+	printf("errno = %d\n", errno);  errno = 0;
+
 	printf("strerror(437) yeilds: %s\n", strerror(437));
+	printf("errno = %d\n", errno);  errno = 0;
+
 	printf("strerror(LONG_MAX) yeilds: %s\n", strerror(LONG_MAX));
 	printf("strerror(LONG_MIN) yeilds: %s\n", strerror(LONG_MIN));
 	printf("strerror(ULONG_MAX) yeilds: %s\n", strerror(ULONG_MAX));