Squashfs: Valid filesystems are flagged as bad by the corrupted fs patch
authorPhillip Lougher <phillip@lougher.demon.co.uk>
Thu, 12 Mar 2009 03:23:48 +0000 (03:23 +0000)
committerPhillip Lougher <phillip@lougher.demon.co.uk>
Thu, 12 Mar 2009 03:23:48 +0000 (03:23 +0000)
commit363911d027d1de1c6df79eb3f487f5476b9619f4
tree6aa93b837f9719b0f51c11330f56288bc5acf51b
parentebdcc81c71937b30e09110c02a1e8a21fa770b6f
Squashfs: Valid filesystems are flagged as bad by the corrupted fs patch

The corrupted filesystem patch added a check against zlib trying to
output too much data in the presence of data corruption.  This check
triggered if zlib_inflate asked to be called again (Z_OK) with
avail_out == 0 and no more output buffers available.  This check proves
to be rather dumb, as it incorrectly catches the case where zlib has
generated all the output, but there are still input bytes to be processed.

This patch does a number of things.  It removes the original check and
replaces it with code to not move to the next output buffer if there
are no more output buffers available, relying on zlib to error if it
wants an extra output buffer in the case of data corruption.  It
also replaces the Z_NO_FLUSH flag with the more correct Z_SYNC_FLUSH
flag, and makes the error messages more understandable to
non-technical users.

Signed-off-by: Phillip Lougher <phillip@lougher.demon.co.uk>
Reported-by: Stefan Lippers-Hollmann <s.L-H@gmx.de>
fs/squashfs/block.c