diff options
author | Tao Ma <boyu.mt@taobao.com> | 2012-06-07 19:04:19 -0400 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2012-06-07 19:04:19 -0400 |
commit | b22b1f178f6799278d3178d894f37facb2085765 (patch) | |
tree | 7d5b9559492a79ebddcfdab5c9fc2743dec26922 /fs/ext4 | |
parent | b0dd6b70f0fda17ae9762fbb72d98e40a4f66556 (diff) | |
download | linux-b22b1f178f6799278d3178d894f37facb2085765.tar.gz linux-b22b1f178f6799278d3178d894f37facb2085765.tar.bz2 linux-b22b1f178f6799278d3178d894f37facb2085765.zip |
ext4: don't set i_flags in EXT4_IOC_SETFLAGS
Commit 7990696 uses the ext4_{set,clear}_inode_flags() functions to
change the i_flags automatically but fails to remove the error setting
of i_flags. So we still have the problem of trashing state flags.
Fix this by removing the assignment.
Signed-off-by: Tao Ma <boyu.mt@taobao.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Cc: stable@kernel.org
Diffstat (limited to 'fs/ext4')
-rw-r--r-- | fs/ext4/ioctl.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/ext4/ioctl.c b/fs/ext4/ioctl.c index 8ad112ae0ade..e34deac3f366 100644 --- a/fs/ext4/ioctl.c +++ b/fs/ext4/ioctl.c @@ -123,7 +123,6 @@ long ext4_ioctl(struct file *filp, unsigned int cmd, unsigned long arg) else ext4_clear_inode_flag(inode, i); } - ei->i_flags = flags; ext4_set_inode_flags(inode); inode->i_ctime = ext4_current_time(inode); |