Re: Re: PROPOSAL: make PostgreSQL sanitizers-friendly (and prevent information disclosure) - Mailing list pgsql-hackers

From Chapman Flack
Subject Re: Re: PROPOSAL: make PostgreSQL sanitizers-friendly (and prevent information disclosure)
Date
Msg-id 56F006C4.2030409@anastigmatix.net
Whole thread Raw
In response to Re: Re: PROPOSAL: make PostgreSQL sanitizers-friendly (and prevent information disclosure)  (Aleksander Alekseev <a.alekseev@postgrespro.ru>)
Responses Re: Re: PROPOSAL: make PostgreSQL sanitizers-friendly (and prevent information disclosure)
Re: Re: PROPOSAL: make PostgreSQL sanitizers-friendly (and prevent information disclosure)
List pgsql-hackers
On 03/21/2016 10:21 AM, Aleksander Alekseev wrote:

> Well in this case here is a patch that fixes "use of uninitialized
> value" reports by MemorySanitizer I managed to catch so far.

I'm new here so someone more experienced would have to weigh in,
but I would wonder a couple of things:

a. whether a braced struct assignment is supported in every  C compiler that PostgreSQL still intends to support

b. whether such a struct assignment is guaranteed to initialize  padding spaces as well as declared fields (in all
supported C versions/compilers).
 

It's possible that memset() would be more convincing.

-Chap




pgsql-hackers by date:

Previous
From: David Fetter
Date:
Subject: Re: [COMMITTERS] pgsql: Support parallel aggregation.
Next
From: Robert Haas
Date:
Subject: Re: [COMMITTERS] pgsql: Support parallel aggregation.