[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lwip-devel] [bug #25705] Memory Leak in HTTPD.C
From: |
Dave Wilson |
Subject: |
[lwip-devel] [bug #25705] Memory Leak in HTTPD.C |
Date: |
Wed, 25 Feb 2009 17:24:57 +0000 |
User-agent: |
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.0.6) Gecko/2009011913 Firefox/3.0.6 |
Follow-up Comment #2, bug #25705 (project lwip):
Sorry - I forgot to mention that I agree that the original code would appear
to leak the hs structure if it's not freed when you abort after multiple
retries. You should also clean up the hs structure before freeing it, though -
there's a file handle and buffer pointer in there, for example. As far as I
can tell, the correct thing to do would be:
if(hs->handle) {
fs_close(hs->handle);
}
if(hs->buf) {
mem_free(hs->buf);
}
mem_free(hs);
_______________________________________________________
Reply to this item at:
<http://savannah.nongnu.org/bugs/?25705>
_______________________________________________
Message sent via/by Savannah
http://savannah.nongnu.org/