Karina Litskevich <litskevichkar...@gmail.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