Commit a9743fcd authored by Mark Fasheh's avatar Mark Fasheh

ocfs2: Add directory entry later in ocfs2_symlink() and ocfs2_mknod()

If we get a failure during creation of an inode we'll allow the orphan code
to remove the inode, which is correct. However, we need to ensure that we
don't get any errors after the call to ocfs2_add_entry(), otherwise we could
leave a dangling directory reference. The solution is simple - in both
cases, all I had to do was move ocfs2_dentry_attach_lock() above the
ocfs2_add_entry() call.
Signed-off-by: default avatarMark Fasheh <mfasheh@suse.com>
parent 062d3403
...@@ -408,23 +408,28 @@ static int ocfs2_mknod(struct inode *dir, ...@@ -408,23 +408,28 @@ static int ocfs2_mknod(struct inode *dir,
} }
} }
status = ocfs2_add_entry(handle, dentry, inode, /*
OCFS2_I(inode)->ip_blkno, parent_fe_bh, * Do this before adding the entry to the directory. We add
&lookup); * also set d_op after success so that ->d_iput() will cleanup
if (status < 0) { * the dentry lock even if ocfs2_add_entry() fails below.
*/
status = ocfs2_dentry_attach_lock(dentry, inode,
OCFS2_I(dir)->ip_blkno);
if (status) {
mlog_errno(status); mlog_errno(status);
goto leave; goto leave;
} }
dentry->d_op = &ocfs2_dentry_ops;
status = ocfs2_dentry_attach_lock(dentry, inode, status = ocfs2_add_entry(handle, dentry, inode,
OCFS2_I(dir)->ip_blkno); OCFS2_I(inode)->ip_blkno, parent_fe_bh,
if (status) { &lookup);
if (status < 0) {
mlog_errno(status); mlog_errno(status);
goto leave; goto leave;
} }
insert_inode_hash(inode); insert_inode_hash(inode);
dentry->d_op = &ocfs2_dentry_ops;
d_instantiate(dentry, inode); d_instantiate(dentry, inode);
status = 0; status = 0;
leave: leave:
...@@ -1777,22 +1782,27 @@ static int ocfs2_symlink(struct inode *dir, ...@@ -1777,22 +1782,27 @@ static int ocfs2_symlink(struct inode *dir,
} }
} }
status = ocfs2_add_entry(handle, dentry, inode, /*
le64_to_cpu(fe->i_blkno), parent_fe_bh, * Do this before adding the entry to the directory. We add
&lookup); * also set d_op after success so that ->d_iput() will cleanup
if (status < 0) { * the dentry lock even if ocfs2_add_entry() fails below.
*/
status = ocfs2_dentry_attach_lock(dentry, inode, OCFS2_I(dir)->ip_blkno);
if (status) {
mlog_errno(status); mlog_errno(status);
goto bail; goto bail;
} }
dentry->d_op = &ocfs2_dentry_ops;
status = ocfs2_dentry_attach_lock(dentry, inode, OCFS2_I(dir)->ip_blkno); status = ocfs2_add_entry(handle, dentry, inode,
if (status) { le64_to_cpu(fe->i_blkno), parent_fe_bh,
&lookup);
if (status < 0) {
mlog_errno(status); mlog_errno(status);
goto bail; goto bail;
} }
insert_inode_hash(inode); insert_inode_hash(inode);
dentry->d_op = &ocfs2_dentry_ops;
d_instantiate(dentry, inode); d_instantiate(dentry, inode);
bail: bail:
if (status < 0 && did_quota) if (status < 0 && did_quota)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment