IB/ipath: Fix pointer-to-pointer thinko in ipath_fs.c
authorMichael Ellerman <michael@ellerman.id.au>
Tue, 2 Dec 2008 04:59:07 +0000 (20:59 -0800)
committerRoland Dreier <rolandd@cisco.com>
Tue, 2 Dec 2008 04:59:07 +0000 (20:59 -0800)
The return from lookup_one_len() is assigned to *dentry, so that's
what we should be checking with IS_ERR().

Signed-off-by: Michael Ellerman <michael@ellerman.id.au>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
drivers/infiniband/hw/ipath/ipath_fs.c

index 8bb5170b4e416b1b660f333561ffa7c8bb4c7360..53912c327bfea25dc4bab902d788042973441140 100644 (file)
@@ -86,7 +86,7 @@ static int create_file(const char *name, mode_t mode,
        *dentry = NULL;
        mutex_lock(&parent->d_inode->i_mutex);
        *dentry = lookup_one_len(name, parent, strlen(name));
-       if (!IS_ERR(dentry))
+       if (!IS_ERR(*dentry))
                error = ipathfs_mknod(parent->d_inode, *dentry,
                                      mode, fops, data);
        else