Re: Applying XLR_INFO_MASK correctly when looking at WAL record information - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: Applying XLR_INFO_MASK correctly when looking at WAL record information
Date
Msg-id CAB7nPqQ=b8V64JNRPAHpaHwmayNxxHXxiB1jqNxSONPrvqSDfA@mail.gmail.com
Whole thread Raw
In response to Re: Applying XLR_INFO_MASK correctly when looking at WAL record information  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On Sat, Nov 5, 2016 at 2:29 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Michael Paquier <michael.paquier@gmail.com> writes:
>> I have just been trapped by the fact that in some code paths we look
>> at the RMGR information of a WAL record (record->xl_info) but the
>> filter dedicated to it, ~XLR_INFO_MASK, is not applied. This is
>> harmful now, but this could lead to problems if in the future new
>> record-level flags, similar to XLR_SPECIAL_REL_UPDATE, are added.
>> Disclaimer: I am looking at a patch where a record-level flag makes
>> sense, still this looks like a separate problem.
>
>> What about the patch attached to make things more consistent?
>
> Grepping found a couple of additional places that needed the same
> fix.

Ah, where wasShutdown is assigned. I thought I grepped it as well,
thanks for double-checking.

> Pushed with those additions.

Thanks.
-- 
Michael



pgsql-hackers by date:

Previous
From: Andrew Dunstan
Date:
Subject: Re: btree_gin and btree_gist for enums
Next
From: Peter Eisentraut
Date:
Subject: Re: add more NLS to bin