Opened 6 weeks ago

Closed 6 weeks ago

Last modified 6 weeks ago

#7040 closed defect (fixed)

Issue with GDALProxyRasterBand::FlushCache() and Fill() / IWriteBlock() on ISIS3 with GeoTIFF bands

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

Description

Different but somehow related issues :

  • the FlushCache?() implementation on GDALProxyRasterBand() directly delegates to the underlying raster band. It should also flush the blocks that might have been instanciated on the proxy band itself
  • inverted logic in ISIS3WrapperRasterBand::IWriteBlock() regarding m_bGeoTIFFInitDone which could potentially result in a missing initialization of the geotiff file and thus out-of-order blocks
  • ISIS3WrapperRasterBand::Fill() should be implemented, otherwise the init done in ISIS3WrapperRasterBand::InitFile?() will override the user value passed to ISIS3WrapperRasterBand::Fill()

Change History (4)

comment:1 Changed 6 weeks ago by Even Rouault

In 40067:

Fix issue with GDALProxyRasterBand::FlushCache?() not flushing everything required (refs #7040)

comment:2 Changed 6 weeks ago by Even Rouault

Resolution: fixed
Status: assignedclosed

In 40068:

ISIS3: fix logic to initialize underlying GeoTIFF file in IWriteBlock(), and implement Fill() (fixes #7040)

comment:3 Changed 6 weeks ago by Even Rouault

In 40069:

Fix issue with GDALProxyRasterBand::FlushCache?() not flushing everything required (refs #7040)

comment:4 Changed 6 weeks ago by Even Rouault

In 40070:

ISIS3: fix logic to initialize underlying GeoTIFF file in IWriteBlock(), and implement Fill() (fixes #7040)

Note: See TracTickets for help on using tickets.