Opened 15 years ago
Closed 15 years ago
#227 closed defect (fixed)
Deleting 'void*' is undefined in IntervalRTreeLeafNode class.
Reported by: | mloskot | Owned by: | strk |
---|---|---|---|
Priority: | major | Milestone: | 3.2.0 |
Component: | Core | Version: | main |
Severity: | Significant | Keywords: | void |
Cc: |
Description
According to section 5.3.5/3 of C++ Standard, there is undefined behavior in IntervalRTreeLeafNode destructor (line 50) because item (line 39) is a pointer of type void.
Change History (5)
comment:1 by , 15 years ago
comment:2 by , 15 years ago
Milestone: | → 3.2.0 |
---|
comment:3 by , 15 years ago
Owner: | changed from | to
---|
comment:4 by , 15 years ago
I checked. That class is unused by core GEOS and headers are not installed. It's a teoreticall bug, but I'd rather drop the whole class from GEOS before it's too late :)
comment:5 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Alright, I reworked ownership of those objects and tracked all users (I did find one: algorithm::locate::IndexedPointInAreaLocator) to properly avoid memory leaks.
Note:
See TracTickets
for help on using tickets.
A little irc discussion with Sandro about this issue from Jan 30, 2009: