diff options
| author | Al Viro <viro@zeniv.linux.org.uk> | 2008-12-04 10:06:33 -0500 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2009-01-05 11:54:28 -0500 |
| commit | acfa4380efe77e290d3a96b11cd4c9f24f4fbb18 (patch) | |
| tree | d656232c7ef39c83681c2de4c8e28ba439242f66 /fs/stat.c | |
| parent | 9742df331deb3fce95b321f38d4ea0c4e75edb63 (diff) | |
| download | linux-stable-acfa4380efe77e290d3a96b11cd4c9f24f4fbb18.tar.gz linux-stable-acfa4380efe77e290d3a96b11cd4c9f24f4fbb18.tar.bz2 linux-stable-acfa4380efe77e290d3a96b11cd4c9f24f4fbb18.zip | |
inode->i_op is never NULL
We used to have rather schizophrenic set of checks for NULL ->i_op even
though it had been eliminated years ago. You'd need to go out of your
way to set it to NULL explicitly _and_ a bunch of code would die on
such inodes anyway. After killing two remaining places that still
did that bogosity, all that crap can go away.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/stat.c')
| -rw-r--r-- | fs/stat.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/stat.c b/fs/stat.c index 7c46fbeb8b76..7e12a6f82795 100644 --- a/fs/stat.c +++ b/fs/stat.c @@ -305,7 +305,7 @@ asmlinkage long sys_readlinkat(int dfd, const char __user *pathname, struct inode *inode = path.dentry->d_inode; error = -EINVAL; - if (inode->i_op && inode->i_op->readlink) { + if (inode->i_op->readlink) { error = security_inode_readlink(path.dentry); if (!error) { touch_atime(path.mnt, path.dentry); |