Don't parse %s format argument as multibyte string

(cherry picked from commit 715a900c9085907fa749589bf738b192b1a2bda5)

2012-09-28  Andreas Schwab  <schwab@linux-m68k.org>

	[BZ #6530]
	* stdio-common/vfprintf.c (process_string_arg): Revert
	2000-07-22 change.

2011-09-28  Jonathan Nieder  <jrnieder@gmail.com>

	* stdio-common/Makefile (tst-sprintf-ENV): Set environment
	for testcase.
	* stdio-common/tst-sprintf.c: Include <locale.h>
	(main): Test sprintf's handling of incomplete multibyte
	characters.

diff --git a/ChangeLog b/ChangeLog
index cddc90e4bf0b9934865465d3f97f37253cacba38..7c7230058c3ee63a4ee52e64d49c47ef70567e8c 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,17 @@
+2012-09-28  Andreas Schwab  <schwab@linux-m68k.org>
+
+       [BZ #6530]
+       * stdio-common/vfprintf.c (process_string_arg): Revert
+       2000-07-22 change.
+
+2011-09-28  Jonathan Nieder  <jrnieder@gmail.com>
+
+       * stdio-common/Makefile (tst-sprintf-ENV): Set environment
+       for testcase.
+       * stdio-common/tst-sprintf.c: Include <locale.h>
+       (main): Test sprintf's handling of incomplete multibyte
+       characters.
+
 2012-09-25  Joseph Myers  <joseph@codesourcery.com>
 
        [BZ #14621]

diff --git a/NEWS b/NEWS
index 48358b5124ef5e3b4d65c071c8906ea99ed317fc..50aec9ddc8fb9939f701c5b77d253155a3b27cdd 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -9,7 +9,7 @@ Version 2.16.1
 
 * The following bugs are resolved with this release:
 
-  14195, 14459, 14476, 14562, 14621
+  6530, 14195, 14459, 14476, 14562, 14621
 
 Version 2.16
 

diff --git a/stdio-common/Makefile b/stdio-common/Makefile
index a29924bc7cfb6010a5af296efdf8c5357ed412b1..42e45dc1c3a7c3d14ce7b0626bade7a542325dd1 100644 (file)
--- a/stdio-common/Makefile
+++ b/stdio-common/Makefile
@@ -116,6 +116,7 @@ CFLAGS-scanf17.c = -I../libio -I../stdlib -I../wcsmbs -I../time -I../string \
 
 # We know the test has a format string problem.
 CFLAGS-tst-sprintf.c = -Wno-format
+tst-sprintf-ENV = LOCPATH=$(common-objpfx)localedata
 tst-sscanf-ENV = LOCPATH=$(common-objpfx)localedata
 tst-swprintf-ENV = LOCPATH=$(common-objpfx)localedata
 test-vfprintf-ENV = LOCPATH=$(common-objpfx)localedata

diff --git a/stdio-common/tst-sprintf.c b/stdio-common/tst-sprintf.c
index c04fef18f4b14cc22943a71985e00fcbb45e22fb..c4e911fe94db2d174a0b05f1d890800d688aa2b8 100644 (file)
--- a/stdio-common/tst-sprintf.c
+++ b/stdio-common/tst-sprintf.c
@@ -1,5 +1,6 @@
 #include <stdio.h>
 #include <stdlib.h>
+#include <locale.h>
 #include <string.h>
 
 
@@ -58,5 +59,17 @@ main (void)
       result = 1;
     }
 
+  if (setlocale (LC_ALL, "de_DE.UTF-8") == NULL)
+    {
+      puts ("cannot set locale");
+      result = 1;
+    }
+  else if (sprintf (buf, "%.8s\n", "Foo: \277") != 7
+          || strcmp (buf, "Foo: \277\n") != 0)
+    {
+      printf ("sprintf (buf, \"%%.8s\\n\", \"Foo: \\277\") produced '%s' output\n", buf);
+      result = 1;
+    }
+
   return result;
 }

diff --git a/stdio-common/vfprintf.c b/stdio-common/vfprintf.c
index d5690342536bc8cf948c786f663bb63f73f91f3a..17d3f42a9752557f1db4c84e179c521cac47944e 100644 (file)
--- a/stdio-common/vfprintf.c
+++ b/stdio-common/vfprintf.c
@@ -1168,42 +1168,9 @@ vfprintf (FILE *s, const CHAR_T *format, va_list ap)
        else if (!is_long && spec != L_('S'))                                 \
          {                                                                   \
            if (prec != -1)                                                   \
-             {                                                               \
-               /* Search for the end of the string, but don't search past    \
-                  the length (in bytes) specified by the precision.  Also    \
-                  don't use incomplete characters.  */                       \
-               if (_NL_CURRENT_WORD (LC_CTYPE, _NL_CTYPE_MB_CUR_MAX) == 1)   \
-                 len = __strnlen (string, prec);                             \
-               else                                                          \
-                 {                                                           \
-                   /* In case we have a multibyte character set the          \
-                      situation is more complicated.  We must not copy       \
-                      bytes at the end which form an incomplete character. */\
-                   size_t ignore_size = (unsigned) prec > 1024 ? 1024 : prec;\
-                   wchar_t ignore[ignore_size];                              \
-                   const char *str2 = string;                                \
-                   const char *strend = string + prec;                       \
-                   if (strend < string)                                      \
-                     strend = (const char *) UINTPTR_MAX;                    \
-                                                                             \
-                   mbstate_t ps;                                             \
-                   memset (&ps, '\0', sizeof (ps));                          \
-                                                                             \
-                   while (str2 != NULL && str2 < strend)                     \
-                     if (__mbsnrtowcs (ignore, &str2, strend - str2,         \
-                                       ignore_size, &ps) == (size_t) -1)     \
-                       {                                                     \
-                         /* Conversion function has set errno.  */           \
-                         done = -1;                                          \
-                         goto all_done;                                      \
-                       }                                                     \
-                                                                             \
-                   if (str2 == NULL)                                         \
-                     len = strlen (string);                                  \
-                   else                                                      \
-                     len = str2 - string - (ps.__count & 7);                 \
-                 }                                                           \
-             }                                                               \
+             /* Search for the end of the string, but don't search past      \
+                the length (in bytes) specified by the precision.  */        \
+             len = __strnlen (string, prec);                                 \
            else                                                              \
              len = strlen (string);                                          \
          }                                                                   \