Where to clean up stuff in injected DLL

[iconic]

EasY,

DLL_Xxx events are executed within APCs, also have you factored in potentially odd TLS callback behavior?

--Iconic

[madshi]

I've now changed the code so that only the first uninjection thread executes AutoUnhook. If there are more uninjection threads, in the moment when they try to call AutoUnhook, they're simply closed immediately with ExitThread. This way they can't even try to actually call FreeLibrary. Here's a beta build with this change:

http://madshi.net/madCollectionBeta.exe (2.7.4.4)

Can you confirm that the problem is solved with this build?

[princezna]

Yes, I can confirm this solved the issue. We haven't been able to reproduce it with the beta release in the previous post.

Thanks.

[madshi]

Good to hear, thanks for the feedback.