diff options
author | Darrick J. Wong <darrick.wong@oracle.com> | 2020-05-16 10:19:29 -0700 |
---|---|---|
committer | Darrick J. Wong <darrick.wong@oracle.com> | 2020-05-19 09:40:58 -0700 |
commit | 5fd68bdb5a87c929fff5f7bbd947034368f36f4f (patch) | |
tree | c72e5927a34957bf09dc4b177c334ad9e9570617 /fs/ufs/balloc.c | |
parent | 4b516ff4e772993a99fc9bf36503d23ce5bd5ba9 (diff) | |
download | linux-5fd68bdb5a87c929fff5f7bbd947034368f36f4f.tar.bz2 |
xfs: clean up xchk_bmap_check_rmaps usage of XFS_IFORK_Q
XFS_IFORK_Q is supposed to be a predicate, not a function returning a
value. Its usage is in xchk_bmap_check_rmaps is incorrect, but that
function only cares about whether or not the "size" of the data is zero
or not. Convert that logic to use a proper boolean, and teach the
caller to skip the call entirely if the end result would be that we'd do
nothing anyway. This avoids a crash later in this series.
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
[hch: generalized the NULL ifor check]
Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: Brian Foster <bfoster@redhat.com>
Diffstat (limited to 'fs/ufs/balloc.c')
0 files changed, 0 insertions, 0 deletions