Crash when re-initializing as static library

[Klaus Fischer <klaus dot fischer at tara-systems dot de>]

Dear pthreads-win32 developers,

I have experienced a crash when building pthreads-win32 as static 
library and re-initializing it using the following sequence:

- pthread_win32_process_attach_np()
- pthread_win32_process_detach_np()
- pthread_win32_process_attach_np()

The global variable ptw32_threadReuseTop still points to memory used 
between the first attach/detach run, but this memory was already freed 
in function ptw32_processTerminate(), which was called during detaching.

When using e.g. pthread_self() afterwards, the global 
ptw32_threadReuseTop now points to invalid memory, causing an access 
violation writing to that memory location.

A simple code change fixed that problem by assigning the default value 
PTW32_THREAD_REUSE_EMPTY to that global variable at the end of function 
ptw32_processTerminate(), after the while loop freeing all still 
allocated thread handles.

A better way to fix that would probably be to initialize all the library 
globals of global.c during the attaching stage. In a static library, 
those globals are only initialized once when the process starts, but 
_should_ be re-initialized on every attach.

I know this is not a concern when using this library as dynamic library, 
but since there is an option to use it as static library and other 
people also use it that way according to the mailing list, it would be 
great if it could survive multiple re-initializations.

Thanks in advance,
Klaus

--
_________________________________________________________

Dipl. Inf. Klaus FISCHER  -  Software Engineer

    mailto: klaus.fischer@tara-systems.de
_________________________________________________________