diff options
| author | Chao Yu <chao@kernel.org> | 2023-01-09 11:44:50 +0800 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk@kernel.org> | 2023-01-30 14:46:20 -0800 |
| commit | 0e8d040bfa4c476d7d2a23119527c744c7de13cd (patch) | |
| tree | ab75a5cb5a5f7e2ae9eabf4f163bffcf509b5a55 | |
| parent | 2f3a9ae990a7881c9a57a073bb52ebe34fdc3160 (diff) | |
| download | linux-stable-0e8d040bfa4c476d7d2a23119527c744c7de13cd.tar.gz linux-stable-0e8d040bfa4c476d7d2a23119527c744c7de13cd.tar.bz2 linux-stable-0e8d040bfa4c476d7d2a23119527c744c7de13cd.zip | |
f2fs: clear atomic_write_task in f2fs_abort_atomic_write()
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>
| -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 ebfa759527ac..5e603b808487 100644 --- a/fs/f2fs/segment.c +++ b/fs/f2fs/segment.c @@ -201,9 +201,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; } } |