diff options
| author | Ulrich Drepper <drepper@redhat.com> | 2000-06-10 18:20:09 +0000 |
|---|---|---|
| committer | Ulrich Drepper <drepper@redhat.com> | 2000-06-10 18:20:09 +0000 |
| commit | 3593973b3b44ac65939fea0855e78eb1a96656b1 (patch) | |
| tree | e278e7a81a5ddb11078c24dc458b426c877252dd | |
| parent | 3f912ceeba47447a6299957b8bc9dc331c08f294 (diff) | |
| download | glibc-3593973b3b44ac65939fea0855e78eb1a96656b1.tar.gz glibc-3593973b3b44ac65939fea0855e78eb1a96656b1.tar.xz glibc-3593973b3b44ac65939fea0855e78eb1a96656b1.zip | |
Update.
* iconv/gconv_simple.c: Fix various small bugs in several functions.
| -rw-r--r-- | ChangeLog | 3 | ||||
| -rw-r--r-- | iconv/gconv_simple.c | 33 |
2 files changed, 15 insertions, 21 deletions
diff --git a/ChangeLog b/ChangeLog index 1f78fd4cf2..31a8ca0013 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,8 @@ 2000-06-10 Ulrich Drepper <drepper@redhat.com> + * iconv/gconv_simple.c: Fix various small bugs in several + functions. + * iconv/loop.c: Fix comment. 2000-06-10 Jakub Jelinek <jakub@redhat.com> diff --git a/iconv/gconv_simple.c b/iconv/gconv_simple.c index c2d873e21d..7a1e6344a2 100644 --- a/iconv/gconv_simple.c +++ b/iconv/gconv_simple.c @@ -238,7 +238,7 @@ ucs4_internal_loop (const unsigned char **inptrp, const unsigned char *inend, return __GCONV_ILLEGAL_INPUT; } - *((uint32_t *) outptr)++ = bswap_32 (*(uint32_t *) inptr); + *((uint32_t *) outptr)++ = inval; } *inptrp = inptr; @@ -297,18 +297,6 @@ ucs4_internal_loop_unaligned (const unsigned char **inptrp, outptr[2] = inptr[2]; outptr[3] = inptr[3]; # endif - -# if __BYTE_ORDER == __LITTLE_ENDIAN - outptr[3] = inptr[0]; - outptr[2] = inptr[1]; - outptr[1] = inptr[2]; - outptr[0] = inptr[3]; -# else - outptr[0] = inptr[0]; - outptr[1] = inptr[1]; - outptr[2] = inptr[2]; - outptr[3] = inptr[3]; -# endif } *inptrp = inptr; @@ -353,7 +341,10 @@ ucs4_internal_loop_single (const unsigned char **inptrp, { /* The value is too large. */ if (!(flags & __GCONV_IGNORE_ERRORS)) - return __GCONV_ILLEGAL_INPUT; + { + *inptrp -= cnt - (state->__count & 7); + return __GCONV_ILLEGAL_INPUT; + } } else { @@ -448,7 +439,7 @@ internal_ucs4le_loop_unaligned (const unsigned char **inptrp, /* Sigh, we have to do some real work. */ size_t cnt; - for (cnt = 0; cnt < n_convert; ++cnt, inptr += 4) + for (cnt = 0; cnt < n_convert; ++cnt, inptr += 4, outptr += 4) { outptr[0] = inptr[3]; outptr[1] = inptr[2]; @@ -566,7 +557,7 @@ ucs4le_internal_loop (const unsigned char **inptrp, const unsigned char *inend, return __GCONV_ILLEGAL_INPUT; } - *((uint32_t *) outptr)++ = bswap_32 (*(uint32_t *) inptr); + *((uint32_t *) outptr)++ = inval; } *inptrp = inptr; @@ -752,6 +743,7 @@ ucs4le_internal_loop_single (const unsigned char **inptrp, #define LOOPFCT FROM_LOOP #define BODY \ { \ + /* XXX unaligned. */ \ if (__builtin_expect (*((uint32_t *) inptr), 0) > 0x7f) \ { \ if (! ignore_errors_p ()) \ @@ -766,7 +758,6 @@ ucs4le_internal_loop_single (const unsigned char **inptrp, } \ else \ /* It's an one byte sequence. */ \ - /* XXX unaligned. */ \ *outptr++ = *((uint32_t *) inptr)++; \ } #include <iconv/loop.c> @@ -875,13 +866,13 @@ ucs4le_internal_loop_single (const unsigned char **inptrp, cnt = 2; \ ch &= 0x1f; \ } \ - else if (__builtin_expect (ch & 0xf0, 0xf0) == 0xe0) \ + else if (__builtin_expect (ch & 0xf0, 0xe0) == 0xe0) \ { \ /* We expect three bytes. */ \ cnt = 3; \ ch &= 0x0f; \ } \ - else if (__builtin_expect (ch & 0xf8, 0xf8) == 0xf0) \ + else if (__builtin_expect (ch & 0xf8, 0xf0) == 0xf0) \ { \ /* We expect four bytes. */ \ cnt = 4; \ @@ -1012,13 +1003,13 @@ ucs4le_internal_loop_single (const unsigned char **inptrp, cnt = 3; \ ch &= 0x0f; \ } \ - else if (__builtin_expect (ch & 0xf8, 0xf8) == 0xf0) \ + else if (__builtin_expect (ch & 0xf8, 0xf0) == 0xf0) \ { \ /* We expect four bytes. */ \ cnt = 4; \ ch &= 0x07; \ } \ - else if (__builtin_expect (ch & 0xfc, 0xfc) == 0xf8) \ + else if (__builtin_expect (ch & 0xfc, 0xf8) == 0xf8) \ { \ /* We expect five bytes. */ \ cnt = 5; \ |