提交 7e79eedb 编写于 作者: T Tetsuo Handa 提交者: Al Viro

[patch 4/5] vfs: reuse local variable in vfs_link()

Why not reuse "inode" which is assigned as

  struct inode *inode = old_dentry->d_inode;

in the beginning of vfs_link() ?
Signed-off-by: NTetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Signed-off-by: NMiklos Szeredi <mszeredi@suse.cz>
上级 2f1936b8
...@@ -2524,19 +2524,19 @@ int vfs_link(struct dentry *old_dentry, struct inode *dir, struct dentry *new_de ...@@ -2524,19 +2524,19 @@ int vfs_link(struct dentry *old_dentry, struct inode *dir, struct dentry *new_de
return -EPERM; return -EPERM;
if (!dir->i_op || !dir->i_op->link) if (!dir->i_op || !dir->i_op->link)
return -EPERM; return -EPERM;
if (S_ISDIR(old_dentry->d_inode->i_mode)) if (S_ISDIR(inode->i_mode))
return -EPERM; return -EPERM;
error = security_inode_link(old_dentry, dir, new_dentry); error = security_inode_link(old_dentry, dir, new_dentry);
if (error) if (error)
return error; return error;
mutex_lock(&old_dentry->d_inode->i_mutex); mutex_lock(&inode->i_mutex);
DQUOT_INIT(dir); DQUOT_INIT(dir);
error = dir->i_op->link(old_dentry, dir, new_dentry); error = dir->i_op->link(old_dentry, dir, new_dentry);
mutex_unlock(&old_dentry->d_inode->i_mutex); mutex_unlock(&inode->i_mutex);
if (!error) if (!error)
fsnotify_link(dir, old_dentry->d_inode, new_dentry); fsnotify_link(dir, inode, new_dentry);
return error; return error;
} }
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册