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.
And, of course, by "trio", I really mean "pair"...