diff options
author | Darrick J. Wong <darrick.wong@oracle.com> | 2017-01-09 10:38:33 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-01-12 05:39:39 -0500 |
commit | a11f90ca5f306a1c38db4f24db8bff97ffb87bd2 (patch) | |
tree | 120410e3a0339707c1f79254bf3eb91d3cbb1852 /fs | |
parent | ae8b6cb40cb29dfa760641ce55d5b9c0c153a1f2 (diff) |
xfs: check return value of _trans_reserve_quota_nblks
commit 4fd29ec47212c8cbf98916af519019ccc5e58e49 upstream.
Check the return value of xfs_trans_reserve_quota_nblks for errors.
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>
Cc: Christoph Hellwig <hch@lst.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/xfs/libxfs/xfs_bmap.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/fs/xfs/libxfs/xfs_bmap.c b/fs/xfs/libxfs/xfs_bmap.c index c6eb21940783..71dd6d783710 100644 --- a/fs/xfs/libxfs/xfs_bmap.c +++ b/fs/xfs/libxfs/xfs_bmap.c | |||
@@ -4902,8 +4902,11 @@ xfs_bmap_del_extent_delay( | |||
4902 | * sb counters as we might have to borrow some blocks for the | 4902 | * sb counters as we might have to borrow some blocks for the |
4903 | * indirect block accounting. | 4903 | * indirect block accounting. |
4904 | */ | 4904 | */ |
4905 | xfs_trans_reserve_quota_nblks(NULL, ip, -((long)del->br_blockcount), 0, | 4905 | error = xfs_trans_reserve_quota_nblks(NULL, ip, |
4906 | -((long)del->br_blockcount), 0, | ||
4906 | isrt ? XFS_QMOPT_RES_RTBLKS : XFS_QMOPT_RES_REGBLKS); | 4907 | isrt ? XFS_QMOPT_RES_RTBLKS : XFS_QMOPT_RES_REGBLKS); |
4908 | if (error) | ||
4909 | return error; | ||
4907 | ip->i_delayed_blks -= del->br_blockcount; | 4910 | ip->i_delayed_blks -= del->br_blockcount; |
4908 | 4911 | ||
4909 | if (whichfork == XFS_COW_FORK) | 4912 | if (whichfork == XFS_COW_FORK) |