ShuriZma
/
pcsx2
mirror of https://github.com/PCSX2/pcsx2.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

3rdparty/pthreads4w: Fixup process exit logic. (#3056) 

This PR modifies a third party module - I realize normally those should be fixed upstream, but I really doubt pthreads4w would a) consider it a valid bug and b) fix it. To make all my changes easily visible, I wrap them all in #if PCSX2_FIX.

This PR fixes a process exit routine in pthreads. This third party module exploits CRT initialization order to inject their initializer/deinitializer earlier than the others by putting their functions in .CRT$XCU and .CRT$XPU pseudo regions. The problem comes when a module gets build with dynamic CRT (/MD or /MDd), like most of PCSX2 plugins, it doesn't actually use .CRT$XPx regions as terminators, and instead lets dynamic CRT handle them.

This PR corrects this issue by registering the terminator via atexit, so it works with both static and dynamic CRT. This resolves an issue where SPU2-X plugin (and potentially more) leaks TLS handles when unloaded.
This commit is contained in:
Silent 2019-08-13 18:32:41 +02:00 committed by lightningterror
parent ed6ac00186
commit af6f040202
1 changed files with 14 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
3rdparty/pthreads4w/autostatic.c vendored
View File

@ -36,15 +36,19 @@
#if defined(PTW32_STATIC_LIB)
#define PCSX2_FIX 1 // Comment out to restore code to pristine state
#if defined(__MINGW64__) || defined(__MINGW32__) || defined(_MSC_VER)
#include "pthread.h"
#include "implement.h"
#if !PCSX2_FIX
static void on_process_init(void)
{
pthread_win32_process_attach_np ();
}
#endif
static void on_process_exit(void)
{
@ -52,6 +56,14 @@ static void on_process_exit(void)
pthread_win32_process_detach_np ();
}
#if PCSX2_FIX
static void on_process_init(void)
{
pthread_win32_process_attach_np();
atexit(on_process_exit);
}
#endif
#if defined(__MINGW64__) || defined(__MINGW32__)
# define attribute_section(a) __attribute__((section(a)))
#elif defined(_MSC_VER)
@ -62,7 +74,9 @@ attribute_section(".ctors") void *gcc_ctor = on_process_init;
attribute_section(".dtors") void *gcc_dtor = on_process_exit;
attribute_section(".CRT$XCU") void *msc_ctor = on_process_init;
#if !PCSX2_FIX
attribute_section(".CRT$XPU") void *msc_dtor = on_process_exit;
#endif
#endif /* defined(__MINGW64__) || defined(__MINGW32__) || defined(_MSC_VER) */