BUG #4912: server core dumps on xslt_process with empty param string

The following bug has been logged online:

Bug reference: 4912
Logged by: Tom Bowden
Email address: tom(dot)bowden(at)fis-cal(dot)com
PostgreSQL version: 8.4rc2
Operating system: Linux 2.6.22.5-31-bigsmp
Description: server core dumps on xslt_process with empty param
string
Details:

I found that when using xslt_process(), either with two parameters, or with
an empty/null third parameter -- that I get a 'server restarted' message.

To stop this behavior, I changed the source code in contrib/xml2/xslt_proc.c
(around line 142) .. adding the following:
if ( pstr == NULL || strlen(pstr) < 1 ) {
params[0] = NULL;
return;
}

I don't know if this is the right place to report this; whether there is a
better way (or not) to fix it,(add any other disclaimers you might want to
add - or tell me where you want me to go with this :).

Tom