Fix xslt_process() to ensure that it inserts a NULL terminator after the
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 10 Jul 2009 00:32:12 +0000 (00:32 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 10 Jul 2009 00:32:12 +0000 (00:32 +0000)
commit1201008103de91d30c78beb8cb4db1fd80cfc275
tree8e03e13f53b8f909e4ca74d46b38fcf555776d78
parent96ee4c9726e5895e6cae49638471b25f139514df
Fix xslt_process() to ensure that it inserts a NULL terminator after the
last pair of parameter name/value strings, even when there are MAXPARAMS
of them.  Aboriginal bug in contrib/xml2, noted while studying bug #4912
(though I'm not sure whether there's something else involved in that
report).

This might be thought a security issue, since it's a potential backend
crash; but considering that untrustworthy users shouldn't be allowed
to get their hands on xslt_process() anyway, it's probably not worth
getting excited about.
contrib/xml2/xslt_proc.c