#34359 [Asn->Csd]: http_fopen_wrapper cleanup

From: Date: Thu, 08 Dec 2005 02:54:40 +0000
Subject: #34359 [Asn->Csd]: http_fopen_wrapper cleanup
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-89921@lists.php.net to get a copy of this message 
 ID:               34359
 Updated by:       iliaa@php.net
 Reported By:      nlopess@php.net
-Status:           Assigned
+Status:           Closed
 Bug Type:         HTTP related
 Operating System: *
 PHP Version:      6CVS-2005-09-03 (CVS)
 Assigned To:      pollita
 New Comment:

This bug has been fixed in CVS.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
 
Thank you for the report, and for helping us make PHP better.




Previous Comments:
------------------------------------------------------------------------

[2005-09-19 16:54:40] tony2001@php.net

Sara, could you plz take care of it?

------------------------------------------------------------------------

[2005-09-03 13:46:22] nlopess@php.net

Description:
------------
I've cleaned up the http fopen wrapper, because I got a 2 valgrind
errors on it.

Apart of fixing the bug, I've also removed an unecessary assignement to
http_header_line, because php_stream_gets() already truncates the
string, so there's no need to do it twice (I've tested it by sending a
large (>1024) string in the headers).

Patch: http://mega.ist.utl.pt/~ncpl/php_streams_mem.txt

Reproduce code:
---------------
php -r "file_get_contents('http://www.xpto.com');"



------------------------------------------------------------------------


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

Thread (3 messages)

« previous php.bugs (#89921) next »