(process_block): Handle omit_invalid.
authordrepper <drepper>
Thu, 4 Mar 2004 19:12:54 +0000 (19:12 +0000)
committerdrepper <drepper>
Thu, 4 Mar 2004 19:12:54 +0000 (19:12 +0000)
If iconv returns EILSEQ with omit_invalid, continue converting
and return 1 if no other errors are seen.
(main): Set status to EXIT_FAILURE whenever process_* returns
nonzero, but only stop processing further files if iy returns
negative value.

iconv/iconv_prog.c

index 58451f5..5455974 100644 (file)
@@ -280,7 +280,7 @@ conversion from `%s' and to `%s' are not supported"),
            struct stat st;
            char *addr;
 #endif
-           int fd;
+           int fd, ret;
 
            if (verbose)
              printf ("%s:\n", argv[remaining]);
@@ -313,43 +313,42 @@ conversion from `%s' and to `%s' are not supported"),
                         _("error while closing input `%s'"),
                         argv[remaining]);
 
-               if (process_block (cd, addr, st.st_size, output) < 0)
+               ret = process_block (cd, addr, st.st_size, output);
+
+               /* We don't need the input data anymore.  */
+               munmap ((void *) addr, st.st_size);
+
+               if (ret != 0)
                  {
-                   /* Something went wrong.  */
                    status = EXIT_FAILURE;
 
-                   /* We don't need the input data anymore.  */
-                   munmap ((void *) addr, st.st_size);
-
-                   /* We cannot go on with producing output since it might
-                      lead to problem because the last output might leave
-                      the output stream in an undefined state.  */
-                   break;
+                   if (ret < 0)
+                     /* We cannot go on with producing output since it might
+                        lead to problem because the last output might leave
+                        the output stream in an undefined state.  */
+                     break;
                  }
-
-               /* We don't need the input data anymore.  */
-               munmap ((void *) addr, st.st_size);
              }
            else
 #endif /* _POSIX_MAPPED_FILES */
              {
                /* Read the file in pieces.  */
-               if (process_fd (cd, fd, output) != 0)
+               ret = process_fd (cd, fd, output);
+
+               /* Now close the file.  */
+               close (fd);
+
+               if (ret != 0)
                  {
                    /* Something went wrong.  */
                    status = EXIT_FAILURE;
 
-                   /* We don't need the input file anymore.  */
-                   close (fd);
-
-                   /* We cannot go on with producing output since it might
-                      lead to problem because the last output might leave
-                      the output stream in an undefined state.  */
-                   break;
+                   if (ret < 0)
+                     /* We cannot go on with producing output since it might
+                        lead to problem because the last output might leave
+                        the output stream in an undefined state.  */
+                     break;
                  }
-
-               /* Now close the file.  */
-               close (fd);
              }
          }
        while (++remaining < argc);
@@ -438,6 +437,7 @@ process_block (iconv_t cd, char *addr, size_t len, FILE *output)
   char *outptr;
   size_t outlen;
   size_t n;
+  int ret = 0;
 
   while (len > 0)
     {
@@ -445,6 +445,15 @@ process_block (iconv_t cd, char *addr, size_t len, FILE *output)
       outlen = OUTBUF_SIZE;
       n = iconv (cd, &addr, &len, &outptr, &outlen);
 
+      if (n == (size_t) -1 && omit_invalid && errno == EILSEQ)
+       {
+         ret = 1;
+         if (len == 0)
+           n = 0;
+         else
+           errno = E2BIG;
+       }
+
       if (outptr != outbuf)
        {
          /* We have something to write out.  */
@@ -469,7 +478,7 @@ conversion stopped due to problem in writing the output"));
              character sets we have to flush the state now.  */
          outptr = outbuf;
          outlen = OUTBUF_SIZE;
-         (void) iconv (cd, NULL, NULL, &outptr, &outlen);
+         n = iconv (cd, NULL, NULL, &outptr, &outlen);
 
          if (outptr != outbuf)
            {
@@ -489,7 +498,14 @@ conversion stopped due to problem in writing the output"));
              errno = errno_save;
            }
 
-         break;
+         if (n != (size_t) -1)
+           break;
+
+         if (omit_invalid && errno == EILSEQ)
+           {
+             ret = 1;
+             break;
+           }
        }
 
       if (errno != E2BIG)
@@ -518,7 +534,7 @@ incomplete character or shift sequence at end of buffer"));
        }
     }
 
-  return 0;
+  return ret;
 }