Re: [HACKERS] Re: Potential pointer dereference in plperl.c (caused by transforms patch)

2015-12-16 Thread Alvaro Herrera
Noah Misch wrote: > fcinfo->flinfo->fn_oid==InvalidOid implies an inline block, and those have no > arguments. If it placates Coverity, I lean toward an assert-only change: > > --- a/src/pl/plperl/plperl.c > +++ b/src/pl/plperl/plperl.c This was committed as d4b686af0b. -- Álvaro Herrera

[HACKERS] Re: Potential pointer dereference in plperl.c (caused by transforms patch)

2015-11-27 Thread Noah Misch
On Mon, May 04, 2015 at 02:02:18PM +0900, Michael Paquier wrote: > Coverity is pointing out that as argtypes = NULL in > plperl_call_perl_func@plperl.c, we will have a pointer dereference if > desc->arg_arraytype[i] is not a valid OID, see here: > + Oid*argtypes = NULL; > [...