Re: Unsafe access BufferDescriptors array in BufferGetLSNAtomic()

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: Richard Guo <guofenglinux(at)gmail(dot)com>
Cc: Tender Wang <tndrwang(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Unsafe access BufferDescriptors array in BufferGetLSNAtomic()
Date: 2025-02-19 00:44:34
Message-ID: Z7Upcpl-JOqwCoM6@paquier.xyz
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Feb 19, 2025 at 09:36:35AM +0900, Richard Guo wrote:
> I plan to push this shortly. This is arguably a bug, but it hasn't
> caused any real issues, and nobody has complained about it until now,
> so I don't think it needs to be back-patched. Any thoughts?

I can see a point in doing a backpatch to keep the code consistent
across branches, and this avoids incorrect access patterns when
copy-pasting this code.
--
Michael

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2025-02-19 00:53:09 Re: test_escape: invalid option -- 'c'
Previous Message Melanie Plageman 2025-02-19 00:43:42 Re: Parallel heap vacuum