Re: [PATCH try5] wininet: Added support for persistent HTTP connections

[Alexandre Julliard]

Jacek Caban <jacek(a)codeweavers.com> writes:
> +    FreeLibrary(WININET_hModule);
> +    return ERROR_SUCCESS;

It can still crash here.
> +static void delayed_collect_connections(void)
> +{
> +    static const WCHAR wininet_dllW[] = {'w','i','n','i','n','e','t','.','d','l','l',0};
> +
> +    if(collector_running)
> +        return;
> +
> +    if(!LoadLibraryW(wininet_dllW))
> +        return;

You probably don't want that inside the lock. Also GetModuleHandleEx is
probably better.

-- 
Alexandre Julliard
julliard(a)winehq.org