diff options
| author | Chao Yu <chao@kernel.org> | 2023-01-09 11:44:50 +0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2023-03-11 13:55:18 +0100 |
| commit | 0b65ff13f943a9372f782cf75451c78d435e4388 (patch) | |
| tree | a846f018c04e8a1b9f8db80699d4c4e510383412 /fs/f2fs/segment.c | |
| parent | 4bc488c5ca0e2d3c4a419572faf636e4d9a8dfc9 (diff) | |
| download | linux-0b65ff13f943a9372f782cf75451c78d435e4388.tar.gz | |
f2fs: clear atomic_write_task in f2fs_abort_atomic_write()
[ Upstream commit 0e8d040bfa4c476d7d2a23119527c744c7de13cd ]
Otherwise, last .atomic_write_task will be remained in structure
f2fs_inode_info, resulting in aborting atomic_write accidentally
in race case. Meanwhile, clear original_i_size as well.
Fixes: 7a10f0177e11 ("f2fs: don't give partially written atomic data from process crash")
Signed-off-by: Chao Yu <chao@kernel.org>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'fs/f2fs/segment.c')
| -rw-r--r-- | fs/f2fs/segment.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c index 60d79e427f98..63c689409979 100644 --- a/fs/f2fs/segment.c +++ b/fs/f2fs/segment.c @@ -200,9 +200,12 @@ void f2fs_abort_atomic_write(struct inode *inode, bool clean) clear_inode_flag(inode, FI_ATOMIC_FILE); stat_dec_atomic_inode(inode); + F2FS_I(inode)->atomic_write_task = NULL; + if (clean) { truncate_inode_pages_final(inode->i_mapping); f2fs_i_size_write(inode, fi->original_i_size); + fi->original_i_size = 0; } } |