(PREPARE_LOOP): Return __GCONV_EMPTY_INPUT only if input is really
[kopensolaris-gnu/glibc.git] / iconvdata / utf-16.c
index 91e3390..b816508 100644 (file)
@@ -1,22 +1,22 @@
 /* Conversion module for UTF-16.
-   Copyright (C) 1999, 2000 Free Software Foundation, Inc.
+   Copyright (C) 1999, 2000-2002 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
    Contributed by Ulrich Drepper <drepper@cygnus.com>, 1999.
 
    The GNU C Library is free software; you can redistribute it and/or
-   modify it under the terms of the GNU Library General Public License as
-   published by the Free Software Foundation; either version 2 of the
-   License, or (at your option) any later version.
+   modify it under the terms of the GNU Lesser General Public
+   License as published by the Free Software Foundation; either
+   version 2.1 of the License, or (at your option) any later version.
 
    The GNU C Library is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
-   Library General Public License for more details.
+   Lesser General Public License for more details.
 
-   You should have received a copy of the GNU Library General Public
-   License along with the GNU C Library; see the file COPYING.LIB.  If not,
-   write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
-   Boston, MA 02111-1307, USA.  */
+   You should have received a copy of the GNU Lesser General Public
+   License along with the GNU C Library; if not, write to the Free
+   Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA
+   02111-1307 USA.  */
 
 #include <byteswap.h>
 #include <dlfcn.h>
 #define PREPARE_LOOP \
   enum direction dir = ((struct utf16_data *) step->__data)->dir;            \
   enum variant var = ((struct utf16_data *) step->__data)->var;                      \
-  int swap = ((struct utf16_data *) step->__data)->swap;                     \
+  int swap;                                                                  \
   if (FROM_DIRECTION && var == UTF_16)                                       \
     {                                                                        \
       if (data->__invocation_counter == 0)                                   \
        {                                                                     \
          /* We have to find out which byte order the file is encoded in.  */ \
          if (inptr + 2 > inend)                                              \
-           return __GCONV_EMPTY_INPUT;                                       \
+           return (inptr == inend                                            \
+                   ? __GCONV_EMPTY_INPUT : __GCONV_INCOMPLETE_INPUT);        \
                                                                              \
          if (get16u (inptr) == BOM)                                          \
            /* Simply ignore the BOM character.  */                           \
-           inptr += 2;                                                       \
+           *inptrp = inptr += 2;                                             \
          else if (get16u (inptr) == BOM_OE)                                  \
            {                                                                 \
              ((struct utf16_data *) step->__data)->swap = 1;                 \
-             inptr += 2;                                                     \
+             *inptrp = inptr += 2;                                           \
            }                                                                 \
        }                                                                     \
     }                                                                        \
@@ -72,7 +73,8 @@
                                                                              \
       put16u (outbuf, BOM);                                                  \
       outbuf += 2;                                                           \
-    }
+    }                                                                        \
+  swap = ((struct utf16_data *) step->__data)->swap;
 #define EXTRA_LOOP_ARGS                , var, swap
 
 
@@ -100,6 +102,7 @@ struct utf16_data
 };
 
 
+extern int gconv_init (struct __gconv_step *step);
 int
 gconv_init (struct __gconv_step *step)
 {
@@ -158,7 +161,7 @@ gconv_init (struct __gconv_step *step)
          if (dir == from_utf16)
            {
              step->__min_needed_from = MIN_NEEDED_FROM;
-             step->__max_needed_from = MIN_NEEDED_FROM;
+             step->__max_needed_from = MAX_NEEDED_FROM;
              step->__min_needed_to = MIN_NEEDED_TO;
              step->__max_needed_to = MIN_NEEDED_TO;
            }
@@ -167,7 +170,7 @@ gconv_init (struct __gconv_step *step)
              step->__min_needed_from = MIN_NEEDED_TO;
              step->__max_needed_from = MIN_NEEDED_TO;
              step->__min_needed_to = MIN_NEEDED_FROM;
-             step->__max_needed_to = MIN_NEEDED_FROM;
+             step->__max_needed_to = MAX_NEEDED_FROM;
            }
 
          step->__stateful = 0;
@@ -180,6 +183,7 @@ gconv_init (struct __gconv_step *step)
 }
 
 
