summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDave Kleikamp <dave.kleikamp@oracle.com>2013-09-06 21:49:56 -0500
committerDave Kleikamp <dave.kleikamp@oracle.com>2013-09-17 10:05:19 -0500
commit8660998608cfa1077e560034db81885af8e1e885 (patch)
treee4b7e7dc0fa1806a9241f1bb7eb9f386391d0960
parent44598f98b98e54beca34dc836b38eaea40be1abf (diff)
downloadop-kernel-dev-8660998608cfa1077e560034db81885af8e1e885.zip
op-kernel-dev-8660998608cfa1077e560034db81885af8e1e885.tar.gz
jfs: fix error path in ialloc
If insert_inode_locked() fails, we shouldn't be calling unlock_new_inode(). Signed-off-by: Dave Kleikamp <dave.kleikamp@oracle.com> Tested-by: Michael L. Semon <mlsemon35@gmail.com> Cc: stable@vger.kernel.org
-rw-r--r--fs/jfs/jfs_inode.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/jfs/jfs_inode.c b/fs/jfs/jfs_inode.c
index c1a3e60..7f464c5 100644
--- a/fs/jfs/jfs_inode.c
+++ b/fs/jfs/jfs_inode.c
@@ -95,7 +95,7 @@ struct inode *ialloc(struct inode *parent, umode_t mode)
if (insert_inode_locked(inode) < 0) {
rc = -EINVAL;
- goto fail_unlock;
+ goto fail_put;
}
inode_init_owner(inode, parent, mode);
@@ -156,7 +156,6 @@ struct inode *ialloc(struct inode *parent, umode_t mode)
fail_drop:
dquot_drop(inode);
inode->i_flags |= S_NOQUOTA;
-fail_unlock:
clear_nlink(inode);
unlock_new_inode(inode);
fail_put:
OpenPOWER on IntegriCloud