From 531950cebdce5556d5ad7fe3dcd74f78956f3161 Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Sun, 7 Jan 2001 04:34:56 +0000 Subject: Update. 2001-01-06 Ulrich Drepper * iconvdata/ibm937.c (BODY to IBM937): Correct several mistakes in buffer and table handling. --- iconvdata/ibm937.c | 20 +++++++++++--------- 1 file changed, 11 insertions(+), 9 deletions(-) (limited to 'iconvdata/ibm937.c') diff --git a/iconvdata/ibm937.c b/iconvdata/ibm937.c index 6da55ae883..4fee56a506 100644 --- a/iconvdata/ibm937.c +++ b/iconvdata/ibm937.c @@ -188,13 +188,15 @@ enum \ /* Use the UCS4 table for single byte. */ \ cp = __ucs4_to_ibm937sb[ch]; \ - if (__builtin_expect (ch >= sizeof (__ucs4_to_ibm937sb) \ - / sizeof (__ucs4_to_ibm937sb[0]), 0) \ + if (__builtin_expect (ch >= (sizeof (__ucs4_to_ibm937sb) \ + / sizeof (__ucs4_to_ibm937sb[0])), 0) \ || (__builtin_expect (cp[0], '\1') == '\0' && ch != 0)) \ { \ /* Use the UCS4 table for double byte. */ \ cp = __ucs4_to_ibm937db[ch]; \ - if (__builtin_expect (cp[0], '\1') == '\0' && ch != 0) \ + if (__builtin_expect (ch >= (sizeof (__ucs4_to_ibm937db) \ + / sizeof (__ucs4_to_ibm937db[0])), 0) \ + || __builtin_expect (cp[0], '\1') == '\0') \ { \ /* This is an illegal character. */ \ if (! ignore_errors_p ()) \ @@ -209,13 +211,13 @@ enum if (curcs == sb) \ { \ *outptr++ = SO; \ - if (__builtin_expect (outptr == outend, 0)) \ - { \ - result = __GCONV_FULL_OUTPUT; \ - break; \ - } \ curcs = db; \ } \ + if (__builtin_expect (outptr + 1 >= outend, 0)) \ + { \ + result = __GCONV_FULL_OUTPUT; \ + break; \ + } \ *outptr++ = cp[0]; \ *outptr++ = cp[1]; \ } \ @@ -225,13 +227,13 @@ enum if (curcs == db) \ { \ *outptr++ = SI; \ + curcs = sb; \ if (__builtin_expect (outptr == outend, 0)) \ { \ result = __GCONV_FULL_OUTPUT; \ break; \ } \ } \ - curcs = sb; \ *outptr++ = cp[0]; \ } \ \ -- cgit v1.2.3