Author: aurel32
Date: 2012-11-13 16:08:24 +0000 (Tue, 13 Nov 2012)
New Revision: 5372

Added:
   
glibc-package/branches/eglibc-2.16/debian/patches/any/cvs-vfprintf-binary.diff
Modified:
   glibc-package/branches/eglibc-2.16/debian/changelog
   glibc-package/branches/eglibc-2.16/debian/patches/series
Log:
  * patches/any/cvs-vfprintf-binary.diff: new patch from upstream to
    print incomplete multibyte characters without complaint with %.<num>s.
    Closes: #208308.


Modified: glibc-package/branches/eglibc-2.16/debian/changelog
===================================================================
--- glibc-package/branches/eglibc-2.16/debian/changelog 2012-11-13 15:54:37 UTC 
(rev 5371)
+++ glibc-package/branches/eglibc-2.16/debian/changelog 2012-11-13 16:08:24 UTC 
(rev 5372)
@@ -374,6 +374,11 @@
   * Don't run the x32 testsuite on i386/amd64, buildds may lack support.
   * Fix some ln calls in sysdeps/{amd64,i386} for binary-arch idempotence.
 
+  [ Jonathan Nieder ]
+  * patches/any/cvs-vfprintf-binary.diff: new patch from upstream to
+    print incomplete multibyte characters without complaint with %.<num>s.
+    Closes: #208308.
+ 
  -- Aurelien Jarno <[email protected]>  Wed, 25 Jul 2012 23:45:58 +0200
 
 eglibc (2.13-36) unstable; urgency=low

Added: 
glibc-package/branches/eglibc-2.16/debian/patches/any/cvs-vfprintf-binary.diff
===================================================================
--- 
glibc-package/branches/eglibc-2.16/debian/patches/any/cvs-vfprintf-binary.diff  
                            (rev 0)
+++ 
glibc-package/branches/eglibc-2.16/debian/patches/any/cvs-vfprintf-binary.diff  
    2012-11-13 16:08:24 UTC (rev 5372)
@@ -0,0 +1,110 @@
+2012-09-28  Andreas Schwab  <[email protected]>
+
+       [BZ #6530]
+       * stdio-common/vfprintf.c (process_string_arg): Revert
+       2000-07-22 change.
+
+2011-09-28  Jonathan Nieder  <[email protected]>
+
+       * 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.
+
+ stdio-common/Makefile      |  1 +
+ stdio-common/tst-sprintf.c | 13 +++++++++++++
+ stdio-common/vfprintf.c    | 39 +++------------------------------------
+ 3 files changed, 17 insertions(+), 36 deletions(-)
+
+diff --git a/stdio-common/Makefile b/stdio-common/Makefile
+index 1431a211..5625bd3e 100644
+--- a/stdio-common/Makefile
++++ b/stdio-common/Makefile
+@@ -136,6 +136,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 bfa79c9c..42159a26 100644
+--- 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>
+ #include <gnu/option-groups.h>
+ 
+@@ -61,5 +62,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 927c0c26..25a2c5dd 100644
+--- a/stdio-common/vfprintf.c
++++ b/stdio-common/vfprintf.c
+@@ -1180,42 +1180,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 (! LOCALE_SUPPORT                                          \
+-                    ||_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)     \
+-                      {                                                     \
+-                        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);                                          \
+         }                                                                   \

Modified: glibc-package/branches/eglibc-2.16/debian/patches/series
===================================================================
--- glibc-package/branches/eglibc-2.16/debian/patches/series    2012-11-13 
15:54:37 UTC (rev 5371)
+++ glibc-package/branches/eglibc-2.16/debian/patches/series    2012-11-13 
16:08:24 UTC (rev 5372)
@@ -231,3 +231,4 @@
 any/local-disable-libnss-db.diff
 any/local-sunrpc-dos.diff
 any/local-revert-bz13979.diff
+any/cvs-vfprintf-binary.diff


-- 
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]
Archive: http://lists.debian.org/[email protected]

Reply via email to