diff options
| author |   Michael Ellerman <michael@ellerman.id.au> | 2008-12-01 20:59:07 -0800 |
|---|---|---|
| committer |   Roland Dreier <rolandd@cisco.com> | 2008-12-01 20:59:07 -0800 |
| commit | 64f22fa17c1a531e682ebc882566856ea5718495 (patch) | |
| tree | 29737ea55e7ddf7f00267b96c2d179a62dff4835 /drivers/infiniband/hw/ipath | |
| parent | Linux 2.6.28-rc7 (diff) | |
| download | linux-dev-64f22fa17c1a531e682ebc882566856ea5718495.tar.xz linux-dev-64f22fa17c1a531e682ebc882566856ea5718495.zip | |
IB/ipath: Fix pointer-to-pointer thinko in ipath_fs.c
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>
Diffstat (limited to 'drivers/infiniband/hw/ipath')
| -rw-r--r-- | drivers/infiniband/hw/ipath/ipath_fs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/infiniband/hw/ipath/ipath_fs.c b/drivers/infiniband/hw/ipath/ipath_fs.c index 8bb5170b4e41..53912c327bfe 100644 --- a/drivers/infiniband/hw/ipath/ipath_fs.c +++ b/drivers/infiniband/hw/ipath/ipath_fs.c @@ -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 |