Re: revised patch for PL/PgSQL table functions

Neil Conway <neilc(at)samurai(dot)com> writes:
> Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> writes:
>> There is a rather nasty bug left (Sir Mordred would likely call it a
>> DOS possibility ;-)) --- RETURN NEXT doesn't seem to be checking that
>> the row or record variable it is given actually matches the declared
>> return type of the plpgsql function.

> Yes, I probably should have mentioned that.

I've applied a fix for this.

The fix actually uses the "expected tuple desc" that's now passed by
ExecMakeTableFunctionResult as the target descriptor. This should mean
that it'd be possible to support plpgsql functions returning RECORD, but
I didn't have time to look into that. Anyone want to try?

regards, tom lane