From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Andreas Seltenreich <seltenreich(at)gmx(dot)de> |
Cc: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [sqlsmith] crashes in RestoreSnapshot on hot standby |
Date: | 2016-07-01 13:04:32 |
Message-ID: | CA+TgmoZM-vbKzhz--kigdvv1FKx5ZQzq-LCjW5iXdHE3o58ysw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, Jul 1, 2016 at 1:41 AM, Andreas Seltenreich <seltenreich(at)gmx(dot)de> wrote:
> Amit Kapila writes:
>> On Fri, Jul 1, 2016 at 9:38 AM, Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com> wrote:
>>> Or maybe just like this?
>>>
>>> - snapshot->subxip = snapshot->xip + serialized_snapshot->xcnt;
>>> + snapshot->subxip = ((TransactionId *) (snapshot + 1)) +
>>> + serialized_snapshot->xcnt;
>>>
>>
>> This way it looks better to me. Thanks for the patch.
>
> I no longer see these crashes when testing with the patch applied.
Committed and back-patched to 9.5 where this code was added. Thanks
to all for the report, patch, review, and testing.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2016-07-01 13:18:14 | Re: Broken handling of lwlocknames.h |
Previous Message | Robert Haas | 2016-07-01 12:17:08 | Re: [COMMITTERS] pgsql: Avoid extra locks in GetSnapshotData if old_snapshot_threshold < |