Re: [PATCH] mshtml: Make sure actual host length matches returned BSTR length

[Jacek Caban]

Hi Nikolay,

On 21.11.2017 21:17, Nikolay Sivov wrote:
> The issue our jscript gets confused by embedded NULs, something like
> var a = "prefix://" + document.location.host + "suffix";
>
> can give "prefix://host:80\0\0\0suffix".

This should be easy to add a test for it, something like:
ok(SysStringLen(str) == strlen(test->host), "...");
in test_host should catch the problem.

Ideally, the implementation would use IUri instead of InternetCrackUrl,
but the patch looks good to me as well.

Thanks,
Jacek