Hi Jaegeuk, Could you update dev-test branch? so I can rebase my patch on it. :)
On 2018/9/5 12:28, Jaegeuk Kim wrote: > On 08/30, Chengguang Xu wrote: >> Add additinal sanity check for irregular case(e.g. corruption). >> If size of extended attribution is smaller than size of acl header, >> then return -EINVAL. >> >> Signed-off-by: Chengguang Xu <cgxu...@gmx.com> >> --- >> fs/f2fs/acl.c | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/fs/f2fs/acl.c b/fs/f2fs/acl.c >> index 111824199a88..79e9ea773070 100644 >> --- a/fs/f2fs/acl.c >> +++ b/fs/f2fs/acl.c >> @@ -53,6 +53,9 @@ static struct posix_acl *f2fs_acl_from_disk(const char >> *value, size_t size) >> struct f2fs_acl_entry *entry = (struct f2fs_acl_entry *)(hdr + 1); >> const char *end = value + size; >> >> + if (size < sizeof(f2fs_acl_header)) > > sizeof(struct f2fs_acl_header) > > I fixed the above, and merged. > > Thanks, > >> + return ERR_PTR(-EINVAL); >> + >> if (hdr->a_version != cpu_to_le32(F2FS_ACL_VERSION)) >> return ERR_PTR(-EINVAL); >> >> -- >> 2.17.1 > > . > ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot _______________________________________________ Linux-f2fs-devel mailing list Linux-f2fs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
