Re: execl() sentinel

From: Peter Eisentraut <peter_e(at)gmx(dot)net>
To: pgsql-patches(at)postgresql(dot)org
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Bruce Momjian <bruce(at)momjian(dot)us>, Alvaro Herrera <alvherre(at)commandprompt(dot)com>
Subject: Re: execl() sentinel
Date: 2007-07-18 06:27:16
Message-ID: 200707180827.18123.peter_e@gmx.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

Tom Lane wrote:
> It's too bad that gcc doesn't have a
> -Wno-snarkiness-about-system-headers-thank-you switch.

It does have a switch to *add* snarkiness about system headers, but does
not do it by default.

The problem in this case is that an uncast null pointer constant is not
always a sufficient sentinel for variadic functions, as explained here:
<http://c-faq.com/null/null2.html>.

--
Peter Eisentraut
http://developer.postgresql.org/~petere/

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Peter Eisentraut 2007-07-18 10:12:02 Re: Async Commit, v21 (now: v22)
Previous Message ITAGAKI Takahiro 2007-07-18 04:18:50 Re: docfix - DELETE doesn't affect auto-analyze