[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/6] canonicalize: fix EOVERFLOW bug
From: |
Bruno Haible |
Subject: |
Re: [PATCH 2/6] canonicalize: fix EOVERFLOW bug |
Date: |
Mon, 14 Dec 2020 19:05:39 +0100 |
User-agent: |
KMail/5.1.3 (Linux/4.4.0-193-generic; KDE/5.18.0; x86_64; ; ) |
Paul Eggert wrote:
> > This patch introduced a unit test regression on AIX 7.2....
> > ../../gltests/test-canonicalize.c:112: assertion 'result1 == NULL' failed
> > FAIL test-canonicalize (exit status: 134)
> ...
> I installed the attached patches to work around these issues, along with
> other stuff I noticed while testing this under AIX 7.2 (gcc119, which is
> power8-aix). The relevant unit tests succeed for me now.
Thanks; the unit tests succeed for me too, now.
But canonicalize-lgpl.c had similar changes, and it has a similar failure
on AIX 7.2, still today:
../../gltests/test-canonicalize-lgpl.c:100: assertion 'result == NULL' failed
FAIL test-canonicalize-lgpl (exit status: 134)
Probably your changes to canonicalize.c could be applied to canonicalize-lgpl.c.
But Adhemerval Zanella has come up with another set of patches, from the glibc
point of view:
https://lists.gnu.org/archive/html/bug-gnulib/2020-12/msg00110.html
Bruno
[PATCH 6/6] canonicalize: refactor can_mode flag, Paul Eggert, 2020/12/02
[PATCH 5/6] canonicalize: prefer signed integer types, Paul Eggert, 2020/12/02
Re: [PATCH 1/6] canonicalize-lgpl: fix EOVERFLOW bug, Adhemerval Zanella, 2020/12/11