Re: BUG #6379: SQL Function Causes Back-end Crash

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Paul Ramsey <pramsey(at)cleverelephant(dot)ca>
Cc: Pgsql Hackers <pgsql-hackers(at)postgresql(dot)org>, pgsql-bugs(at)postgresql(dot)org
Subject: Re: BUG #6379: SQL Function Causes Back-end Crash
Date: 2012-01-04 21:21:37
Message-ID: 22487.1325712097@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-hackers

Paul Ramsey <pramsey(at)cleverelephant(dot)ca> writes:
> Further notes, from Andrew (RhodiumToad) on IRC about the cause of this crasher:
> [12:31pm] RhodiumToad: there's no trivial fix

IMO the main bug here is that functions.c isn't expecting qd->dest to be
overwritten, so we could work around it by keeping a separate private
copy of the dest pointer. However, it would also be fair to ask whether
there's not a cleaner solution. Perhaps the intoRel stuff should be
saving/restoring the original destreceiver instead of just blindly
overwriting it.

regards, tom lane

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message dean.w.schulze 2012-01-04 22:58:20 BUG #6380: dropdb gives error while loading shared libraries: liblber-2.3.so.0
Previous Message Guillaume Smet 2012-01-04 20:45:23 Re: BUG #5741: syslog line length

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2012-01-04 21:25:00 Re: 16-bit page checksums for 9.2
Previous Message Tom Lane 2012-01-04 21:06:45 Re: Regarding Checkpoint Redo Record