Re: 5.2.4RC1 Released
| From: | Antony Dovgal | Date: | Fri, 03 Aug 2007 13:40:19 +0000 |
| Subject: | Re: 5.2.4RC1 Released | ||
| References: | 1 2 3 4 5 6 7 8 9 10 11 12 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-31422@lists.php.net to get a copy of this message | ||
On 03.08.2007 17:29, Uwe Schindler wrote:
I reopened the original bug reported that lead to your change. At the moment I am trying to fix that. I moved your code a few lines down in zend_alter_ini_entry but until now with no success.No, that won't work I guess. At the moment I can only think of a special hashtable storing all the values that were set during the INI_SYSTEM stage, so that users could not override them in their scripts.
I suppose there is something special with error reporting that corrupts it.That special thing is in your config file =)
It seems that it does not like it to be changed to ZEND_INI_SYSTEM because the @operator tries to change the value (e.g. in zend_vm_execute.h), which fails silently:This's a special case and it's really great you noticed it in RC.. We need a workaround for this special case, as if we make all INI directives set using php_admin_value non-changeable, we break the @ thing. So we either need to change the @ not to use zend_alter_ini_entry, or make an exception in that function, which I believe would be a hack. -- Wbr, Antony Dovgal