diff options
| author | Joseph Qi <joseph.qi@linux.alibaba.com> | 2020-12-04 22:38:42 +0100 |
|---|---|---|
| committer | Darrick J. Wong <darrick.wong@oracle.com> | 2020-12-09 18:49:38 +0100 |
| commit | 2e984badbcc0f1cf284441c566ca4309fe59ac05 (patch) | |
| tree | 8c6196c52bcabcaa0d34a7623a1bb73618018870 /fs/xfs/scrub/agheader_repair.c | |
| parent | xfs: introduce xfs_validate_stripe_geometry() (diff) | |
| download | linux-2e984badbcc0f1cf284441c566ca4309fe59ac05.tar.xz linux-2e984badbcc0f1cf284441c566ca4309fe59ac05.zip | |
xfs: remove unneeded return value check for *init_cursor()
Since *init_cursor() can always return a valid cursor, the NULL check
in caller is unneeded. So clean them up.
This also keeps the behavior consistent with other callers.
Signed-off-by: Joseph Qi <joseph.qi@linux.alibaba.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Diffstat (limited to 'fs/xfs/scrub/agheader_repair.c')
| -rw-r--r-- | fs/xfs/scrub/agheader_repair.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/xfs/scrub/agheader_repair.c b/fs/xfs/scrub/agheader_repair.c index 401f71579ce6..23690f824ffa 100644 --- a/fs/xfs/scrub/agheader_repair.c +++ b/fs/xfs/scrub/agheader_repair.c @@ -829,8 +829,6 @@ xrep_agi_calc_from_btrees( cur = xfs_inobt_init_cursor(mp, sc->tp, agi_bp, sc->sa.agno, XFS_BTNUM_FINO); - if (error) - goto err; error = xfs_btree_count_blocks(cur, &blocks); if (error) goto err; |