#33942 [Asn-Csd]: the patch to #33156 crash cygwin except cli

2005-08-02 Thread dmitry
 ID:   33942
 Updated by:   [EMAIL PROTECTED]
 Reported By:  xuefer at 21cn dot com
-Status:   Assigned
+Status:   Closed
 Bug Type: Scripting Engine problem
 Operating System: cygwin only
 PHP Version:  5CVS-2005-08-01 (dev)
 Assigned To:  dmitry
 New Comment:

Nuno's patch is applied to CVS HEAD and PHP_5_0.


Previous Comments:


[2005-08-02 15:10:46] [EMAIL PROTECTED]

I've tested fastcgi and it seems to work fine.
I've also fixed the indentation in the patch.



[2005-08-02 03:06:32] xuefer at 21cn dot com

fastcgi, and maybe other sapi except cli/CGI!, as both cli/cgi exit
immediately after request shutdown

the patch nlopess posted should work (i did it after my report too)

but btw, the indention is not consistent with previous patch



[2005-08-01 23:25:06] [EMAIL PROTECTED]

I've tested the CLI and CGI sapis and eveything works fine. I couldn't
test further, because you didn't specify the sapi you were using.
Anyway, you are right and I missed the zend_unset_timeout() in my
patch.

Here is a patch that should fix your problem:
http://mega.ist.utl.pt/~ncpl/patch_cygwin_php.txt
Please test it.


Thanks,
Nuno



[2005-08-01 12:07:40] [EMAIL PROTECTED]

Dmitry, check your patch please. :)




[2005-08-01 06:47:30] xuefer at 21cn dot com

Description:

in reply to: http://bugs.php.net/bug.php?id=33156 (can't open the bug)

for CYGWIN
in zend_set_timeout()
ITIMER_REAL is set
in zend_unset_timeout()
ITIMER_REAL is NOT unset, but unset ITIMER_PROF instead

it has no problem with make test as the test won't last long. but
with any SAPI other than cli. ITIMER_REAL is triggered after php
request is shutdown, result a crash

Reproduce code:
---
set timeout to 5
./sapi/php -b 1026
request it with an empty php script
wait for 5 second countdown
crash






-- 
Edit this bug report at http://bugs.php.net/?id=33942edit=1


#33942 [Asn-Csd]: the patch to #33156 crash cygwin except cli

2005-08-02 Thread sniper
 ID:   33942
 Updated by:   [EMAIL PROTECTED]
 Reported By:  xuefer at 21cn dot com
-Status:   Assigned
+Status:   Closed
 Bug Type: Scripting Engine problem
 Operating System: cygwin only
 PHP Version:  5CVS-2005-08-01 (dev)
 Assigned To:  dmitry
 New Comment:

Done.



Previous Comments:


[2005-08-02 19:24:52] [EMAIL PROTECTED]

the patch should also be applied to the 4_4 branch.



[2005-08-02 19:02:52] [EMAIL PROTECTED]

Nuno's patch is applied to CVS HEAD and PHP_5_0.



[2005-08-02 15:10:46] [EMAIL PROTECTED]

I've tested fastcgi and it seems to work fine.
I've also fixed the indentation in the patch.



[2005-08-02 03:06:32] xuefer at 21cn dot com

fastcgi, and maybe other sapi except cli/CGI!, as both cli/cgi exit
immediately after request shutdown

the patch nlopess posted should work (i did it after my report too)

but btw, the indention is not consistent with previous patch



[2005-08-01 23:25:06] [EMAIL PROTECTED]

I've tested the CLI and CGI sapis and eveything works fine. I couldn't
test further, because you didn't specify the sapi you were using.
Anyway, you are right and I missed the zend_unset_timeout() in my
patch.

Here is a patch that should fix your problem:
http://mega.ist.utl.pt/~ncpl/patch_cygwin_php.txt
Please test it.


Thanks,
Nuno



The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
http://bugs.php.net/33942

-- 
Edit this bug report at http://bugs.php.net/?id=33942edit=1