Bug #70110 [Com]: preg_match problem

From:	donatj at gmail dot com	Date:	Wed, 31 May 2017 13:16:51 +0000
Subject:	Bug #70110 [Com]: preg_match problem
References:	1	Groups:	php.bugs
Request:	Send a blank email to php-bugs+get-209320@lists.php.net to get a copy of this message

Edit report at https://bugs.php.net/bug.php?id=70110&edit=1

 ID:                 70110
 Comment by:         donatj at gmail dot com
 Reported by:        bugzilla77 at gmail dot com
 Summary:            preg_match problem
 Status:             Open
 Type:               Bug
 Package:            PCRE related
 Operating System:   *
 PHP Version:        7.0.0beta1
 Block user comment: N
 Private report:     N

 New Comment:

As it stands, retuning false is a very bad behavior. We have thousands of regexes in hundreds of
code bases that just worked but fail at random in PHP7. Having to now check every single one for JIT
stack over flow every single time is a MAJOR hassle. I would have strongly preferred an exception or
a warning that would at least automatically shown up in our logs, but right now our existing code
bases fail silently which is the worst of all worlds.


Previous Comments:
------------------------------------------------------------------------
[2017-05-03 11:31:45] mk at bio dot logis dot de

Just want to point out that Drupal's and Symfony's YAML parsing are affected by this bug
as well:
https://www.drupal.org/node/2792877

------------------------------------------------------------------------
[2017-05-03 11:23:14] cmb@php.net

Thanks for the reminder – I don't have time for this, though.

------------------------------------------------------------------------
[2017-05-03 10:01:02] hristochonov at gmail dot com

cmb@php.net has commented on [2015-08-25 23:56 UTC]:
"Further improvements have to be postponed until PHP 7.1 at least,
so I'm suspending the ticket for now."


PHP 7.1.0 has been released on 01 Dec 2016. It will be great if it would be possible to bring the
issue forward now. We are experiencing this bug pretty often and have to always manually turn off
PCRE-JIT in code for known regular expressions going to crash.

------------------------------------------------------------------------
[2016-07-20 19:37:37] cmb@php.net

Related To: Bug #72632

------------------------------------------------------------------------
[2016-07-04 14:12:15] cmb@php.net

Related To: Bug #70699

------------------------------------------------------------------------


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=70110


--
Edit this bug report at https://bugs.php.net/bug.php?id=70110&edit=1


   

Thread (13 messages)

• bugzilla77 at gmail dot comWed, 22 Jul 2015 09:04:54 +0000
  • cmb@php.netWed, 22 Jul 2015 12:05:15 +0000Bug #70110 [Opn->Ver]: preg_match problem
  • cmb@php.netWed, 22 Jul 2015 14:03:30 +0000Bug #70110 [Ver->Sus]: preg_match problem
  • cmb@php.netThu, 23 Jul 2015 10:16:11 +0000Bug #70110 [Sus->Ana]: preg_match problem
  • me at kelunik dot comThu, 13 Aug 2015 11:26:18 +0000Bug #70110 [Com]: preg_match problem
  • cmb@php.netTue, 25 Aug 2015 23:56:24 +0000Bug #70110 [Ana->Sus]: preg_match problem
  • hristochonov at gmail dot comWed, 03 May 2017 10:01:08 +0000Bug #70110 [Com]: preg_match problem
  • cmb@php.netWed, 03 May 2017 11:23:16 +0000Bug #70110 [Sus->Opn]: preg_match problem
  • mk at bio dot logis dot deWed, 03 May 2017 11:31:49 +0000Bug #70110 [Com]: preg_match problem
  • donatj at gmail dot comWed, 31 May 2017 13:16:51 +0000Bug #70110 [Com]: preg_match problem
  • alec@php.netMon, 04 Sep 2017 10:12:14 +0000Bug #70110 [Com]: preg_match problem
  • stanislav dot khromov at gmail dot comThu, 09 Nov 2017 11:31:46 +0000Bug #70110 [Com]: preg_match problem
  • sjon@php.netTue, 16 Jun 2020 13:17:20 +0000Bug #70110 [Opn->Csd]: preg_match problem