squashfs metadata 2: electric boogaloo
authorLinus Torvalds <torvalds@linux-foundation.org>
Wed, 1 Aug 2018 17:38:43 +0000 (10:38 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Wed, 1 Aug 2018 17:38:43 +0000 (10:38 -0700)
commitcdbb65c4c7ead680ebe54f4f0d486e2847a500ea
tree3b564081bbd292c32cc7d0cfad6be6961ec9e8e0
parent44960f2a7b63e224b1091b3e1d6f60e0cdf4be0c
squashfs metadata 2: electric boogaloo

Anatoly continues to find issues with fuzzed squashfs images.

This time, corrupt, missing, or undersized data for the page filling
wasn't checked for, because the squashfs_{copy,read}_cache() functions
did the squashfs_copy_data() call without checking the resulting data
size.

Which could result in the page cache pages being incompletely filled in,
and no error indication to the user space reading garbage data.

So make a helper function for the "fill in pages" case, because the
exact same incomplete sequence existed in two places.

[ I should have made a squashfs branch for these things, but I didn't
  intend to start doing them in the first place.

  My historical connection through cramfs is why I got into looking at
  these issues at all, and every time I (continue to) think it's a
  one-off.

  Because _this_ time is always the last time. Right?   - Linus ]

Reported-by: Anatoly Trosinenko <anatoly.trosinenko@gmail.com>
Tested-by: Willy Tarreau <w@1wt.eu>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Cc: Phillip Lougher <phillip@squashfs.org.uk>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
fs/squashfs/file.c
fs/squashfs/file_direct.c
fs/squashfs/squashfs.h