aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorUlrich Drepper <drepper@redhat.com>2000-06-10 18:20:09 +0000
committerUlrich Drepper <drepper@redhat.com>2000-06-10 18:20:09 +0000
commit3593973b3b44ac65939fea0855e78eb1a96656b1 (patch)
treee278e7a81a5ddb11078c24dc458b426c877252dd
parent3f912ceeba47447a6299957b8bc9dc331c08f294 (diff)
downloadglibc-3593973b3b44ac65939fea0855e78eb1a96656b1.zip
glibc-3593973b3b44ac65939fea0855e78eb1a96656b1.tar.gz
glibc-3593973b3b44ac65939fea0855e78eb1a96656b1.tar.bz2
Update.
* iconv/gconv_simple.c: Fix various small bugs in several functions.
-rw-r--r--ChangeLog3
-rw-r--r--iconv/gconv_simple.c33
2 files changed, 15 insertions, 21 deletions
diff --git a/ChangeLog b/ChangeLog
index 1f78fd4..31a8ca0 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 c2d873e..7a1e634 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; \