Opened 12 years ago

Closed 12 years ago

#4372 closed defect (fixed)

[libtiff patch] Fix crash in ChopUpSingleUncompressedStrip on invalid TIF file

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

Description

The new DIFFER_STRILE_LOAD mechanism exposes segfault in libtiff in face on invalid TIF files. Attached a proposed fix

Attachments (4)

byte_4372.tif​ (2.6 KB ) - added by Even Rouault 12 years ago.
ticket_4372.patch​ (858 bytes ) - added by Even Rouault 12 years ago.
ticket_4372_bis.tif​ (937 bytes ) - added by Even Rouault 12 years ago.
ticket_4372_bis.patch​ (3.2 KB ) - added by Even Rouault 12 years ago.

Download all attachments as: .zip

Change History (7)

by Even Rouault, 12 years ago

Attachment: byte_4372.tif​ added

by Even Rouault, 12 years ago

Attachment: ticket_4372.patch​ added

comment:1 by warmerdam, 12 years ago

Component: default → GDAL_Raster
Keywords: gtiff added
Resolution: → fixed
Status: new → closed

Patch applied upstream and pull down in r23496.

comment:2 by Even Rouault, 12 years ago

Resolution: fixed
Status: closed → reopened

Actually, the patch wasn't sufficient. "gdalinfo -checksum ticket_4372_bis.tif" segfaults. Complementary patch included

by Even Rouault, 12 years ago

Attachment: ticket_4372_bis.tif​ added

by Even Rouault, 12 years ago

Attachment: ticket_4372_bis.patch​ added

comment:3 by warmerdam, 12 years ago

Resolution: → fixed
Status: reopened → closed

Patch applied upstream, GDAL resynced (r23499).

Note: See TracTickets for help on using tickets.