Re: mshtml: Added IHTMLTable::width property implementation. (try 5)

[Jacek Caban]

Hi Zhenbo,

This looks better, but there are two more things I didn't catch earlier.

On 04/27/14 15:08, Zhenbo Li wrote:
> +    nsres = nsIDOMHTMLTableElement_GetWidth(This->nstable, &val);
> +    if (NS_FAILED(nsres)){
> +        ERR("Get Width(%s) failed!\n", debugstr_variant(p));

You can't access output argument (p) here. This will be undefined.
> +        nsAString_Finish(&val);
> +        return E_FAIL;
> +    }
> +
> +    hres = nsstr_to_truncated_bstr(&val, &bstr);
> +
> +    V_VT(p) = VT_BSTR;
> +    V_BSTR(p) = bstr;
> +    return hres;

You leak val here.

Jacek