From patchwork Fri Mar 27 21:18:37 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Carlos O'Donell X-Patchwork-Id: 38644 Return-Path: X-Original-To: libc-alpha@sourceware.org Delivered-To: libc-alpha@sourceware.org Received: from us-smtp-delivery-74.mimecast.com (us-smtp-delivery-74.mimecast.com [63.128.21.74]) by sourceware.org (Postfix) with ESMTP id 1FE3F385E009 for ; Fri, 27 Mar 2020 21:18:51 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 1FE3F385E009 Received: from mail-qv1-f71.google.com (mail-qv1-f71.google.com [209.85.219.71]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-446-kFvfP8fPNVSkVWs3bXt6cQ-1; Fri, 27 Mar 2020 17:18:40 -0400 X-MC-Unique: kFvfP8fPNVSkVWs3bXt6cQ-1 Received: by mail-qv1-f71.google.com with SMTP id j7so8812413qvy.22 for ; Fri, 27 Mar 2020 14:18:40 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:to:from:subject:organization:message-id:date :user-agent:mime-version:content-language:content-transfer-encoding; bh=XdVI8D4+lFd8Jqx/DwQ9BVjuNUoYnF24wNzDQgT61Ps=; b=TLnKQ7NJr1zdzu995naCxCu0yDpLEO4YiurqPf58PZjqx9aQeRiJIWd+ngP8QUM2AX xUsAE6dtydOM5uRulh8niLMhkQbBQfcooSgkOmcKv2+LP4sf7chsHm9vDeVgtJzCx8iv sPqBYOU68nYbyhBRgnp28/84olinqg/6eNhuseyjDiwrmNE0E/VnZgn+xtKrrrp6nNfx YGBdFu5YvVfhQVyPpRintSFU8iqWqu6slx8kRPbseSYeJJGXmpcdHij8Hb6xIzEmxeMj tJPQLvLfdO3y1sDCVHUf6Q4BUQDqHYLSwS0GKqm7RS/T2WfeCWKVg4FawzSDC6kOAjVK wxNA== X-Gm-Message-State: ANhLgQ3wM455jkS1TDZOOXxxrzALjCnIMpppa6E1M4X6IW2luxtrl+g8 9WMeG3E7TE8aQeLxJyD3Yo7+iyN5VFeZKi6fjdWxgukX7+fqkefgsYQ4NlQ7V42zJ+kV03ahy8H 2RgbXpoD5IUBnxaYlr0/6 X-Received: by 2002:aed:2314:: with SMTP id h20mr1358367qtc.184.1585343919620; Fri, 27 Mar 2020 14:18:39 -0700 (PDT) X-Google-Smtp-Source: ADFU+vvPmnnATCBk03yec6stcSWm8qTZ/G5QM4ueZpYNbLiTPtkv9g+P29RvhC7zZuxFalcAwLboew== X-Received: by 2002:aed:2314:: with SMTP id h20mr1358343qtc.184.1585343919304; Fri, 27 Mar 2020 14:18:39 -0700 (PDT) Received: from [192.168.1.4] (198-84-170-103.cpe.teksavvy.com. [198.84.170.103]) by smtp.gmail.com with ESMTPSA id s188sm4491379qkh.67.2020.03.27.14.18.38 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 27 Mar 2020 14:18:38 -0700 (PDT) To: Andreas Schwab , libc-alpha , Rafal Luzynski From: Carlos O'Donell Subject: [PATCH] Reset converter state after second wchar_t output (Bug 25734) Organization: Red Hat Message-ID: Date: Fri, 27 Mar 2020 17:18:37 -0400 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.5.0 MIME-Version: 1.0 Content-Language: en-US X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com X-Spam-Status: No, score=-23.7 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, KAM_SHORT, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: libc-alpha@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Libc-alpha mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 27 Mar 2020 21:18:53 -0000 Andreas, Does this look good to you? I admit it is odd that the converter returns 0-bytes consumed on the second conversion, it could be enhanced to correctly show that we're really consume 1-byte at a time, but that's another bug. 8< --- 8< --- 8< --- >From ea8a94fafeebc87ccd030c825484328a6da43f47 Mon Sep 17 00:00:00 2001 From: Carlos O'Donell Date: Fri, 27 Mar 2020 17:03:36 -0400 Subject: [PATCH] Reset converter state after second wchar_t output (Bug 25734) An input BIG5-HKSCS character may be converted into at most 2 whcar_t characters. After outputting the second whcar_t character (which was saved in the converter state) we must reset the state. If we fail to reset the state we will be stuck continually copying that character to the output even if we have further input to consider. We add a new test case that covers the 4 BIG5-HKSCS characters that may become 2 wchar_t characters. Reviewed-by: Tom Honermann --- iconvdata/Makefile | 6 +- iconvdata/big5hkscs.c | 3 + iconvdata/tst-iconv-big5-hkscs-to-2ucs4.c | 146 ++++++++++++++++++++++ 3 files changed, 154 insertions(+), 1 deletion(-) create mode 100644 iconvdata/tst-iconv-big5-hkscs-to-2ucs4.c diff --git a/iconvdata/Makefile b/iconvdata/Makefile index c83962f351..71ddef09db 100644 --- a/iconvdata/Makefile +++ b/iconvdata/Makefile @@ -73,7 +73,7 @@ modules.so := $(addsuffix .so, $(modules)) ifeq (yes,$(build-shared)) tests = bug-iconv1 bug-iconv2 tst-loading tst-e2big tst-iconv4 bug-iconv4 \ tst-iconv6 bug-iconv5 bug-iconv6 tst-iconv7 bug-iconv8 bug-iconv9 \ - bug-iconv10 bug-iconv11 bug-iconv12 + bug-iconv10 bug-iconv11 bug-iconv12 tst-iconv-big5-hkscs-to-2ucs4 ifeq ($(have-thread-library),yes) tests += bug-iconv3 endif @@ -340,3 +340,7 @@ tst-tables-clean: $(objpfx)gconv-modules: gconv-modules cat $(sysdeps-gconv-modules) $^ > $@ + +# Test requires BIG5HKSCS. +$(objpfx)tst-iconv-big5-hkscs-to-2ucs4.out: $(objpfx)gconv-modules \ + $(addprefix $(objpfx),BIG5HKSCS.so) diff --git a/iconvdata/big5hkscs.c b/iconvdata/big5hkscs.c index 01fcfeba76..ef325119b1 100644 --- a/iconvdata/big5hkscs.c +++ b/iconvdata/big5hkscs.c @@ -17895,6 +17895,9 @@ static struct else \ ++inptr; \ } \ + else \ + /* Clear the queue and proceed to output the saved character. */ \ + *statep = 0; \ \ put32 (outptr, ch); \ outptr += 4; \ diff --git a/iconvdata/tst-iconv-big5-hkscs-to-2ucs4.c b/iconvdata/tst-iconv-big5-hkscs-to-2ucs4.c new file mode 100644 index 0000000000..a10b8fefff --- /dev/null +++ b/iconvdata/tst-iconv-big5-hkscs-to-2ucs4.c @@ -0,0 +1,146 @@ +/* Verify the BIG5HKSCS outputs that generate 2 wchar_t's (Bug 25734). + Copyright (C) 2020 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + 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 + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + . */ + +#include +#include +#include +#include +#include +#include + +/* A few BIG5-HKSCS characters map in two unicode code points. + They are: + /x88/x62 => + /x88/x64 => + /x88/xa3 => + /x88/xa5 => + Each of these is special cased in iconvdata/big5hkscs.c. + This test ensures that we correctly reset the shift state after + outputting any of these characters. We do this by converting + each them followed by converting an ASCII character. If we fail + to reset the shift state (bug 25734) then we'll see the last + character in the queue output again. */ + +/* Each test has name, input bytes, and expected wide character + output. */ +struct testdata { + const char *name; + const char input[3]; + wchar_t expected[3]; +}; + +/* In BIG5-HKSCS (2008) there are 4 characters that generate multiple + wide characters. */ +struct testdata tests[4] = { + /* X => X */ + { "", "\x88\x62\x58", { 0x00CA, 0x0304, 0x0058 } }, + /* X => X */ + { "", "\x88\x64\x58", { 0x00CA, 0x030C, 0x0058 } }, + /* X => X */ + { "", "\x88\xa3\x58", { 0x00EA, 0x0304, 0x0058 } }, + /* X => X */ + { "", "\x88\xa5\x58", { 0x00EA, 0x030C, 0x0058 } } +}; + +/* Each test is of the form: + - Translate first code sequence (two bytes) + - Translate second (zero bytes) + - Translate the third (one byte). */ +static int +check_conversion (struct testdata test) +{ + wchar_t wc; + mbstate_t st; + size_t ret; + const char *mbs = test.input; + int consumed = 0; + /* Input is always 3 bytes long. */ + int inlen = 3; + + memset (&st, 0, sizeof (st)); + /* First conversion: Consumes first 2 bytes. */ + ret = mbrtowc (&wc, mbs, inlen - consumed, &st); + if (ret != 2) + { + printf ("error: First conversion consumed only %zd bytes.\n", ret); + return 1; + } + /* Advance the two consumed bytes. */ + mbs += ret; + consumed += ret; + if (wc != test.expected[0]) + { + printf ("error: Result of first conversion was wrong.\n"); + return 1; + } + /* Second conversion: Consumes 0 bytes. */ + ret = mbrtowc (&wc, mbs, inlen - consumed, &st); + if (ret != 0) + { + printf ("error: Second conversion consumed only %zd bytes.\n", ret); + return 1; + } + /* Advance the zero consumed bytes. */ + mbs += ret; + consumed += ret; + if (wc != test.expected[1]) + { + printf ("error: Result of second conversion was wrong.\n"); + return 1; + } + /* Third conversion: Consumes 1 byte (it's an ASCII character). */ + ret = mbrtowc (&wc, mbs, inlen - consumed, &st); + if (ret != 1) + { + printf ("error: Third conversion consumed only %zd bytes.\n", ret); + return 1; + } + /* Don't advance any bytes becuase none were consumed. */ + if (wc != test.expected[2]) + { + printf ("error: Result of first conversion was wrong.\n"); + return 1; + } + /* Success. */ + return 0; +} + +static int +do_test (void) +{ + int err = 0; + /* Testing BIG5-HKSCS. */ + xsetlocale (LC_ALL, "zh_HK.BIG5-HKSCS"); + + /* Run all the special conversions. */ + for (int i = 0; i < (sizeof (tests) / sizeof (struct testdata)); i++) + { + printf ("Running test for %s\n", tests[i].name); + err += check_conversion (tests[i]); + if (err > 0) + printf ("Test %s failed.\n", tests[i].name); + } + + /* Fail if any conversion had an error. */ + if (err > 0) + FAIL_EXIT1 ("One of the conversions failed."); + + return 0; +} + +#include