lwIP - A Lightweight TCP/IP stack - Bugs: bug #31525, dhcp_stop() should be free the...
You are not allowed to post comments on this tracker with your current authentication level.
bug #31525: dhcp_stop() should be free the dhcp memory, which in netif.
Submitter: | hanhui <hanhui03> | ||
Submitted: | Tue 02 Nov 2010 02:36:04 PM UTC | ||
Category: | None | Severity: | 3 - Normal |
Item Group: | Change Request | Status: | Fixed |
Privacy: | Public | Assigned to: | goldsimon |
Open/Closed: | Closed | Planned Release: | None |
lwIP version: | CVS Head |
Sun 21 Nov 2010 01:41:31 PM UTC, comment #4: |
Simon Goldschmidt <goldsimon> |
Tue 02 Nov 2010 03:42:34 PM UTC, comment #3: Then I'd rather vote for an extra function dhcp_cleanup() which does not affect code size if not used (since it will then be thrown away by the linker).
|
Simon Goldschmidt <goldsimon> |
Tue 02 Nov 2010 03:38:14 PM UTC, comment #2: I know that dhcp_start() checks to see if there is an existing client and reuses it if it can. But if run the following codes, it will lead to memory leaks.
|
hanhui <hanhui03> |
Tue 02 Nov 2010 03:16:33 PM UTC, comment #1: It looks like this is deliberate: dhcp_start() checks to see if there is an existing client and reuses it if it can. |
Kieran Mansley <kieranm> |
Tue 02 Nov 2010 02:36:04 PM UTC, original submission:
|
hanhui <hanhui03> |
No files currently attached
Depends on the following items: None found
Items that depend on this one: None found
There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.
Follow 3 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2010-11-21 | goldsimon | Status | None | Fixed | |
Assigned to | None | goldsimon | |||
Open/Closed | Open | Closed |
Powered by Savane 3.13-4448.
Corresponding source code
Fixed by adding the function dhcp_cleanup(), which you would need to call before free(netif).
Thanks for reporting.