Re: [f2fs-dev] [PATCH v4] f2fs: add check for deleted inode

2025-02-27 Thread Chao Yu via Linux-f2fs-devel
On 2025/2/27 1:15, Jaegeuk Kim wrote: On 02/26, Chao Yu wrote: On 2/26/25 11:28, Jaegeuk Kim wrote: On 02/24, Chao Yu wrote: On 2/14/25 09:44, Chao Yu wrote: On 2/14/25 01:38, Jaegeuk Kim wrote: On 02/13, Chao Yu wrote: On 2/13/25 00:47, Jaegeuk Kim wrote: On 02/12, Chao Yu wrote: From: L

Re: [f2fs-dev] [PATCH v4] f2fs: add check for deleted inode

2025-02-26 Thread Jaegeuk Kim via Linux-f2fs-devel
On 02/26, Chao Yu wrote: > On 2/26/25 11:28, Jaegeuk Kim wrote: > > On 02/24, Chao Yu wrote: > >> On 2/14/25 09:44, Chao Yu wrote: > >>> On 2/14/25 01:38, Jaegeuk Kim wrote: > On 02/13, Chao Yu wrote: > > On 2/13/25 00:47, Jaegeuk Kim wrote: > >> On 02/12, Chao Yu wrote: > >>> From

Re: [f2fs-dev] [PATCH v4] f2fs: add check for deleted inode

2025-02-25 Thread Chao Yu via Linux-f2fs-devel
On 2/26/25 11:28, Jaegeuk Kim wrote: > On 02/24, Chao Yu wrote: >> On 2/14/25 09:44, Chao Yu wrote: >>> On 2/14/25 01:38, Jaegeuk Kim wrote: On 02/13, Chao Yu wrote: > On 2/13/25 00:47, Jaegeuk Kim wrote: >> On 02/12, Chao Yu wrote: >>> From: Leo Stone >>> >>> The syzbot r

Re: [f2fs-dev] [PATCH v4] f2fs: add check for deleted inode

2025-02-25 Thread Jaegeuk Kim via Linux-f2fs-devel
On 02/24, Chao Yu wrote: > On 2/14/25 09:44, Chao Yu wrote: > > On 2/14/25 01:38, Jaegeuk Kim wrote: > >> On 02/13, Chao Yu wrote: > >>> On 2/13/25 00:47, Jaegeuk Kim wrote: > On 02/12, Chao Yu wrote: > > From: Leo Stone > > > > The syzbot reproducer mounts a f2fs image, then trie

Re: [f2fs-dev] [PATCH v4] f2fs: add check for deleted inode

2025-02-24 Thread Chao Yu via Linux-f2fs-devel
On 2/14/25 09:44, Chao Yu wrote: > On 2/14/25 01:38, Jaegeuk Kim wrote: >> On 02/13, Chao Yu wrote: >>> On 2/13/25 00:47, Jaegeuk Kim wrote: On 02/12, Chao Yu wrote: > From: Leo Stone > > The syzbot reproducer mounts a f2fs image, then tries to unlink an > existing file. Howev

Re: [f2fs-dev] [PATCH v4] f2fs: add check for deleted inode

2025-02-13 Thread Chao Yu via Linux-f2fs-devel
On 2/14/25 01:38, Jaegeuk Kim wrote: > On 02/13, Chao Yu wrote: >> On 2/13/25 00:47, Jaegeuk Kim wrote: >>> On 02/12, Chao Yu wrote: From: Leo Stone The syzbot reproducer mounts a f2fs image, then tries to unlink an existing file. However, the unlinked file already has a link c

Re: [f2fs-dev] [PATCH v4] f2fs: add check for deleted inode

2025-02-13 Thread Jaegeuk Kim via Linux-f2fs-devel
On 02/13, Chao Yu wrote: > On 2/13/25 00:47, Jaegeuk Kim wrote: > > On 02/12, Chao Yu wrote: > >> From: Leo Stone > >> > >> The syzbot reproducer mounts a f2fs image, then tries to unlink an > >> existing file. However, the unlinked file already has a link count of 0 > >> when it is read for the f

Re: [f2fs-dev] [PATCH v4] f2fs: add check for deleted inode

2025-02-12 Thread Chao Yu via Linux-f2fs-devel
On 2/13/25 00:47, Jaegeuk Kim wrote: > On 02/12, Chao Yu wrote: >> From: Leo Stone >> >> The syzbot reproducer mounts a f2fs image, then tries to unlink an >> existing file. However, the unlinked file already has a link count of 0 >> when it is read for the first time in do_read_inode(). >> >> Add

Re: [f2fs-dev] [PATCH v4] f2fs: add check for deleted inode

2025-02-12 Thread Jaegeuk Kim via Linux-f2fs-devel
On 02/12, Chao Yu wrote: > From: Leo Stone > > The syzbot reproducer mounts a f2fs image, then tries to unlink an > existing file. However, the unlinked file already has a link count of 0 > when it is read for the first time in do_read_inode(). > > Add a check to sanity_check_inode() for i_nlink