Edit report at https://bugs.php.net/bug.php?id=66121&edit=1
ID: 66121
Updated by: cmb@php.net
Reported by: danielklein at airpost dot net
Summary: UTF-8 lookbehinds match bytes instead of characters
-Status: Open
+Status: Duplicate
Type: Bug
Package: PCRE related
PHP Version: 5.5.6
-Assigned To:
+Assigned To: cmb
Block user comment: N
Private report: N
New Comment:
As already been mentioned by nhahtdh at gmail dot com, this
problem has the same cause as bug #53823. Therefore I'm marking
this ticket as duplicate.
Previous Comments:
------------------------------------------------------------------------
[2015-01-13 08:35:31] danielklein at airpost dot net
Although your test doesn't split the code point into bytes, it also does not give the correct
result. There should be a second asterisk after the Sinhala letter.
------------------------------------------------------------------------
[2015-01-08 22:24:51] cmbecker69 at gmx dot de
Rather interestingly, this seems to have worked before PHP
5.2.0: <http://3v4l.org/tL64Y>.
BTW: when talking about Unicode it might be best to avoid the term
"character". In this case it is about code points.
------------------------------------------------------------------------
[2014-12-16 11:01:25] nhahtdh at gmail dot com
In the "non-zero-width match and not reached limit" branch, if the startoffset is taken
from the match result, then the offset is advanced correctly, except for the case of using \C (the
behavior of PHP in such case is different from pcretest, but it should belong in a different bug
report).
In the "zero-width match or limit == 0" branch, except for the case in question, we break
out of the loop.
In the else branch, we also break out of the loop.
So I think everything should be fixed if we change this branch for all related functions.
------------------------------------------------------------------------
[2014-12-16 03:55:37] danielklein at airpost dot net
Same error seems to be on lines 838, 1296 & 1720. No idea if that would fix everything though.
------------------------------------------------------------------------
[2014-12-16 03:28:53] danielklein at airpost dot net
Good find but it looks like that code is just for pcre_replace(). It needs to be fixed for all
affected functions at the same time.
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
https://bugs.php.net/bug.php?id=66121
--
Edit this bug report at https://bugs.php.net/bug.php?id=66121&edit=1