diff options
| author | Jan Kara <jack@suse.cz> | 2024-02-09 14:21:01 +0300 |
|---|---|---|
| committer | Theodore Ts'o <tytso@mit.edu> | 2024-03-07 21:32:54 +0300 |
| commit | 7f48212678e91a057259b3e281701f7feb1ee397 (patch) | |
| tree | 6e5b1ff719f044a57dd94b43eb884e367ef0d2e0 /fs/ext4 | |
| parent | 8208c41c43ad5e9b63dce6c45a73e326109ca658 (diff) | |
| download | linux-7f48212678e91a057259b3e281701f7feb1ee397.tar.xz | |
ext4: drop duplicate ea_inode handling in ext4_xattr_block_set()
ext4_xattr_block_set() drops ea_inode reference in two places. Handling
it just under the 'cleanup' label is enough so drop the second
occurence.
Signed-off-by: Jan Kara <jack@suse.cz>
Link: https://lore.kernel.org/r/20240209112107.10585-3-jack@suse.cz
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Diffstat (limited to 'fs/ext4')
| -rw-r--r-- | fs/ext4/xattr.c | 11 |
1 files changed, 0 insertions, 11 deletions
diff --git a/fs/ext4/xattr.c b/fs/ext4/xattr.c index 146690c10c73..b67a176bfcf9 100644 --- a/fs/ext4/xattr.c +++ b/fs/ext4/xattr.c @@ -2158,17 +2158,6 @@ getblk_failed: ENTRY(header(s->base)+1)); if (error) goto getblk_failed; - if (ea_inode) { - /* Drop the extra ref on ea_inode. */ - error = ext4_xattr_inode_dec_ref(handle, - ea_inode); - if (error) - ext4_warning_inode(ea_inode, - "dec ref error=%d", - error); - iput(ea_inode); - ea_inode = NULL; - } lock_buffer(new_bh); error = ext4_journal_get_create_access(handle, sb, |