pgsql: Fix some more bugs in contrib/xml2's xslt_process().

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Fix some more bugs in contrib/xml2's xslt_process().
Date: 2012-06-05 00:13:54
Message-ID: E1SbhPG-0005CF-RE@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Fix some more bugs in contrib/xml2's xslt_process().

It failed to check for error return from xsltApplyStylesheet(), as reported
by Peter Gagarinov. (So far as I can tell, libxslt provides no convenient
way to get a useful error message in failure cases. There might be some
inconvenient way, but considering that this code is deprecated it's hard to
get enthusiastic about putting lots of work into it. So I just made it say
"failed to apply stylesheet", in line with the existing error checks.)

While looking at the code I also noticed that the string returned by
xsltSaveResultToString was never freed, resulting in a session-lifespan
memory leak.

Back-patch to all supported versions.

Branch
------
REL8_3_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/66fb03f53842e58d1025ad04888edc99f04793fa

Modified Files
--------------
contrib/xml2/xslt_proc.c | 14 ++++++++++++++
1 files changed, 14 insertions(+), 0 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2012-06-05 00:44:03 pgsql: Fix bogus handling of control characters in json_lex_string().
Previous Message Christopher Browne 2012-06-04 19:49:37 Re: Re: [COMMITTERS] pgsql: Send new protocol keepalive messages to standby servers.