+extern void gconv_end (struct __gconv_step *data);
 void
 gconv_end (struct __gconv_step *data)
 {
@@ -202,11 +206,9 @@ gconv_end (struct __gconv_step *data)
           We must catch this.  If we let surrogates pass through,            \
           attackers could make a security hole exploit by                    \
           synthesizing any desired plane 1-16 character.  */                 \
+       result = __GCONV_ILLEGAL_INPUT;                                       \
        if (! ignore_errors_p ())                                             \
-         {                                                                   \
-           result = __GCONV_ILLEGAL_INPUT;                                   \
-           break;                                                            \
-         }                                                                   \
+         break;                                                              \
        inptr += 4;                                                           \
        ++*irreversible;                                                      \
        continue;                                                             \
@@ -214,11 +216,11 @@ gconv_end (struct __gconv_step *data)
                                                                              \
     if (swap)                                                                \
       {                                                                              \
-       if (__builtin_expect (c, 0) >= 0x10000)                               \
+       if (__builtin_expect (c >= 0x10000, 0))                               \
          {                                                                   \
-           if (__builtin_expect (c, 0) >= 0x110000)                          \
+           if (__builtin_expect (c >= 0x110000, 0))                          \
              {                                                               \
-               STANDARD_ERR_HANDLER (4);                                     \
+               STANDARD_TO_LOOP_ERR_HANDLER (4);                             \
              }                                                               \
                                                                              \
            /* Generate a surrogate character.  */                            \
@@ -238,11 +240,11 @@ gconv_end (struct __gconv_step *data)
       }                                                                              \
     else                                                                     \
       {                                                                              \
-       if (__builtin_expect (c, 0) >= 0x10000)                               \
+       if (__builtin_expect (c >= 0x10000, 0))                               \
          {                                                                   \
-           if (__builtin_expect (c, 0) >= 0x110000)                          \
+           if (__builtin_expect (c >= 0x110000, 0))                          \
              {                                                               \
-               STANDARD_ERR_HANDLER (4);                                     \
+               STANDARD_TO_LOOP_ERR_HANDLER (4);                             \
              }                                                               \
                                                                              \
            /* Generate a surrogate character.  */                            \
@@ -282,7 +284,7 @@ gconv_end (struct __gconv_step *data)
       {                                                                              \
        u1 = bswap_16 (u1);                                                   \
                                                                              \
-       if (__builtin_expect (u1, 0) < 0xd800 || u1 > 0xdfff)                 \
+       if (__builtin_expect (u1 < 0xd800, 1) || u1 > 0xdfff)                 \
          {                                                                   \
            /* No surrogate.  */                                              \
            put32 (outptr, u1);                                               \
@@ -304,19 +306,12 @@ gconv_end (struct __gconv_step *data)
                                                                              \
            inptr += 2;                                                       \
            u2 = bswap_16 (get16 (inptr));                                    \
-           if (__builtin_expect (u2, 0xdc00) < 0xdc00                        \
-               || __builtin_expect (u2, 0xdc00) >= 0xdfff)                   \
+           if (__builtin_expect (u2 < 0xdc00, 0)                             \
+               || __builtin_expect (u2 == 0xdfff, 0))                        \
              {                                                               \
                /* This is no valid second word for a surrogate.  */          \
-               if (! ignore_errors_p ())                                     \
-                 {                                                           \
-                   inptr -= 2;                                               \
-                   result = __GCONV_ILLEGAL_INPUT;                           \
-                   break;                                                    \
-                 }                                                           \
-                                                                             \
-               ++*irreversible;                                              \
-               continue;                                                     \
+               inptr -= 2;                                                   \
+               STANDARD_FROM_LOOP_ERR_HANDLER (2);                           \
              }                                                               \
                                                                              \
            put32 (outptr, ((u1 - 0xd7c0) << 10) + (u2 - 0xdc00));            \
@@ -325,7 +320,7 @@ gconv_end (struct __gconv_step *data)
       }                                                                              \
     else                                                                     \
       {                                                                              \
-       if (__builtin_expect (u1, 0) < 0xd800 || u1 > 0xdfff)                 \
+       if (__builtin_expect (u1 < 0xd800, 1) || u1 > 0xdfff)                 \
          {                                                                   \
            /* No surrogate.  */                                              \
            put32 (outptr, u1);                                               \
@@ -347,19 +342,12 @@ gconv_end (struct __gconv_step *data)
                                                                              \
            inptr += 2;                                                       \
            u2 = get16 (inptr);                                               \
-           if (__builtin_expect (u2, 0xdc00) < 0xdc00                        \
-               || __builtin_expect (u2, 0xdc00) >= 0xdfff)                   \
+           if (__builtin_expect (u2 < 0xdc00, 0)                             \
+               || __builtin_expect (u2 >= 0xdfff, 0))                        \
              {                                                               \
                /* This is no valid second word for a surrogate.  */          \
-               if (! ignore_errors_p ())                                     \
-                 {                                                           \
-                   inptr -= 2;                                               \
-                   result = __GCONV_ILLEGAL_INPUT;                           \
-                   break;                                                    \
-                 }                                                           \
-                                                                             \
-               ++*irreversible;                                              \
-               continue;                                                     \
+               inptr -= 2;                                                   \
+               STANDARD_FROM_LOOP_ERR_HANDLER (2);                           \
              }                                                               \
                                                                              \
            put32 (outptr, ((u1 - 0xd7c0) << 10) + (u2 - 0xdc00));            \