On Fri, Nov 23, 2018 at 7:21 AM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com> writes:
> > Accessing of NULL pointer leads to crash. Attached quick simple patch
> fixes
> > the problem.
>
> While that stops the crash, it doesn't do what we want, which is to
> return a valid tupdesc. Without that you get "set-valued function called
> in context that cannot accept a set" errors. I think the right fix is
> to force a call of update_cached_tupdesc so that there always is a
> tupdesc to copy.
>
Thanks for checking, I thought of the same thing, but I wasn't sure.
Regards,
Haribabu Kommi
Fujitsu Australia