From ba21046d032462b4bb1f601e0014a0306c201913 Mon Sep 17 00:00:00 2001 From: Eric Blake Date: Mon, 17 Sep 2007 20:14:29 +0000 Subject: Obey POSIX on printf("%.s", (char*)NULL). * libc/stdio/vfprintf.c (_VFPRINTF_R): Take precision into account for %s on NULL. Skip NULL check when optimizing for size. --- newlib/libc/stdio/vfprintf.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) (limited to 'newlib/libc/stdio/vfprintf.c') diff --git a/newlib/libc/stdio/vfprintf.c b/newlib/libc/stdio/vfprintf.c index 491369012..44d46f20c 100644 --- a/newlib/libc/stdio/vfprintf.c +++ b/newlib/libc/stdio/vfprintf.c @@ -1029,10 +1029,17 @@ reswitch: switch (ch) { case 'S': #endif sign = '\0'; - if ((cp = GET_ARG (N, ap, char_ptr_t)) == NULL) { + cp = GET_ARG (N, ap, char_ptr_t); +#ifndef __OPTIMIZE_SIZE__ + /* Behavior is undefined if the user passed a + NULL string when precision is not 0. + However, if we are not optimizing for size, + we might as well mirror glibc behavior. */ + if (cp == NULL) { cp = "(null)"; - size = 6; + size = ((unsigned) prec > 6U) ? 6 : prec; } +#endif /* __OPTIMIZE_SIZE__ */ #ifdef _MB_CAPABLE else if (ch == 'S' || (flags & LONGINT)) { mbstate_t ps; -- cgit v1.2.3