mirror of
https://github.com/BobTheBlinker/android_kernel_motorola_sm6375.git
synced 2026-10-30 11:41:38 -04:00
hfsplus: Fix potential buffer overflows
Commit ec81aecb29 ("hfs: fix a potential buffer overflow") fixed a few
potential buffer overflows in the hfs filesystem. But as Timo Warns
pointed out, these changes also need to be made on the hfsplus
filesystem as well.
Reported-by: Timo Warns <[email protected]>
Acked-by: WANG Cong <[email protected]>
Cc: Alexey Khoroshilov <[email protected]>
Cc: Miklos Szeredi <[email protected]>
Cc: Sage Weil <[email protected]>
Cc: Eugene Teo <[email protected]>
Cc: Roman Zippel <[email protected]>
Cc: Al Viro <[email protected]>
Cc: Christoph Hellwig <[email protected]>
Cc: Alexey Dobriyan <[email protected]>
Cc: Dave Anderson <[email protected]>
Cc: stable <[email protected]>
Cc: Andrew Morton <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
This commit is contained in:
parent
f756beba94
commit
6f24f89287
2 changed files with 15 additions and 0 deletions
|
|
@ -366,6 +366,10 @@ int hfsplus_rename_cat(u32 cnid,
|
|||
err = hfs_brec_find(&src_fd);
|
||||
if (err)
|
||||
goto out;
|
||||
if (src_fd.entrylength > sizeof(entry) || src_fd.entrylength < 0) {
|
||||
err = -EIO;
|
||||
goto out;
|
||||
}
|
||||
|
||||
hfs_bnode_read(src_fd.bnode, &entry, src_fd.entryoffset,
|
||||
src_fd.entrylength);
|
||||
|
|
|
|||
|
|
@ -150,6 +150,11 @@ static int hfsplus_readdir(struct file *filp, void *dirent, filldir_t filldir)
|
|||
filp->f_pos++;
|
||||
/* fall through */
|
||||
case 1:
|
||||
if (fd.entrylength > sizeof(entry) || fd.entrylength < 0) {
|
||||
err = -EIO;
|
||||
goto out;
|
||||
}
|
||||
|
||||
hfs_bnode_read(fd.bnode, &entry, fd.entryoffset,
|
||||
fd.entrylength);
|
||||
if (be16_to_cpu(entry.type) != HFSPLUS_FOLDER_THREAD) {
|
||||
|
|
@ -181,6 +186,12 @@ static int hfsplus_readdir(struct file *filp, void *dirent, filldir_t filldir)
|
|||
err = -EIO;
|
||||
goto out;
|
||||
}
|
||||
|
||||
if (fd.entrylength > sizeof(entry) || fd.entrylength < 0) {
|
||||
err = -EIO;
|
||||
goto out;
|
||||
}
|
||||
|
||||
hfs_bnode_read(fd.bnode, &entry, fd.entryoffset,
|
||||
fd.entrylength);
|
||||
type = be16_to_cpu(entry.type);
|
||||
|
|
|
|||
Loading…
Reference in a new issue