about summary refs log tree commit diff
diff options
context:
space:
mode:
authorUlrich Drepper <drepper@redhat.com>2003-01-16 07:21:24 +0000
committerUlrich Drepper <drepper@redhat.com>2003-01-16 07:21:24 +0000
commit41f112ad228d4cd587712b747472a3a9f7e20e5a (patch)
tree6aa35ba2205129873989df4aef484c8b2504205e
parentea31b613feba1c5f34ff2cf9f2c0bab3be4f8d2a (diff)
downloadglibc-41f112ad228d4cd587712b747472a3a9f7e20e5a.tar.gz
glibc-41f112ad228d4cd587712b747472a3a9f7e20e5a.tar.xz
glibc-41f112ad228d4cd587712b747472a3a9f7e20e5a.zip
Update.
	* iconv/loop.c: Use CLEAR_STATE if defined to clear __state element.
-rw-r--r--ChangeLog1
-rw-r--r--iconv/gconv_simple.c11
-rw-r--r--iconv/loop.c7
3 files changed, 14 insertions, 5 deletions
diff --git a/ChangeLog b/ChangeLog
index 57e1c6584a..43a4e7cfda 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -3,6 +3,7 @@
 	* iconv/gconv_simple.c (STORE_REST): Explicitly store the total
 	expected size into state.
 	(UNPACK_BYTES): Do the reverse.
+	* iconv/loop.c: Use CLEAR_STATE if defined to clear __state element.
 	* wcsmbs/tst-mbrtowc.c (utf8_test_1): Add test for the bug.
 	Reported by Al Viro <aviro@redhat.com>.
 
diff --git a/iconv/gconv_simple.c b/iconv/gconv_simple.c
index 124c8e7727..69489d892b 100644
--- a/iconv/gconv_simple.c
+++ b/iconv/gconv_simple.c
@@ -1081,7 +1081,7 @@ ucs4le_internal_loop_single (struct __gconv_step *step,
     ch <<= r * 6;							      \
 									      \
     /* Store the number of bytes expected for the entire sequence.  */	      \
-    ch = ch << 3 | cnt;							      \
+    state->__count |= cnt << 8;						      \
 									      \
     /* Store the value.  */						      \
     state->__value.__wch = ch;						      \
@@ -1091,10 +1091,9 @@ ucs4le_internal_loop_single (struct __gconv_step *step,
   {									      \
     static const unsigned char inmask[5] = { 0xc0, 0xe0, 0xf0, 0xf8, 0xfc };  \
     wint_t wch = state->__value.__wch;					      \
-    size_t ntotal = wch & 7;						      \
-    wch >>= 3;								      \
+    size_t ntotal = state->__count >> 8;				      \
 									      \
-    inlen = state->__count;						      \
+    inlen = state->__count & 255;					      \
 									      \
     bytebuf[0] = inmask[ntotal - 2];					      \
 									      \
@@ -1109,6 +1108,10 @@ ucs4le_internal_loop_single (struct __gconv_step *step,
     bytebuf[0] |= wch;							      \
   }
 
+#define CLEAR_STATE \
+  state->__count = 0
+
+
 #include <iconv/loop.c>
 #include <iconv/skeleton.c>
 
diff --git a/iconv/loop.c b/iconv/loop.c
index b0b76f32e8..d792f4a8a2 100644
--- a/iconv/loop.c
+++ b/iconv/loop.c
@@ -1,5 +1,5 @@
 /* Conversion loop frame work.
-   Copyright (C) 1998, 1999, 2000, 2001, 2002 Free Software Foundation, Inc.
+   Copyright (C) 1998-2002, 2003 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
    Contributed by Ulrich Drepper <drepper@cygnus.com>, 1998.
 
@@ -428,7 +428,11 @@ SINGLE(LOOPFCT) (struct __gconv_step *step,
       result = __GCONV_OK;
 
       /* Clear the state buffer.  */
+#ifdef CLEAR_STATE
+      CLEAR_STATE;
+#else
       state->__count &= ~7;
+#endif
     }
   else if (result == __GCONV_INCOMPLETE_INPUT)
     {
@@ -479,6 +483,7 @@ gconv_btowc (struct __gconv_step *step, unsigned char c)
 #undef UPDATE_PARAMS
 #undef ONEBYTE_BODY
 #undef UNPACK_BYTES
+#undef CLEAR_STATE
 #undef LOOP_NEED_STATE
 #undef LOOP_NEED_FLAGS
 #undef LOOP_NEED_DATA