Re: pgsql: Fix valgrind failure

Hello,

On 2026-Apr-07, Bertrand Drouvot wrote:

> - SerializedSnapshotData serialized_snapshot;
> + SerializedSnapshotData serialized_snapshot = {0};
>
> Should we use memset() instead? As "= {0}" is guaranteed to zero padding bytes
> (for object that has automatic storage duration) as of C23 (see Andres's reply
> in [0]) but we require C11 so far.

Hmm ... I'll wait for skink to fail with the fix that was committed,
before making further changes. It may well be that it works okay as it
is now.

> I wonder if we should introduce a new macro say "INITIALIZE_PADDING"
> that I proposed in [1].
>
> [1]: https://postgr.es/m/ablMZvnGhX8h5GgK%40ip-10-97-1-34.eu-west-3.compute.internal

It's not clear to me how that would be used exactly (surely it needs to
be part of the initialization? But what if you have to use memset?), but
the idea looks rather ugly.

--
Álvaro Herrera Breisgau, Deutschland — https://www.EnterpriseDB.com/