diff options
| author | Dave Kleikamp <dave.kleikamp@oracle.com> | 2013-09-06 21:49:56 -0500 |
|---|---|---|
| committer | Dave Kleikamp <dave.kleikamp@oracle.com> | 2013-09-17 10:05:19 -0500 |
| commit | 8660998608cfa1077e560034db81885af8e1e885 (patch) | |
| tree | e4b7e7dc0fa1806a9241f1bb7eb9f386391d0960 | |
| parent | 44598f98b98e54beca34dc836b38eaea40be1abf (diff) | |
| download | op-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.c | 3 |
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: |
