Squashfs: fix i_blocks calculation with extended regular files
commit3d4a1c80c4eb97187b3a61b3bfa8c804327f7a45
authorPhillip Lougher <phillip@squashfs.org.uk>
Tue, 3 Jan 2012 02:58:13 +0000 (3 02:58 +0000)
committerPhillip Lougher <phillip@squashfs.org.uk>
Tue, 3 Jan 2012 04:00:43 +0000 (3 04:00 +0000)
treef845494ae3eaa257256095e009b55885517de0c5
parentcc37f75a9ffbbfcb1c3297534f293c8284e3c5a6
Squashfs: fix i_blocks calculation with extended regular files

The le64_to_cpu() forces the calculation to be unsigned, with
the effect that it can underflow leading to an incorrect large
value.

This bug only triggers in rare(ish) circumstances, an empty file
encoded as an extended regular file or a completely sparse file.
Normally empty files are encoded as a regular file rather than as
an extended regular file (and the regular file i_blocks calculation
doesn't have this bug).  To save space regular file inodes are
optimised to encode the most commonly occurring files.  Less
common regular files are encoded using extended regular file inodes
which contain extra information.

Empty files with nlinks greater than 1, and or empty files
with extended attributes are encoded using extended regular file
inodes and they will hit this bug.

Signed-off-by: Phillip Lougher <phillip@squashfs.org.uk>
fs/squashfs/inode.c