On 09/05, Chao Yu wrote: > Hi Jaegeuk, > > Could you update dev-test branch? so I can rebase my patch on it. :)
Will do soon. :) > > 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
