snprintf return value fixes to conform Single Unix specs:

1) if buffer size is smaller than arguments size, return buffer
size, not arguments size as before.

2) if buffer size is 0, return 0, not EOF as before.
(now it is compatible with Linux and Apache implementations too).

NOTE: Single Unix specs says:

If the value of n {buffer size} is zero on a call to snprintf(), an
unspecified value less than 1 is returned.

It means we can't return EOF since EOF can take *any* value in general
not especially < 1. Better variant will be return -1 (it is less then
1 and different with n == 1 case) but -1 value is already occuped by
EOF in our implementation, so we can't distinguish true IO error
in that case. So 0 here is only possible case still conforming
to Single Unix specs.

1 files changed, 8 insertions, 3 deletions

diff --git a/lib/libc/stdio/snprintf.c b/lib/libc/stdio/snprintf.c
index 0e3c456..086f8b2 100644
--- a/lib/libc/stdio/snprintf.c
+++ b/lib/libc/stdio/snprintf.c
@@ -39,7 +39,7 @@
 static char sccsid[] = "@(#)snprintf.c	8.1 (Berkeley) 6/4/93";
 #endif
 static const char rcsid[] =
-		"$Id$";
+		"$Id: snprintf.c,v 1.5 1997/02/22 15:02:29 peter Exp $";
 #endif /* LIBC_SCCS and not lint */
 
 #include <stdio.h>
@@ -65,8 +65,11 @@ snprintf(str, n, fmt, va_alist)
 	va_list ap;
 	FILE f;
 
+	if (n == 0)
+		return (0);
 	if ((int)n < 1)
 		return (EOF);
+	n--;
 #if __STDC__
 	va_start(ap, fmt);
 #else
@@ -75,9 +78,11 @@ snprintf(str, n, fmt, va_alist)
 	f._file = -1;
 	f._flags = __SWR | __SSTR;
 	f._bf._base = f._p = (unsigned char *)str;
-	f._bf._size = f._w = n - 1;
+	f._bf._size = f._w = n;
 	ret = vfprintf(&f, fmt, ap);
 	*f._p = 0;
 	va_end(ap);
-	return (ret);
+	if (ret == EOF)
+		return (ret);
+	return (ret > (int)n ? n : ret);
 }