diff options
| author | Eric Sandeen <sandeen@sandeen.net> | 2011-12-06 17:06:06 -0600 |
|---|---|---|
| committer | David Woodhouse <David.Woodhouse@intel.com> | 2012-01-09 18:18:03 +0000 |
| commit | 53466710202900ce49e471f480cac11275e1d0c4 (patch) | |
| tree | c9a4d9458dff37cd7e15f2181c8bc1d5edcf3f91 | |
| parent | fa3ae714c763f3e9d8fd876879338d2b674b8db2 (diff) | |
| download | op-kernel-dev-53466710202900ce49e471f480cac11275e1d0c4.zip op-kernel-dev-53466710202900ce49e471f480cac11275e1d0c4.tar.gz | |
jffs2: fix up error handling for insert_inode_locked
after 250df6ed274d767da844a5d9f05720b804240197
(fs: protect inode->i_state with inode->i_lock), insert_inode_locked()
no longer returns the inode with I_NEW set on failure. However,
the error handler still calls unlock_new_inode() on failure,
which does a WARN_ON if I_NEW is not set, so any failure spews
a lot of warnings.
We can just drop the unlock_new_inode() if insert_inode_locked()
fails here.
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@linux.intel.com>
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
| -rw-r--r-- | fs/jffs2/fs.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/jffs2/fs.c b/fs/jffs2/fs.c index 4b8afe3..2e01238 100644 --- a/fs/jffs2/fs.c +++ b/fs/jffs2/fs.c @@ -466,7 +466,6 @@ struct inode *jffs2_new_inode (struct inode *dir_i, umode_t mode, struct jffs2_r if (insert_inode_locked(inode) < 0) { make_bad_inode(inode); - unlock_new_inode(inode); iput(inode); return ERR_PTR(-EINVAL); } |
