Re: pgsql: Make cancel request keys longer

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

In response to

Responses

Browse pgsql-committers by date

  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

Browse pgsql-hackers by date

  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