[PATCH] Add STRICT to some regression test C functions.

Started by Andreas Seltenreichover 10 years ago3 messageshackers
Jump to latest
#1Andreas Seltenreich
seltenreich@gmx.de

Hi,

some of the C functions in the regression test DB readily crash when
passing NULL input values. The regression tests themselves do not pass
NULL values to them, but when the regression database is used as a basis
for fuzz testing, they cause a lot of noise. Maybe someone can sneak
this patch in?

Thanks,
Andreas

Attachments:

0001-Add-STRICT-to-some-regression-test-C-functio.patchtext/x-diffDownload+10-11
#2Piotr Stefaniak
postgres@piotr-stefaniak.me
In reply to: Andreas Seltenreich (#1)
Re: [PATCH] Add STRICT to some regression test C functions.

On 01/09/2016 12:05 AM, Andreas Seltenreich wrote:

Maybe someone can sneak this patch in?

Exactly this is done by a part of another patch, by Michael Paquier,
that he sent to an off-list thread.

Michael had asked for feedback there, but I didn't have the time to test
the patch. Just from reading it I think it's good, though. And it still
applies and passes HEAD's make check-world tests.

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

#3Tom Lane
tgl@sss.pgh.pa.us
In reply to: Andreas Seltenreich (#1)
Re: [PATCH] Add STRICT to some regression test C functions.

Andreas Seltenreich <seltenreich@gmx.de> writes:

some of the C functions in the regression test DB readily crash when
passing NULL input values. The regression tests themselves do not pass
NULL values to them, but when the regression database is used as a basis
for fuzz testing, they cause a lot of noise. Maybe someone can sneak
this patch in?

Pushed, thanks.

regards, tom lane

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers