• Darrick J. Wong's avatar
    xfs: remove redundant ASSERT on insufficient bestfree length in _leaf_addname · b3986010
    Darrick J. Wong authored
    In xfs_dir2_leaf_addname we ASSERT if the length of the unused space
    described by bestfree[0] is less the amount of space we wish to consume.
    Immediately after it is a call to xfs_dir2_data_use_free where the
    offset parameter is offset of the unused space and the length parameter
    is the amount of space we wish to consume.  Both values (and the unused
    space pointer) are passed into xfs_dir2_data_check_free, which also
    validates that the region of unused space is big enough to cover the
    space we wish to consume.  This is effectively the same check that the
    ASSERT covers, and since a check failure results in a corruption message
    being logged we can remove the ASSERT.
    Signed-off-by: default avatarDarrick J. Wong <darrick.wong@oracle.com>
    Reviewed-by: default avatarDave Chinner <dchinner@redhat.com>
    b3986010
xfs_dir2_leaf.c 50.5 KB