Re: Lack of Sanity Checking in file 'pctcl.c' for PostgreSQL 9.4.x

On Sat, Jun 13, 2015 at 12:41 PM, Tom Lane  wrote:
> Alvaro Herrera  writes:
>> Michael Paquier wrote:
>>> By the way, your patch does not compile properly and is not in-line
>>> with the project's code format. See the updated patch attached ;)
>
>> ... or the conventions for allocating memory.  Why not just use palloc()?
>
> That's hardly the fault of the proposed patch.  But yeah, it seems like
> much the best fix here is to get rid of the malloc (and strdup) calls in
> this code in favor of using the palloc infrastructure.  Even the calls
> that *do* have manual failure checks are not compliant with our usual
> coding standards.

Hm. Regarding the code path mentioned by Bill something like the patch
attached is enough with a memory context for the query description.
Now, perhaps we could do more efforts with prodesc as well, see for
example compile_pltcl_function for pltcl and similarly for plperl.
Thoughts?
-- 
Michael