Re: ntdll: Fix the file mapping view leak in the case of module loading failure. Take 2.

[Alexandre Julliard]

Dmitry Timoshkov <dmitry(a)baikal.ru> writes:
> @@ -1548,6 +1549,11 @@ static NTSTATUS load_native_dll( LPCWSTR load_path, LPCWSTR name, HANDLE file,
>      *pwm = wm;
>      status = STATUS_SUCCESS;
>  done:
> +    if (status)
> +    {
> +        if (wm) free_modref( wm );
> +        else if (module) NtUnmapViewOfSection( NtCurrentProcess(), module );
> +    }

I don't see any evidence that you are addressing the issues that caused
us to not do that in the first place.

-- 
Alexandre Julliard
julliard(a)winehq.org