pgsql: Fix xmlelement() to initialize libxml correctly before using it,

From: tgl(at)postgresql(dot)org (Tom Lane)
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Fix xmlelement() to initialize libxml correctly before using it,
Date: 2007-11-05 22:23:07
Message-ID: 20071105222307.B6448754229@cvs.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Log Message:
-----------
Fix xmlelement() to initialize libxml correctly before using it, and to avoid
assuming that evaluation of its input expressions won't change the state of
libxml. This requires refactoring xml_init() to not call xmlInitParser(),
since now not all of its callers want that. I also tweaked things to avoid
repeated execution of one-time-only tests inside xml_init(), though this is
mostly for clarity rather than in hopes of saving any noticeable amount of
runtime. Per report from Sheikh Amjad and subsequent discussion.
In passing, fix a couple of inadequately schema-qualified queries.

Modified Files:
--------------
pgsql/src/backend/utils/adt:
xml.c (r1.49 -> r1.50)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/xml.c?r1=1.49&r2=1.50)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2007-11-06 03:06:28 pgsql: Some code review for xml.c: Add some more xml_init() calls that
Previous Message User Fxjr 2007-11-05 21:49:35 npgsql - Npgsql2: Added initial information about System.Transactions