Revise memory management for libxml calls. Instead of keeping libxml's data
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 15 Jan 2008 18:57:00 +0000 (18:57 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 15 Jan 2008 18:57:00 +0000 (18:57 +0000)
commit27998dda4b23237432b9e225e11a401c187119c5
tree338f8130e4aa0674e542b73ba7189f0773accfe2
parentf997a326b053a686996429d9d63ca606f30b48fc
Revise memory management for libxml calls.  Instead of keeping libxml's data
in whichever context happens to be current during a call of an xml.c function,
use a dedicated context that will not go away until we explicitly delete it
(which we do at transaction end or subtransaction abort).  This makes recovery
after an error much simpler --- we don't have to individually delete the data
structures created by libxml.  Also, we need to initialize and cleanup libxml
only once per transaction (if there's no error) instead of once per function
call, so it should be a bit faster.  We'll need to keep an eye out for
intra-transaction memory leaks, though.  Alvaro and Tom.
src/backend/access/transam/xact.c
src/backend/utils/adt/xml.c
src/include/utils/xml.h