Re: PCRE error handling patch
| From: | Andrei Zmievski | Date: | Tue, 11 Apr 2006 19:25:29 +0000 |
| Subject: | Re: PCRE error handling patch | ||
| References: | 1 2 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-22718@lists.php.net to get a copy of this message | ||
It's hard to pick a default that will work for everyone. I can probably see setting recursion limit somewhat lower than backtracking one, though. Maybe 100,000/50,000 or something like that.
-Andrei
On Apr 11, 2006, at 11:51 AM, Nuno Lopes wrote:
I would say that 100.000 recursions is too much. When I made my patch it was segfaulting with such value. 50.000 is a much safer choice (at least on my gentoo box, 8 MB of stack). Nuno ----- Original Message -----I made a patch (against HEAD currently only) that makes it possible to set runtime limits on backtracking and recursion. It also implements preg_last_error() that lets users check whether there was an execution error (such as limit exceeded or bad UTF-8 or whatever). Ilia, do you think it might make it into 5.1.3? http://www.php.net/~andrei/pcre.diff -Andrei -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php