From f2fcb018ad4fe9a6cdb7b984d9ace36c39a67882 Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Thu, 4 Mar 2004 19:14:56 +0000 Subject: [PATCH] Update. 2004-03-04 Jakub Jelinek * iconv/iconv_prog.c (process_block): Handle omit_invalid. 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. --- ChangeLog | 9 +++ iconv/iconv_prog.c | 72 +++++++++++-------- .../linux/powerpc/powerpc64/syscalls.list | 4 -- 3 files changed, 53 insertions(+), 32 deletions(-) delete mode 100644 sysdeps/unix/sysv/linux/powerpc/powerpc64/syscalls.list diff --git a/ChangeLog b/ChangeLog index f038f5bc70..dd024437db 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,12 @@ +2004-03-04 Jakub Jelinek + + * iconv/iconv_prog.c (process_block): Handle omit_invalid. + 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. + 2004-03-03 Jakub Jelinek * sysdeps/unix/sysv/linux/x86_64/posix_fadvise.c: New file. diff --git a/iconv/iconv_prog.c b/iconv/iconv_prog.c index 58451f532a..5455974f5c 100644 --- a/iconv/iconv_prog.c +++ b/iconv/iconv_prog.c @@ -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) - { - /* 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; - } + 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) + { + status = EXIT_FAILURE; + + 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; + } } 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; } diff --git a/sysdeps/unix/sysv/linux/powerpc/powerpc64/syscalls.list b/sysdeps/unix/sysv/linux/powerpc/powerpc64/syscalls.list deleted file mode 100644 index fc3553bd67..0000000000 --- a/sysdeps/unix/sysv/linux/powerpc/powerpc64/syscalls.list +++ /dev/null @@ -1,4 +0,0 @@ -# File name Caller Syscall name # args Strong name Weak names - -# System calls with wrappers. -posix_fadvise64 - fadvise64 Vi:iiii posix_fadvise64 posix_fadvise