Re: [screen-devel] [bug #60030] Screen segfaults by displaying some UTF-8 character combination

[Axel Beckert]

Hi Michael,

Michael Schroeder wrote:
> On Thu, Feb 11, 2021 at 11:39:09PM +0100, Axel Beckert wrote:
> > The only thing it didn't fix so far is
> > https://savannah.gnu.org/bugs/?31336 aka
> > https://bugs.debian.org/600246 although I really had some hope that
> > this might be fixed as a side-effect of your patch. :-)
> 
> Oh, that happens because the U+3099 combining char is also matched
> in utf8_isdouble(). The code in ansi.c does not expect this. Thus
> this is another fallout from that commit. ;)
> 
> A simple fix (other than removing the entries from the isdouble
> table) is to move the curr->w_mbcs = 0xff setting after the
> combining character handling:

Can confirm! This fixes the above mentioned bug without reopening the
regression (https://bugs.debian.org/677512) which the initially
proposed patch in Debian had.

Thanks once again!

                Regards, Axel
-- 
 ,''`.  |  Axel Beckert <abe@debian.org>, https://people.debian.org/~abe/
: :' :  |  Debian Developer, ftp.ch.debian.org Admin
`. `'   |  4096R: 2517 B724 C5F6 CA99 5329  6E61 2FF9 CD59 6126 16B5
  `-    |  1024D: F067 EA27 26B9 C3FC 1486  202E C09E 1D89 9593 0EDE

signature.asc
Description: PGP signature