Alvaro Herrera <alvhe...@alvh.no-ip.org> writes:
> I just noticed that HeapTupleHeaderAdvanceLatestRemovedXid is comparing Xmax
> as a TransactionId without verifying whether it is a multixact or not. Since
> they advance separately, this could lead to bogus answers. This probably
> needs to be fixed. I didn't look into past releases to see if there's a live
> released bug here or not.
> I think the fix is simply to ignore the Xmax if the HEAP_XMAX_IS_MULTI bit is
> set.
> Additionally I think it should check HEAP_XMAX_INVALID before reading the
> Xmax at all.
If it's failing to even check XMAX_INVALID, surely it's completely
broken? Perhaps it assumes its caller has checked all this?
regards, tom lane
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
