RE: [PHP-DEV] PCRE jit bug with UTF-8 and lookbehind assertion
| From: | Anatol Belski | Date: | Thu, 25 Feb 2016 20:29:14 +0000 |
| Subject: | RE: [PHP-DEV] PCRE jit bug with UTF-8 and lookbehind assertion | ||
| References: | 1 2 3 4 5 6 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-91422@lists.php.net to get a copy of this message | ||
Hi Ángel,
> -----Original Message-----
> From: Ángel González [mailto:keisial@gmail.com]
> Sent: Thursday, February 25, 2016 9:12 PM
> To: Anatol Belski <anatol.php@belski.net>
> Cc: 'Christian Schneider' <cschneid@cschneid.com>; 'PHP internals'
> <internals@lists.php.net>
> Subject: Re: [PHP-DEV] PCRE jit bug with UTF-8 and lookbehind assertion
>
> On 21/02/16 11:42, Anatol Belski wrote:
> > Were you putting the snippets into a file or testing on the console? I
> > had an issue while testing this on the console, that some chars was
> > partially swallowed by terminal (which was a utf-8 terminal). When
> > putting into a file, the output is same for both - "x°z". Please see
> > also the continued discussion in the original ticket
> > https://bugs.exim.org/show_bug.cgi?id=1189 . The offsets
> > delivered by
> > PCRE also seem to be correct, and valgrind doesn't find anything. It
> > were great if you could confirm these insights. Thanks Anatol
> I was testing on a console and piping into hexdump. Using files showed -as
> expected- the same failure.
> The pcre-8.39 trunk (r1635), once you enable all the required options, does fix
> both the C test case and the php one (loading the new dynamic library instead of
> the system one).
>
> So yes, seems fixed in libpcre :)
Thanks for the further check. Yeah, 8.39 looks to be fixing the issue, just waiting for it to be
released to upgrade in 7.0.
Regards
Anatol