Commit a6247b01 authored by Darrick J. Wong's avatar Darrick J. Wong Committed by Greg Kroah-Hartman

xfs: check _btree_check_block value

commit 1e86eabe upstream.

Check the _btree_check_block return value for the firstrec and lastrec
functions, since we have the ability to signal that the repositioning
did not succeed.

Fixes-coverity-id: 114067
Fixes-coverity-id: 114068
Signed-off-by: default avatarDarrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: default avatarBrian Foster <bfoster@redhat.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent e76496fa
...@@ -714,7 +714,8 @@ xfs_btree_firstrec( ...@@ -714,7 +714,8 @@ xfs_btree_firstrec(
* Get the block pointer for this level. * Get the block pointer for this level.
*/ */
block = xfs_btree_get_block(cur, level, &bp); block = xfs_btree_get_block(cur, level, &bp);
xfs_btree_check_block(cur, block, level, bp); if (xfs_btree_check_block(cur, block, level, bp))
return 0;
/* /*
* It's empty, there is no such record. * It's empty, there is no such record.
*/ */
...@@ -743,7 +744,8 @@ xfs_btree_lastrec( ...@@ -743,7 +744,8 @@ xfs_btree_lastrec(
* Get the block pointer for this level. * Get the block pointer for this level.
*/ */
block = xfs_btree_get_block(cur, level, &bp); block = xfs_btree_get_block(cur, level, &bp);
xfs_btree_check_block(cur, block, level, bp); if (xfs_btree_check_block(cur, block, level, bp))
return 0;
/* /*
* It's empty, there is no such record. * It's empty, there is no such record.
*/ */
......
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