AWARE SYSTEMS
TIFF and LibTiff Mail List Archive

Thread

2018.11.10 19:39 "[Tiff] Libtiff 4.0.10 is now available", by Bob Friesenhahn
2018.11.10 21:24 "Re: [Tiff] Libtiff 4.0.10 is now available", by
2018.11.15 23:37 "[Tiff] vcpkg port for 4.0.10", by Roger Leigh
2018.11.19 15:21 "Re: [Tiff] vcpkg port for 4.0.10", by Edward Lam
2018.11.21 10:18 "Re: [Tiff] vcpkg port for 4.0.10", by Roger Leigh
2018.12.08 20:43 "[Tiff] Zlib inflate recovery action required?", by Bob Friesenhahn
2018.12.08 21:14 "Re: [Tiff] Zlib inflate recovery action required?", by Bob Friesenhahn
2018.12.08 21:22 "Re: [Tiff] Zlib inflate recovery action required?", by Olivier Paquet
2018.12.08 21:29 "Re: [Tiff] Zlib inflate recovery action required?", by Even Rouault
2018.12.08 21:42 "Re: [Tiff] Zlib inflate recovery action required?", by Bob Friesenhahn
2018.12.08 22:03 "Re: [Tiff] Zlib inflate recovery action required?", by Toby Thain

2018.12.08 22:03 "Re: [Tiff] Zlib inflate recovery action required?", by Toby Thain

On 2018-12-08 3:43 PM, Bob Friesenhahn wrote:

I have a TIFF pixar log compressed file for which zlib is reporting Z_DATA_ERROR from inflate() in tif_pixarlog.c line 816.

Is it the consensus of opinion that an attempt to recover from

> Z_DATA_ERROR is valuable?  The alternative is to quit right away and

hopefully not plow into reading unitialized memory (considered to be a

> security problem rather than a virtue).  Should libtiff try really hard

to read apparently corrupted files or can we safely assume that modern computers do not produce useful corrupted files?

Thanks in advance for any informed opinions.

Speaking only as the author of a PSD parser/scavenger (extensively fuzz tested) I think libtiff should make a reasonable effort to read the file, but the activity of trying to recover or scavenge data doesn't really belong in a general purpose library, especially if it increases attack surface or complexity. Most important imho is to ensure the error reporting chain is as reliable and comprehensive as possible.

--Toby

>
> Bob