pgsql: Code review for spi_query/spi_fetchrow patch: handle errors

From: tgl(at)svr1(dot)postgresql(dot)org (Tom Lane)
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Code review for spi_query/spi_fetchrow patch: handle errors
Date: 2005-10-18 17:13:14
Message-ID: 20051018171314.60F25D94DD@svr1.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Log Message:
-----------
Code review for spi_query/spi_fetchrow patch: handle errors sanely,
avoid leaking memory. I would add a regression test for error handling
except it seems eval{} can't be used in unprivileged plperl :-(

Modified Files:
--------------
pgsql/src/pl/plperl:
SPI.xs (r1.16 -> r1.17)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/pl/plperl/SPI.xs.diff?r1=1.16&r2=1.17)
plperl.c (r1.93 -> r1.94)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/pl/plperl/plperl.c.diff?r1=1.93&r2=1.94)

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message David Fetter 2005-10-18 18:28:16 Re: pgsql: Code review for spi_query/spi_fetchrow patch: handle errors
Previous Message User Doudou586 2005-10-18 16:05:55 press - pr: version 1.0