Re: Fix possible bogus array out of bonds (src/backend/access/brin/brin_minmax_multi.c)
| От | Ranier Vilela |
|---|---|
| Тема | Re: Fix possible bogus array out of bonds (src/backend/access/brin/brin_minmax_multi.c) |
| Дата | |
| Msg-id | CAEudQApjZ5FPobhssXj2=+8TPK-BnWqiSvdj4sW3sUdKHc1sBg@mail.gmail.com обсуждение исходный текст |
| Ответ на | Re: Fix possible bogus array out of bonds (src/backend/access/brin/brin_minmax_multi.c) (David Rowley <dgrowleyml@gmail.com>) |
| Ответы |
Re: Fix possible bogus array out of bonds (src/backend/access/brin/brin_minmax_multi.c)
|
| Список | pgsql-hackers |
Em seg., 5 de set. de 2022 às 22:29, David Rowley <dgrowleyml@gmail.com> escreveu:
On Tue, 6 Sept 2022 at 06:07, Ranier Vilela <ranier.vf@gmail.com> wrote:
> I did a search and found a few more places.
> v1 attached.
Thanks. I've done a bit more looking and found a few more places that
we can improve and I've pushed the result.
Thanks.
It feels like it would be good if we had a way to detect a few of
these issues much earlier than we are currently. There's been a long
series of commits fixing up this sort of thing. If we had a tool to
parse the .c files and look for things like a function call to
appendPQExpBuffer() and appendStringInfo() with only 2 parameters (i.e
no va_arg arguments).
StaticAssert could check va_arg no?
regards,
Ranier Vilela
В списке pgsql-hackers по дате отправления: