Use the correct sizeof() in BufFileLoadBuffer
authorTomas Vondra <tomas.vondra@postgresql.org>
Fri, 24 Jan 2025 23:36:48 +0000 (00:36 +0100)
committerTomas Vondra <tomas.vondra@postgresql.org>
Sat, 25 Jan 2025 01:16:10 +0000 (02:16 +0100)
commit8cedf6a1b9fcbd33f97413b6a37b38d936522624
tree3148e9a207d08d3904c52e9e047116dd3604d559
parent4d6e169c43418d3efed47356d7d004cf12e91c82
Use the correct sizeof() in BufFileLoadBuffer

The sizeof() call should reference buffer.data, because that's the
buffer we're reading data into, not the whole PGAlignedBuffer union.
This was introduced by 44cac93464, which replaced the simple buffer
with a PGAlignedBuffer field.

It's benign, because the buffer is the largest field of the union, so
the sizes are the same. But it's easy to trip over this in a patch, so
fix and backpatch. Commit 44cac93464 went into 12, but that's EOL.

Backpatch-through: 13
Discussion: https://postgr.es/m/928bdab1-6567-449f-98c4-339cd2203b87@vondra.me
src/backend/storage/file/buffile.c