summaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorTejun Heo <htejun@gmail.com>2007-06-11 14:03:27 +0900
committerGreg Kroah-Hartman <gregkh@suse.de>2007-06-12 16:08:46 -0700
commit6aa054aadfea613a437ad0b15d38eca2b963fc0a (patch)
treee7fd512672b8f85fe2463ba4ba98bca06ffe6bb1 /fs
parentdc351252b33f8fede396d6173dba117bcb933607 (diff)
downloadlinux-6aa054aadfea613a437ad0b15d38eca2b963fc0a.tar.gz
linux-6aa054aadfea613a437ad0b15d38eca2b963fc0a.tar.bz2
linux-6aa054aadfea613a437ad0b15d38eca2b963fc0a.zip
sysfs: fix condition check in sysfs_drop_dentry()
The condition check doesn't make much sense as it basically always succeeds. This causes NULL dereferencing on certain cases. It seems that parentheses are put in the wrong place. Fix it. Signed-off-by: Tejun Heo <htejun@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'fs')
-rw-r--r--fs/sysfs/inode.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/sysfs/inode.c b/fs/sysfs/inode.c
index 082e2d430e93..38bbe071cc15 100644
--- a/fs/sysfs/inode.c
+++ b/fs/sysfs/inode.c
@@ -252,7 +252,7 @@ void sysfs_drop_dentry(struct sysfs_dirent * sd, struct dentry * parent)
if (dentry) {
spin_lock(&dcache_lock);
spin_lock(&dentry->d_lock);
- if (!(d_unhashed(dentry) && dentry->d_inode)) {
+ if (!d_unhashed(dentry) && dentry->d_inode) {
inode = dentry->d_inode;
spin_lock(&inode->i_lock);
__iget(inode);