diff options
| author |   Alan Cox <alan@linux.intel.com> | 2012-09-19 15:49:51 +0100 |
|---|---|---|
| committer |   Al Viro <viro@zeniv.linux.org.uk> | 2012-09-26 22:20:19 -0400 |
| commit | 1fe0c0230a7c2d5f4061e681a3f3be9512446d23 (patch) | |
| tree | 611103e01c1bc579436d376176a69cf4d903c6f5 /fs | |
| parent | switch simple cases of fget_light to fdget (diff) | |
| download | linux-dev-1fe0c0230a7c2d5f4061e681a3f3be9512446d23.tar.xz linux-dev-1fe0c0230a7c2d5f4061e681a3f3be9512446d23.zip | |
vfs: delete surplus inode NULL check
Each iteration of d_delete we reload inode from dentry->d_inode and
then call S_ISDIR(inode-i_mode), so inode cannot possibly be NULL
shortly afterwards unless something went horribly wrong.
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/dcache.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/dcache.c b/fs/dcache.c index 16521a9f2038..fbee67b92651 100644 --- a/fs/dcache.c +++ b/fs/dcache.c @@ -2109,7 +2109,7 @@ again: inode = dentry->d_inode; isdir = S_ISDIR(inode->i_mode); if (dentry->d_count == 1) { - if (inode && !spin_trylock(&inode->i_lock)) { + if (!spin_trylock(&inode->i_lock)) { spin_unlock(&dentry->d_lock); cpu_relax(); goto again; |