Re: revised patch for PL/PgSQL table functions - Mailing list pgsql-patches

From Tom Lane
Subject Re: revised patch for PL/PgSQL table functions
Date
Msg-id 7052.1030753398@sss.pgh.pa.us
Whole thread Raw
In response to Re: revised patch for PL/PgSQL table functions  (Neil Conway <neilc@samurai.com>)
List pgsql-patches
Neil Conway <neilc@samurai.com> writes:
> Tom Lane <tgl@sss.pgh.pa.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

pgsql-patches by date:

Previous
From: Alvaro Herrera
Date:
Subject: CLUSTER all tables
Next
From: Joe Conway
Date:
Subject: Re: revised patch for PL/PgSQL table functions