Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:23828
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
To: Steph Fox <steph@zend.com>
Cc: 'internals' <internals@lists.php.net>,
	'Antony Dovgal' <antony@zend.com>, Dmitry Stogov <dmitry@zend.com>,
	'Xuefer' <xuefer@gmail.com>, 'Andi Gutmans' <andi@zend.com>
Message-ID: <11490874854760000@9866357972520000.9866341568840000>
MIME-Version: 1.0
Reply-To: "Frank M. Kromann" <frank@kromann.info>
Date: Wed, 31 May 2006 07:58:05 -0700
Content-Type: text/plain;
	charset="us-ascii"
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] tsrm_shutdown() and the CLI SAPI
From: frank@kromann.info ("Frank M. Kromann")

tsrm_shutdown() is already comented out in the CGI version, most likely as
a fix to the same kind of problem there. Perhaps enabling that again will
show the same kind of problems?

- Frank

> At the moment I'm seeing it with Tidy under 5_2.  Just about to get Tony
to 
> check.
> 
> ----- Original Message ----- 
> From: "Dmitry Stogov" <dmitry@zend.com>
> To: "'Steph Fox'" <steph@zend.com>; "'Xuefer'" <xuefer@gmail.com>;
"'Andi 
> Gutmans'" <andi@zend.com>
> Cc: "'internals'" <internals@lists.php.net>; "'Antony Dovgal'" 
> <antony@zend.com>
> Sent: Wednesday, May 31, 2006 2:55 PM
> Subject: RE: [PHP-DEV] tsrm_shutdown() and the CLI SAPI
> 
> 
> > Steph,
> >
> > Commenting tsrm_shutdown() or FreeLibrary() is just hidding real
bugs.
> > Of course this is a solution, but not excellent.
> >
> > Right now I haven't enough time even for ZE and PHP themself, so I
don't
> > like spent it looking into php-gtk on windows.
> > (I don't know GTK at all).
> >
> > Do you know any other extension from PHP distribution that cause the
same
> > crash?
> > Could you provide instruction how to reproduce it?
> >
> > Thanks. Dmitry.
> >
> >> -----Original Message-----
> >> From: Steph Fox [mailto:steph@zend.com]
> >> Sent: Wednesday, May 31, 2006 3:28 PM
> >> To: Dmitry Stogov; 'Xuefer'; 'Andi Gutmans'
> >> Cc: 'internals'; Antony Dovgal
> >> Subject: Re: [PHP-DEV] tsrm_shutdown() and the CLI SAPI
> >>
> >>
> >> > Oh well, now I see the difference.
> >> > With my patch PHP starts call FreeLibrary() for dll extension. I
am
> >> > wonder why it wasn't called before. :)
> >>
> >> Pure luck, by the look of it :)
> >>
> >> > Probably some other bug that is the reason of the crash was
> >> masked by
> >> > omitting FreeLibrary(). I know Tony fixed some kind of such bugs
in
> >> > ext/tidy and ext/sybase.
> >>
> >> Yes, he gave those extensions the ts_free_id() workaround I
> >> mentioned a
> >> looooooooooong time ago when I first started looking into this.
> >>
> >> > Probably php-gtk needs similar fix.
> >>
> >> That workaround didn't work for PHP-GTK. I tried that first.
> >> That's why I
> >> need you to fix this bug rather than me just cover it up. The
> >> only thing
> >> that works for PHP-GTK is to either avoid tsrm_shutdown()
> >> completely (via
> >> Zeev's suggestion of commenting out the call in php_cli.c),
> >> or to fix the
> >> bug that makes reaching it a problem.
> >>
> >> > All other systems except win32 already used common config
> >> file, and I
> >> > don't
> >> > see any reason for this exceptions.
> >>
> >> I don't know or care what other systems do... how's that even
> >> relevant?
> >>
> >> > Especially because it was the reason of bug, and masked other bugs
> >> > that
> >> > may
> >> > be reproduced on linux with ZTS.
> >>
> >> Sorry, I didn't parse that... can you rephrase?
> >> >
> >> > Thanks. Dmitry.
> >>
> >> Cheers,
> >> - Steph
> >>
> >> -- 
> >> PHP Internals - PHP Runtime Development Mailing List
> >> To unsubscribe, visit: http://www.php.net/unsub.php
> >>
> >>
> >>
> >
> >
> >
> > __________ NOD32 1.1380 (20060125) Information __________
> >
> > This message was checked by NOD32 antivirus system.
> > http://www.eset.com
> >
> > 
> 
> -- 
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>