On Thu, Nov 07, 2019 at 11:05:04PM -0800, Darrick J. Wong wrote: > From: Darrick J. Wong <darrick.wong@xxxxxxxxxx> > > Coverity points out that xfs_btree_islastblock calls > xfs_btree_check_block, but doesn't act on an error return. This > predicate has no answer if the btree is corrupt, so tweak the helper to > be able to return errors, and then modify the one call site. Could we just kill xfs_btree_islastblock? It has pretty trivial, and only has a single caller which only uses on of the two branches in the function anyway.