|
 |
f239de |
From b7466a55e89aa6d6a649734f2b1b24a03390bcef Mon Sep 17 00:00:00 2001
|
|
|
f239de |
From: Theodore Ts'o <tytso@mit.edu>
|
|
|
f239de |
Date: Wed, 26 Aug 2020 16:29:29 -0400
|
|
|
f239de |
Subject: [PATCH 08/46] libext2fs: fix potential buffer overrun in
|
|
|
f239de |
__get_dirent_tail()
|
|
|
f239de |
Content-Type: text/plain
|
|
|
f239de |
|
|
|
f239de |
If the file system is corrupted, there is a potential of a read-only
|
|
|
f239de |
buffer overrun. Fortunately, we don't actually use the result of that
|
|
|
f239de |
pointer dereference, and the overrun is at most 64k.
|
|
|
f239de |
|
|
|
f239de |
Google-Bug-Id: #158564737
|
|
|
f239de |
Fixes: eb88b751745b ("libext2fs: make ext2fs_dirent_has_tail() more strict")
|
|
|
f239de |
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
|
|
|
f239de |
Signed-off-by: Lukas Czerner <lczerner@redhat.com>
|
|
|
f239de |
---
|
|
|
f239de |
lib/ext2fs/csum.c | 3 +--
|
|
|
f239de |
1 file changed, 1 insertion(+), 2 deletions(-)
|
|
|
f239de |
|
|
|
f239de |
diff --git a/lib/ext2fs/csum.c b/lib/ext2fs/csum.c
|
|
|
f239de |
index a7172580..2151003b 100644
|
|
|
f239de |
--- a/lib/ext2fs/csum.c
|
|
|
f239de |
+++ b/lib/ext2fs/csum.c
|
|
|
f239de |
@@ -266,12 +266,11 @@ static errcode_t __get_dirent_tail(ext2_filsys fs,
|
|
|
f239de |
d = dirent;
|
|
|
f239de |
top = EXT2_DIRENT_TAIL(dirent, fs->blocksize);
|
|
|
f239de |
|
|
|
f239de |
- rec_len = translate(d->rec_len);
|
|
|
f239de |
while ((void *) d < top) {
|
|
|
f239de |
+ rec_len = translate(d->rec_len);
|
|
|
f239de |
if ((rec_len < 8) || (rec_len & 0x03))
|
|
|
f239de |
return EXT2_ET_DIR_CORRUPTED;
|
|
|
f239de |
d = (struct ext2_dir_entry *)(((char *)d) + rec_len);
|
|
|
f239de |
- rec_len = translate(d->rec_len);
|
|
|
f239de |
}
|
|
|
f239de |
|
|
|
f239de |
if ((void *)d > ((void *)dirent + fs->blocksize))
|
|
|
f239de |
--
|
|
|
f239de |
2.35.1
|
|
|
f239de |
|