Re: Potential null pointer dereference in postgres.c

Karina Litskevich <litskevichkarina(at)gmail(dot)com> writes:
> When backporting 66e94448 to older versions it was forgotten to check
> malloc() result. In 16+ versions guc_malloc() is used to allocate
> memory and it checks if the result pointer is NULL, so there is no
> need to check it after guc_malloc(). Versions before 16 have no
> guc_malloc(), and malloc() is used instead, but we have to check if
> return value is NULL.

Yup, you're right. Thanks for the report!

regards, tom lane