On 08/15, Jaegeuk Kim wrote:
> On 08/12, Chao Yu wrote:
> > It tries to let fsck be noticed wrong inline size, and do the fix.
> > 
> > Signed-off-by: Chao Yu <yuch...@huawei.com>
> > ---
> >  fsck/fsck.c | 8 ++++++++
> >  1 file changed, 8 insertions(+)
> > 
> > diff --git a/fsck/fsck.c b/fsck/fsck.c
> > index d53317c..7eb599d 100644
> > --- a/fsck/fsck.c
> > +++ b/fsck/fsck.c
> > @@ -771,6 +771,8 @@ void fsck_chk_inode_blk(struct f2fs_sb_info *sbi, u32 
> > nid,
> >     ofs = get_extra_isize(node_blk);
> >  
> >     if ((node_blk->i.i_inline & F2FS_INLINE_DATA)) {
> > +           unsigned int inline_size = MAX_INLINE_DATA(node_blk);
> > +
> >             if (le32_to_cpu(node_blk->i.i_addr[ofs]) != 0) {
> >                     /* should fix this bug all the time */
> >                     FIX_MSG("inline_data has wrong 0'th block = %x",
> > @@ -779,6 +781,12 @@ void fsck_chk_inode_blk(struct f2fs_sb_info *sbi, u32 
> > nid,
> >                     node_blk->i.i_blocks = cpu_to_le64(*blk_cnt);
> >                     need_fix = 1;
> >             }
> > +           if (!i_size || i_size > inline_size) {
> 
> i_size=0 should be fine?

Missed v2.

> 
> > +                   node_blk->i.i_size = cpu_to_le64(inline_size);
> > +                   FIX_MSG("inline_data has wrong i_size %lu",
> > +                                           (unsigned long)i_size);
> > +                   need_fix = 1;
> > +           }
> >             if (!(node_blk->i.i_inline & F2FS_DATA_EXIST)) {
> >                     char buf[MAX_INLINE_DATA(node_blk)];
> >                     memset(buf, 0, MAX_INLINE_DATA(node_blk));
> > -- 
> > 2.18.0.rc1
> 
> 
> _______________________________________________
> Linux-f2fs-devel mailing list
> Linux-f2fs-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel


_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to