Re: [2/2] dpnet: return a TCP/IP provider in IDirectPlay8PeerImpl_EnumServiceProviders

[Alexandre Julliard]

Louis Lenders <xerox_xerox2000(a)yahoo.co.uk> writes:
> +    info = HeapAlloc(GetProcessHeap(), 0, req_size);
> +
> +    info->pwszName = (LPWSTR)(info + 1);
> +
> +    if(!pguidServiceProvider)
> +    { 
> +        lstrcpyW(info->pwszName, dp_providerW);
> +        info->guid = CLSID_DP8SP_TCPIP;
> +    }
> +    else
> +    {
> +        lstrcpyW(info->pwszName, dp_adapterW);
> +        info->guid = adapter_guid;
> +    }
> +
> +    *pSPInfoBuffer = *info;
> +    *pcReturned = 1;
> +
> +    HeapFree(GetProcessHeap(), 0, info);

This doesn't make sense, you are returning freed data.

-- 
Alexandre Julliard
julliard(a)winehq.org