bugGNU Wget - Bugs: bug #25298, Memory leak when issue HEAD

 
 

bug #25298: Memory leak when issue HEAD

Submitted by:  Xin Zou <zouxin>
Submitted on:  Sun 11 Jan 2009 09:12:36 AM UTC  
 
Category: Memory LeakSeverity: 3 - Normal
Priority: 7 - HighStatus: Fixed
Privacy: PublicAssigned to: None
Originator Name: Open/Closed: Closed
Release: trunkOperating System: GNU/Linux
Reproducibility: Every TimeFixed Release: None
Planned Release: 1.12Regression: None
Work Required: NonePatch Included: Yes

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Fri 12 Jun 2009 02:21:03 AM UTC, comment #1:

Fixed in 9c501d15e3a0

Micah Cowan <micahcowan>
Sun 11 Jan 2009 09:12:36 AM UTC, original submission:

For example
wget --no-proxy --timestamping http://www.google.com

Wget will issue HEAD first, but when returned from gethttp( in http.c), it didn't free "char *head" allocated by "read_http_response_head (sock)". So wget will leak at least 512bytes memory for each url.

Besides, there are several other potential memory leaks in http.c.
Attached patch tries to fix these bugs.

Xin Zou <zouxin>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #17241:  leakhttp.patch added by zouxin (3KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by micahcowan (Updated the item)
  • -unavailable- added by zouxin (Submitted the item)
  •  

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 7 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 12 Jun 2009 02:21:03 AM UTCmicahcowanStatusReady for Merge=>Fixed
      Open/ClosedOpen=>Closed
    Fri 16 Jan 2009 12:41:13 AM UTCmicahcowanPriority5 - Normal=>7 - High
      StatusNone=>Ready for Merge
      Privacy0(Error - Not Found)=>Public
      Planned ReleaseNone=>1.12
    Sun 11 Jan 2009 09:12:36 AM UTCzouxinAttached File-=>Added leakhttp.patch, #17241

    Back to the top


    Powered by Savane 3.1-cleanup1