summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJan Kara <jack@suse.cz>2021-10-18 12:37:41 +0200
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2021-11-12 13:17:07 +0100
commit156ce5bb6cc43a80a743810199defb1dc3f55b7f (patch)
tree1e92c7a01b2eca7b2e66530efa44265276b241a3
parentcbefdf724282e6a948885f379dc92ab841c2fee0 (diff)
downloadlinux-stable-156ce5bb6cc43a80a743810199defb1dc3f55b7f.tar.gz
linux-stable-156ce5bb6cc43a80a743810199defb1dc3f55b7f.tar.bz2
linux-stable-156ce5bb6cc43a80a743810199defb1dc3f55b7f.zip
isofs: Fix out of bound access for corrupted isofs image
commit e96a1866b40570b5950cda8602c2819189c62a48 upstream. When isofs image is suitably corrupted isofs_read_inode() can read data beyond the end of buffer. Sanity-check the directory entry length before using it. Reported-and-tested-by: syzbot+6fc7fb214625d82af7d1@syzkaller.appspotmail.com CC: stable@vger.kernel.org Signed-off-by: Jan Kara <jack@suse.cz> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r--fs/isofs/inode.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/isofs/inode.c b/fs/isofs/inode.c
index 350f67fb5b9c..41ece64f1a34 100644
--- a/fs/isofs/inode.c
+++ b/fs/isofs/inode.c
@@ -1268,6 +1268,8 @@ static int isofs_read_inode(struct inode *inode, int relocated)
de = (struct iso_directory_record *) (bh->b_data + offset);
de_len = *(unsigned char *) de;
+ if (de_len < sizeof(struct iso_directory_record))
+ goto fail;
if (offset + de_len > bufsize) {
int frag1 = bufsize - offset;