Opened 16 years ago

Closed 16 years ago

#2210 closed defect (fixed)

[PATCH] ECW : fix memory leaks

Reported by: Even Rouault Owned by: warmerdam
Priority: normal Milestone: 1.6.0
Component: GDAL_Raster Version: unspecified
Severity: normal Keywords: ecw
Cc:

Description

The attached patch fixes the following memory leaks found by running Valgrind on ecw.py :

  • oError.GetErrorMessage() returned value should be freed with NCSFree
  • pabyBILBuffer was not freed
  • pszEPSGProj and pszEPSGDatum should be freed with NCSFree
  • GDALECWCompressor::WriteJP2Box was leaking newly created JP2UserBox
  • poIOStream was leaking on J2K_SUBFILE: datasets (bUsingCustomStream was never set to TRUE)

Attachments (1)

gdal_svn_trunk_fix2210.patch​ (8.7 KB ) - added by Even Rouault 16 years ago.

Download all attachments as: .zip

Change History (4)

comment:1 by Even Rouault, 16 years ago

Milestone: → 1.6.0

comment:2 by Even Rouault, 16 years ago

Updated version that also fixes a potential buffer overflow with szProjection and szDatum

by Even Rouault, 16 years ago

Attachment: gdal_svn_trunk_fix2210.patch​ added

comment:3 by Even Rouault, 16 years ago

Resolution: → fixed
Status: new → closed

Patch applied in trunk in r13901

Note: See TracTickets for help on using tickets.