Re: Fix Potential Deadlock in LdrLoadDll

[Alexandre Julliard]

Robert Shearman  writes:
> This patch fixes a potential deadlock in LdrLoadDll as it tries to call
> LdrUnloadDll with the loader lock held, resulting in deadlock once
> LdrUnloadDll also tries to enter the loader critical section.
> This only happens on process_attach failure.

This cannot deadlock, a critical section is reentrant. There must be
some other issue here.

Fix Potential Deadlock in LdrLoadDll

[Robert Shearman]

Alexandre Julliard wrote:
> Robert Shearman  writes:
> > This patch fixes a potential deadlock in LdrLoadDll as it tries to call
> > LdrUnloadDll with the loader lock held, resulting in deadlock once
> > LdrUnloadDll also tries to enter the loader critical section.
> > This only happens on process_attach failure.
>
> This cannot deadlock, a critical section is reentrant. There must be
> some other issue here.

Yes, never mind. Back to Win32 Syncronization Primitives 101 for me.
> -- 
> Alexandre Julliard
> [e-mail]

Rob