Fixed: Potential bug in CWinThread, someone using a mutex on the stack again

code/nel/src/misc/win_thread.cpp

@@ -73,6 +73,19 @@ CWinThread::CWinThread (IRunnable *runnable, uint32 stackSize)
 	_MainThread = false;
 }
 
+namespace {
+class CWinCriticalSection
+{
+private:
+	CRITICAL_SECTION cs;
+public:
+	CWinCriticalSection() { InitializeCriticalSection(&cs); }
+	~CWinCriticalSection() { DeleteCriticalSection(&cs); }
+	inline void enter() { EnterCriticalSection(&cs); }
+	inline void leave() { LeaveCriticalSection(&cs); }
+};
+}/* anonymous namespace */
+
 CWinThread::CWinThread (void* threadHandle, uint32 threadId)
 {
 	// Main thread
@@ -99,14 +112,12 @@ CWinThread::CWinThread (void* threadHandle, uint32 threadId)
 		nlassert(0); // WARNING: following code has not tested! don't know if it work fo real ...
 					 // This is just a suggestion of a possible solution, should this situation one day occur ...
 		// Ensure that this thread don't get deleted, or we could suspend the main thread
-		CRITICAL_SECTION cs;
+		static CWinCriticalSection cs;
-		InitializeCriticalSection(&cs);
+		cs.enter();
-		EnterCriticalSection(&cs);
 		// the 2 following statement must be executed atomicaly among the threads of the current process !
 		SuspendThread(threadHandle);
 		_SuspendCount = ResumeThread(threadHandle);
-		LeaveCriticalSection(&cs);
+		cs.leave();
-		DeleteCriticalSection(&cs);
 	}
 }