2018.12.08 21:22 "Re: [Tiff] Zlib inflate recovery action required?", by Olivier Paquet

Le sam. 8 déc. 2018 à 16:14, Bob Friesenhahn <bfriesen@simple.dallas.tx.us> a écrit:

There is identical code in tif_zip.c at line 178. I have another file for which use of uninitialized data is reported in zlib and suspect that it is due to the same problem.

I did verify that if I comment out

     if (inflateSync(&sp->stream) != Z_OK)

and simply return an error (0) that both of my problatic test cases avoid subsequent use of initialized memory.

Is this solution ok?

Seems acceptable to me. The era of bad sectors on floppies is long gone. Besides, such recovery is unlikely to work and even if it does, likely to leave the remainder of a strip garbled. Much better to drop the whole strip/tile IMO.

Olivier