php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Bug #34359 http_fopen_wrapper cleanup
Submitted: 2005-09-03 13:46 UTC Modified: 2005-12-08 03:54 UTC
From: nlopess@php.net Assigned: pollita (profile)
Status: Closed Package: HTTP related
PHP Version: 6CVS-2005-09-03 (CVS) OS: *
Private report: No CVE-ID: None
View Developer Edit
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: nlopess@php.net
New email:
PHP Version: OS:

 

 [2005-09-03 13:46 UTC] 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');"

Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2005-09-19 16:54 UTC] tony2001@php.net 
Sara, could you plz take care of it?
 [2005-12-08 03:54 UTC] iliaa@php.net 
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.
 
PHP Copyright © 2001-2025 The PHP Group
All rights reserved. 		Last updated: Tue Nov 04 05:00:01 2025 UTC