Re: cvs: php-src /build order_by_dep.awk /ext/date config.m4 config0.m4 php_date.c ZendEngine2 zend.c

From:	Jani Taskinen	Date:	Wed, 13 May 2009 18:26:24 +0000
Subject:	Re: cvs: php-src /build order_by_dep.awk /ext/date config.m4 config0.m4 php_date.c ZendEngine2 zend.c
References:	1 2 3 4 5	Groups:	php.cvs
Request:	Send a blank email to php-cvs+get-57781@lists.php.net to get a copy of this message

Dmitry Stogov kirjoitti:

Derick Rethans wrote:
On Wed, 13 May 2009, Stanislav Malyshev wrote:
2) Uninitialized DATEG(tzcache. It's initialized only during RINIT() so any error messages befor this may crash PHP.
This isn't the first time we see extension being used before initialization. Maybe we need some infrastructure to prevent that.
In this case, I don't see why I can't init the tzcache in MINIT() though. But would that solve things?
Not in general, but it'll fix error logging for all extensions initialized after ext/date.

I had a patch first which added the init (also) in GINIT along with that RINIT one but that was useless without fixing the core issues first. And then I lost it. :D In case of ext/date: It should be moved to core. --Jani