Small memory leak in execute.c of ECPG driver - Mailing list pgsql-hackers

Hi all,

In exactly 3 places of the ECPG driver (for numeric, for interval and
for date), we do something as follows:
/* Allocation of mallocedval */
if (!(mallocedval = ecpg_strdup("array [", lineno)))
        return false;

for (element = 0; element < var->arrsize; element++)
{
        int                     result;

        ptr = stuff_alloc();
        if (!ptr)
                return false; <= Leak here of mallocedval

It happens that if the allocation done within this for loop fails we
leak mallocedval that was previously allocated. Attached is a patch to
fix this issue spotted by Coverity.
Regards
--
Michael