Re: Don't pass NULL pointer to strcmp(). - Mailing list pgsql-hackers

From Xing Guo
Subject Re: Don't pass NULL pointer to strcmp().
Date
Msg-id CACpMh+AW5dEE7LoCBkpYAKX+tg4NJE8PhkLP1_nJEjyjEJ8Scg@mail.gmail.com
Whole thread Raw
In response to Re: Don't pass NULL pointer to strcmp().  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
Thank you Tom!

Your comment
"NULL doesn't have semantics that are visibly different from an empty
string" is exactly what I want to confirm :-)

On 11/2/23, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> I wrote:
>> Hmm ... if we're doing it ourselves, I suppose we've got to consider
>> it supported :-(.  But I'm still wondering how many seldom-used
>> code paths didn't get the message.  An example here is that this
>> could lead to GetConfigOptionResetString returning NULL, which
>> I think is outside its admittedly-vague API spec.
>
> After digging around for a bit, I think part of the problem is a lack
> of a clearly defined spec for what should happen with NULL string GUCs.
> In the attached v3, I attempted to remedy that by adding a comment in
> guc_tables.h (which is maybe not the best place but I didn't see a
> better one).  That led me to a couple more changes beyond what you had.
>
> It's possible that some of these are unreachable --- for example,
> given that a NULL could only be the default value, I'm not sure that
> the fix in write_one_nondefault_variable is a live bug.  But we ought
> to code all this stuff defensively, and most of it already was
> NULL-safe.
>
>             regards, tom lane
>
>


-- 
Best Regards,
Xing



pgsql-hackers by date:

Previous
From: Kyotaro Horiguchi
Date:
Subject: Re: Requiring recovery.signal or standby.signal when recovering with a backup_label
Next
From: Bruce Momjian
Date:
Subject: Re: Why is DEFAULT_FDW_TUPLE_COST so insanely low?