Re: PCRE error handling patch

From: 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


« previous php.internals (#22718) next »