Re: [PATCH] Dereference null return value (NULL_RETURNS) (src/backend/commands/statscmds.c)

From: Ranier Vilela <ranier(dot)vf(at)gmail(dot)com>
To: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: [PATCH] Dereference null return value (NULL_RETURNS) (src/backend/commands/statscmds.c)
Date: 2022-02-13 20:26:59
Message-ID: CAEudQAqBTMUAc0scw-r1xp4trAAeMY3Z4V8gufgNReZ+=+0PXg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Em ter., 25 de ago. de 2020 às 12:42, Ranier Vilela <ranier(dot)vf(at)gmail(dot)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.
https://github.com/postgres/postgres/commit/6d554e3fcd6fb8be2dbcbd3521e2947ed7a552cb

regards,
Ranier Vilela

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2022-02-13 20:30:37 Re: Fix overflow in DecodeInterval
Previous Message Andres Freund 2022-02-13 20:24:03 Re: Adding CI to our tree