[PATCH v2] dpnet: Implement IDirectPlay8Address GetURLW/A.

Alexandre Julliard julliard at winehq.org
Thu Aug 16 13:27:28 CDT 2018


Alistair Leslie-Hughes <leslie_alistair at hotmail.com> writes:

> +    hr = IDirectPlay8Address_GetURLW(iface, buffer, length);
> +    if(hr == DPN_OK)
> +    {
> +        DWORD size;
> +        size = WideCharToMultiByte(CP_ACP, 0, buffer, -1, NULL, 0, NULL, NULL);
> +        WideCharToMultiByte(CP_ACP, 0, buffer, -1, url, size, NULL, NULL);

This doesn't make sense. The size should be the initial size of the
output buffer passed by the app, and it should be updated to the actual
output size, with a check for overflows.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list