diff options
| author | Wengang Wang <wen.gang.wang@oracle.com> | 2011-10-12 15:22:15 +0800 |
|---|---|---|
| committer | Joel Becker <jlbec@evilplan.org> | 2011-11-17 01:46:47 -0800 |
| commit | b8a0ae579fb8d9b21008ac386be08b9428902455 (patch) | |
| tree | cab8d712ee687c26bd2e1ce8ba7b069be4a7e92a /fs/ocfs2/ioctl.c | |
| parent | 8298524803339a9a8df053ebdfebc2975ec55be9 (diff) | |
| download | op-kernel-dev-b8a0ae579fb8d9b21008ac386be08b9428902455.zip op-kernel-dev-b8a0ae579fb8d9b21008ac386be08b9428902455.tar.gz | |
ocfs2: Commit transactions in error cases -v2
There are three cases found that in error cases, journal transactions are not
committed nor aborted. We should take care of these case by committing the
transactions. Otherwise, there would left a journal handle which will lead to
, in same process context, the comming ocfs2_start_trans() gets wrong credits.
Signed-off-by: Wengang Wang <wen.gang.wang@oracle.com>
Signed-off-by: Joel Becker <jlbec@evilplan.org>
Diffstat (limited to 'fs/ocfs2/ioctl.c')
| -rw-r--r-- | fs/ocfs2/ioctl.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/ocfs2/ioctl.c b/fs/ocfs2/ioctl.c index d9a6ce7..726ff26 100644 --- a/fs/ocfs2/ioctl.c +++ b/fs/ocfs2/ioctl.c @@ -122,7 +122,7 @@ static int ocfs2_set_inode_attr(struct inode *inode, unsigned flags, if ((oldflags & OCFS2_IMMUTABLE_FL) || ((flags ^ oldflags) & (OCFS2_APPEND_FL | OCFS2_IMMUTABLE_FL))) { if (!capable(CAP_LINUX_IMMUTABLE)) - goto bail_unlock; + goto bail_commit; } ocfs2_inode->ip_attr = flags; @@ -132,6 +132,7 @@ static int ocfs2_set_inode_attr(struct inode *inode, unsigned flags, if (status < 0) mlog_errno(status); +bail_commit: ocfs2_commit_trans(osb, handle); bail_unlock: ocfs2_inode_unlock(inode, 1); |
