Re: [PATCH] Dereference null return value (NULL_RETURNS) (src/backend/commands/statscmds.c) - Mailing list pgsql-hackers

From Ranier Vilela
Subject Re: [PATCH] Dereference null return value (NULL_RETURNS) (src/backend/commands/statscmds.c)
Date
Msg-id CAEudQAqBTMUAc0scw-r1xp4trAAeMY3Z4V8gufgNReZ+=+0PXg@mail.gmail.com
Whole thread Raw
In response to [PATCH] Dereference null return value (NULL_RETURNS) (src/backend/commands/statscmds.c)  (Ranier Vilela <ranier.vf@gmail.com>)
List pgsql-hackers
Em ter., 25 de ago. de 2020 às 12:42, Ranier Vilela <ranier.vf@gmail.com> escreveu:
Hi Tom,

Per Coverity.

The SearchSysCache1 allows return NULL and at function AlterStatistics,
has one mistake, lack of, check of return, which enables a dereference NULL pointer,
at function heap_modify_tuple.

While there is room for improvement.
Avoid calling SearchSysCache1 and table_open if the user "is not the owner of the existing statistics object".
After a long time, finally this bug has been fixed.

regards,
Ranier Vilela

pgsql-hackers by date:

Previous
From: Andres Freund
Date:
Subject: Re: Adding CI to our tree
Next
From: Andres Freund
Date:
Subject: Re: Fix overflow in DecodeInterval