Opened 10 years ago
Closed 10 years ago
#3953 closed defect (fixed)
Memory leak in BAG dataset
Reported by: | rprinceley | Owned by: | warmerdam |
---|---|---|---|
Priority: | normal | Milestone: | 1.8.1 |
Component: | GDAL_Raster | Version: | 1.8.0 |
Severity: | normal | Keywords: | hdf5 bag |
Cc: | gaopeng |
Description
There appears to be one or more cases of missing H5?close calls in the BAG implementation. This is causing memory leaks.
Also, we had to change nAttrSize in HDF5AttrIterate() from:
nAttrSize = H5Tget_size( hAttrTypeID );
to:
nAttrSize = H5Aget_storage_size( hAttrID );
Attached is a patch that adds these calls. We are still testing these, please review and include if appropriate.
Attachments (1)
Change History (2)
Changed 10 years ago by
Attachment: | hdf5.patch added |
---|
comment:1 Changed 10 years ago by
Component: | default → GDAL_Raster |
---|---|
Keywords: | hdf5 bag added |
Milestone: | → 1.8.1 |
Resolution: | → fixed |
Status: | new → closed |
Note: See
TracTickets for help on using
tickets.
I have reviewed and tested the patch and it seems good. Applied in trunk (r21650) and 1.8 branch (r21651).
Thanks!