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

Поиск
Список
Период
Сортировка
От Tom Lane
Тема pgsql: Fix some more bugs in contrib/xml2's xslt_process().
Дата
Msg-id E1SbhPG-0005C7-NI@gemulon.postgresql.org
обсуждение исходный текст
Список 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
------
REL9_1_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/d9d721472f9900e70124ab734dc00f203d049506

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


В списке pgsql-committers по дате отправления:

Предыдущее
От: karl@pgfoundry.org (User Karl)
Дата:
Сообщение: pgtcl - libpgtcl: Change dbinfo option "pass" to "password".
Следующее
От: Tom Lane
Дата:
Сообщение: pgsql: Fix some more bugs in contrib/xml2's xslt_process().