Re: [PATCH v4 0/2] MR9600: msxml: Add support for user-defined functions in msxsl:script blocks.

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/msxml3/node.c:
> - if (FAILED(hr))
> -
>    WARN("User script Invoke() failed %#lx, function %s.\n", hr, xpath->function);
> -
> - if (args)
> - {
> -
>    for (int i = 0; i < nargs; ++i)
> -
>        VariantClear(&args[i]);
> -
>    free(args);
> - }
> -
> - switch (V_VT(&result))
> - {
> -
>    case VT_BSTR:
> -
>    {
> -
>        xmlChar *s = xmlchar_from_wchar(V_BSTR(&result));
> -
>        xmlXPathReturnString(ctxt, s);

I think we should use `xmlMalloc` for the string here.