Skip to content

Commit 875fb3f

Browse files
ploughergregkh
authored andcommitted
Squashfs: reject negative file sizes in squashfs_read_inode()
[ Upstream commit 9f1c14c ] Syskaller reports a "WARNING in ovl_copy_up_file" in overlayfs. This warning is ultimately caused because the underlying Squashfs file system returns a file with a negative file size. This commit checks for a negative file size and returns EINVAL. [phillip@squashfs.org.uk: only need to check 64 bit quantity] Link: https://lkml.kernel.org/r/20250926222305.110103-1-phillip@squashfs.org.uk Link: https://lkml.kernel.org/r/20250926215935.107233-1-phillip@squashfs.org.uk Fixes: 6545b24 ("Squashfs: inode operations") Signed-off-by: Phillip Lougher <phillip@squashfs.org.uk> Reported-by: syzbot+f754e01116421e9754b9@syzkaller.appspotmail.com Closes: https://lore.kernel.org/all/68d580e5.a00a0220.303701.0019.GAE@google.com/ Cc: Amir Goldstein <amir73il@gmail.com> Cc: <stable@vger.kernel.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Sasha Levin <sashal@kernel.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
1 parent 234f6e1 commit 875fb3f

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

fs/squashfs/inode.c

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -193,6 +193,10 @@ int squashfs_read_inode(struct inode *inode, long long ino)
193193
goto failed_read;
194194

195195
inode->i_size = le64_to_cpu(sqsh_ino->file_size);
196+
if (inode->i_size < 0) {
197+
err = -EINVAL;
198+
goto failed_read;
199+
}
196200
frag = le32_to_cpu(sqsh_ino->fragment);
197201
if (frag != SQUASHFS_INVALID_FRAG) {
198202
/*

0 commit comments

Comments
 (0)