Re: another fix for max_input_vars.
| From: | Laruence | Date: | Wed, 04 Jan 2012 07:13:24 +0000 |
| Subject: | Re: another fix for max_input_vars. | ||
| References: | 1 2 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-57176@lists.php.net to get a copy of this message | ||
Hi:
sorry, I didn't put that clearly, I mean, in trigger E_ERROR in
php_register_variable will cause php-cgi process exit.
and dmitry has fixed that by change the E_ERROR to E_WARNING,
my patch is doing the same thing but in another way
thanks
On Wed, Jan 4, 2012 at 3:08 PM, Stas Malyshev <smalyshev@sugarcrm.com> wrote:
> Hi!
>
>
>> it seems you have fix the issue that error in register_variable
>> will cause php process exit.
>>
>> here is a fix I made before:
>> http://pastebin.com/7BLAVaWr , I
>> think maybe this is a lighter fix.
>>
>> could you review this? if you think this is okey, I will commit it.
>
>
> Could you please explain what exactly causes process exit and how it can be
> reproduced? I.e., what is the problem in the patch that is committed now?
> --
> Stanislav Malyshev, Software Architect
> SugarCRM: http://www.sugarcrm.com/
> (408)454-6900 ext. 227
--
Laruence Xinchen Hui
http://www.laruence.com/