• Tom Lane's avatar
    In PageIndexTupleDelete, don't assume stored item lengths are MAXALIGNed. · 984d0a14
    Tom Lane authored
    PageAddItem stores the item length as-is.  It MAXALIGN's the amount of
    space actually allocated for each tuple, but not the stored length.
    PageRepairFragmentation, PageIndexMultiDelete, and PageIndexDeleteNoCompact
    are all on board with this and MAXALIGN item lengths after fetching them.
    But PageIndexTupleDelete expects the stored length to be a MAXALIGN
    multiple already.  This accidentally works for existing index AMs because
    they all maxalign their tuple sizes internally; but we don't do that for
    heap tuples, and it shouldn't be a requirement for index tuples either.
    
    So, sync PageIndexTupleDelete with the rest of bufpage.c by having it
    maxalign the item size after fetching.
    
    Also add a check that pd_special is maxaligned, to ensure that the test
    "(offset + size) > phdr->pd_special" is still doing the right thing.
    (If offset and pd_special are aligned, it doesn't matter whether size is.)
    Again, this is in sync with the rest of the routines here, except for
    PageAddItem which doesn't test because it doesn't actually do anything
    for which pd_special alignment matters.
    
    This shouldn't have any immediate functional impact; it just adds the
    flexibility to use PageIndexTupleDelete on index tuples with non-aligned
    lengths.
    
    Discussion: <3814.1473366762@sss.pgh.pa.us>
    984d0a14
bufpage.c 30.8 KB