Opened 14 years ago
Closed 13 years ago
#3193 closed defect (fixed)
Fix memory leaks in loadMapInternal
| Reported by: | bretlambert | Owned by: | sdlime |
|---|---|---|---|
| Priority: | normal | Milestone: | 6.0 release |
| Component: | MapServer C Library | Version: | svn-trunk (development) |
| Severity: | normal | Keywords: | |
| Cc: |
Description
While looking through #1886 (which appears to be no longer valid, and should be revisited), I came across a trio of error-case memleaks; the following patch is untested, but *should* resolve the issue; consider it for educational purposes only, and make sure that the ultimate bit that gets committed actually compiles/runs!
The diff also cleans the code up a bit, removing a pair of pointless changes of pointer values to NULL, and pulling the break for the case inside of the {}, and lining it up with the rest of the code, so that the full statement is made more legible.
Attachments (1)
Change History (3)
by , 14 years ago
| Attachment: | memleaks.diff added |
|---|
comment:1 by , 14 years ago
comment:2 by , 13 years ago
| Milestone: | → 6.0 release |
|---|---|
| Resolution: | → fixed |
| Status: | new → closed |
fixed in r11602.
Note:
See TracTickets
for help on using tickets.
And, of course, by "trio", I really mean "pair"...