]> granicus.if.org Git - postgresql/commit
Fix null-dereference crash in parse_xml_decl().
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 28 May 2011 16:36:04 +0000 (12:36 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 28 May 2011 16:36:35 +0000 (12:36 -0400)
commitd6a9e7a1e45a3569b3c3558b5817e78756224be2
treef536f1a6dffe193c201611ab36316c5b7f6ff81a
parentd3f52d72ceb3f5fc47c2b49f3b0ad3258ba6a9af
Fix null-dereference crash in parse_xml_decl().

parse_xml_decl's header comment says you can pass NULL for any unwanted
output parameter, but it failed to honor this contract for the "standalone"
flag.  The only currently-affected caller is xml_recv, so the net effect is
that sending a binary XML value containing a standalone parameter in its
xml declaration would crash the backend.  Per bug #6044 from Christopher
Dillard.

In passing, remove useless initializations of parse_xml_decl's output
parameters in xml_parse.

Back-patch to 8.3, where this code was introduced.
src/backend/utils/adt/xml.c