Stephen Frost <sfrost@snowman.net> writes:
> * Xi Wang (xi.wang@gmail.com) wrote:
>> The correct NULL check should use `*newval'; `newval' must be non-null.
> Why isn't this using pstrdup()..?
The GUC API uses malloc, mainly because guc.c can't afford to lose
control on out-of-memory situations.
regards, tom lane