Bug #13920 Updated: close () doesn't check input properly ?

From: Date: Tue, 13 Nov 2001 00:07:35 +0000
Subject: Bug #13920 Updated: close () doesn't check input properly ?
References: 1  Groups: php.dev 
Request: Send a blank email to php-dev+get-70297@lists.php.net to get a copy of this message
ID: 13920 Updated by: sniper Reported By: leen@wirehub.nl Old Status: Open Status: Closed Bug Type: Sockets related Operating System: Linux2.4 Debian/Testing G3 PPC PHP Version: 4.0.5 New Comment: If it works in CVS, it's gonna work in next release too. So closed. --Jani Previous Comments: ------------------------------------------------------------------------ [2001-11-12 09:19:45] leen@wirehub.nl OK, took me some time to find time, but it works for me (although the other was a distributionbuild, this one is mine, so what does it really tell us ?). ------------------------------------------------------------------------ [2001-11-04 19:19:12] sniper@php.net Please try the latest CVS. e.g. a snapshot from http://snaps.php.net/ And note that all the functions have been renamed. --Jani ------------------------------------------------------------------------ [2001-11-04 10:23:49] leen@wirehub.nl Note: Ofcourse I know this function is experimental, I'm just trying to help the bugsquiding. I'm running 4.0.5 and checked the changes in 4.0.6 and 4.0.5 of sockets.c to see if the close function changed, it didn't (as far as I see it), so I think this also applies to 4.0.6 I don't know if this is PPC specific, although I doubt it. here is the code, that crashes the php module (apache still works, but php doesn't output anything and headers don't get send): <? close ($fd); ?> if only crashes after multiple times (and with fast enough subsequent tries ?), this is what I used for testing: #! /bin/sh while true do lynx -source http://127.0.0.1/ done hope this was helpfull ------------------------------------------------------------------------ Edit this bug report at http://bugs.php.net/?id=13920&edit=1

« previous php.dev (#70297) next »