php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Bug #79363 \p{L} doesn't work alongside \p{Arabic} in a character class
Submitted: 2020-03-10 13:37 UTC Modified: 2020-09-17 12:10 UTC
From: halaeiv at gmail dot com Assigned: cmb (profile)
Status: Closed Package: PCRE related
PHP Version: 7.4.3 OS: Ubuntu
Private report: No CVE-ID: None
View Add Comment Developer Edit
 [2020-03-10 13:37 UTC] halaeiv at gmail dot com 
Description:
------------
I ran the following code in both PHP 7.2.28 and PHP 7.4.3. The output of PHP 7.4.3 is wrong. Basically, adding a \p{Arabic} after \p{L} seems to causes \p{L} to match uppercases and spaces, instead of letters.

Test script:
---------------
$str = 'lower UPPER';
echo(preg_replace('/[\p{L}\p{Arabic}]/', '0', $str));
echo "\n";
echo(preg_replace('/[^\p{L}\p{Arabic}]/', '0', $str));
echo "\n";


Expected result:
----------------
00000 00000
lower0UPPER


Actual result:
--------------
lower000000
00000 UPPER

Patches

Add a Patch

Pull Requests

Pull requests:

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2020-03-11 08:04 UTC] halaeiv at gmail dot com 
There is a related bug report in pcre:
https://bugs.exim.org/show_bug.cgi?id=2432
The bug's status is new, but it is mentioned that it was fixed in 10.34.
I hope the next PHP version will fix it. There can be some additional tests to check.
Note: the preg works with (*NO_JIT).
 [2020-03-11 08:49 UTC] cmb@php.net
-Status: Open +Status: Not a bug -Assigned To: +Assigned To: cmb
 [2020-03-11 08:49 UTC] cmb@php.net 
Indeed, this is not a bug in PHP, but rather in libpcre.  In
libpcre 10.34, the behavior has been improved, but is still not
quite right (with JIT enabled):

00000000000
lower UPPER
 [2020-06-30 11:06 UTC] halaeiv at gmail dot com 
PCRE 10.35 is released with the bug being fixed. But it is not still used in PHP 7.4 branch. So I believe this issue is now a PHP bug and should be reopened. Any plan to fix?
 [2020-06-30 12:49 UTC] cmb@php.net
-Status: Not a bug +Status: Re-Opened
 [2020-06-30 12:49 UTC] cmb@php.net 
The following pull request has been associated:

Patch Name: Update to PCRE2 10.35
On GitHub:  https://github.com/php/php-src/pull/5783
Patch:      https://github.com/php/php-src/pull/5783.patch
 [2020-06-30 14:22 UTC] cmb@php.net
-Status: Re-Opened +Status: Suspended -Assigned To: cmb +Assigned To:
 [2020-06-30 14:22 UTC] cmb@php.net 
The master branch has just been updated to PCRE2 10.35, so that
will be available as of PHP 8.0.0alpha2.

Updating for PHP 7.4 will be considered at the earliest in several
weeks; for the time being I'm suspending this ticket.
 [2020-09-17 12:10 UTC] cmb@php.net
-Status: Suspended +Status: Closed -Assigned To: +Assigned To: cmb
 [2020-09-17 12:10 UTC] cmb@php.net 
Fixed as <http://git.php.net/?p=php-src.git;a=commit;h=9f2d03952daf5348d7f18c18e82607979e28df12>.
 [2020-09-17 18:42 UTC] requinix@php.net 
Related To: Bug #80118
 [2020-09-17 22:11 UTC] cmb@php.net 
Related To: Bug #80118
 
PHP Copyright © 2001-2021 The PHP Group
All rights reserved. 		Last updated: Thu Jan 28 07:01:23 2021 UTC