From: | Jacob Champion <jacob(dot)champion(at)enterprisedb(dot)com> |
---|---|
To: | Heikki Linnakangas <hlinnaka(at)iki(dot)fi> |
Cc: | Peter Eisentraut <peter(at)eisentraut(dot)org>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: pgsql: Make cancel request keys longer |
Date: | 2025-05-08 22:28:24 |
Message-ID: | CAOYmi+kJcKkbe3Ng7=jYOz7V2gYEvdNXKESKbmuW=6N7=243tg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On Thu, May 8, 2025 at 12:11 PM Heikki Linnakangas <hlinnaka(at)iki(dot)fi> wrote:
> Polished this up a tiny bit, and committed.
Thanks! I think the uint8->int change for cancel_key_len is more than
just cosmetic; it most likely fixes a bug where a key size of 256
wrapped around to 0. I'll double-check that this fixes that later;
I've gotten side-tracked from the protocol stuff a bit.
While I have you, though, is the following just a really complicated
way to say `msgLength - 4`, or is there some other reason to do the
pointer math?
cancel_key_len = 5 + msgLength - (conn->inCursor - conn->inStart);
--Jacob
From | Date | Subject | |
---|---|---|---|
Next Message | Heikki Linnakangas | 2025-05-09 06:37:58 | Re: pgsql: Make cancel request keys longer |
Previous Message | Heikki Linnakangas | 2025-05-08 19:11:07 | Re: pgsql: Make cancel request keys longer |
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2025-05-08 23:19:39 | Re: Support for runtime parameters in injection points, for AIO tests |
Previous Message | Jacob Champion | 2025-05-08 22:06:34 | Re: [PATCH] Fix missing comma in Requires.private with a Make macro |