Go Premium for a chance to win a PS4. Enter to Win

x
?
Solved

Thread nerver returns from WaitForSingleObject() !!!

Posted on 2000-03-22
2
Medium Priority
?
255 Views
Last Modified: 2013-12-03
I've a Mutex and 3 threads like below

HANDLE HMutex;  
bool   End = false;
//////
/////
void Thread_Running_1()
{
  do
  {
     WaitForSingleObejct( HMutex, INFINITE );
     // ..... do something important
  }
  while( ! End );
}  
//////
/////
void Thread_Running_2()
{
  do
  {
     WaitForSingleObejct( HMutex, INFINITE );
     // ..... do something important
  }
  while( ! End );
}  
//////
//////   The main() function is only a case school
//////
void main()
{
   // I create and lock the mutex

   HMutex = CreateMutex( ..... );
   WaitForSingleObejct( HMutex, INFINITE );

   // I create 2 threads and i start these threads

   CreateThread( ...., Thread_Running_1, ..... );
   CreateThread( ...., Thread_Running_2, ..... );

   // At this time, Thread 1 and 2 are in a wait state.
   // Now, We release and close the Mutex

   End = true;
   ReleaseMutex( HMutex );
   CloseHandle( HMutex );
   
   // At this time, the HMutex is REALLY DESTROYED
}  


The trouble is when i release and close the Mutex handle,
only one thread will continue running. The other thread is
DEFINITELY lost. You can see the thread in "perfmon.exe".
You must call TerminateThread() api to shutdown the thread,
because the thread never returns from WaitForSingleObject().

When i close the Mutex handle, i do not know how many threads
are awaiting the mutex to be released. We could think that
the CloseHandle() will cease the WaitForSingleObject() execution
by returning WAIT_FAILED to threads but it DOES NOT.

This is same thing for Critical Section, Event, Semaphore.

MY QUESTION :
How to prevent the thread from being lost ?
 
0
Comment
Question by:mars
  • 2
2 Comments
 
LVL 22

Accepted Solution

by:
nietod earned 400 total points
ID: 2646034
First of all, you really shoudl be using critical sections for this, not mutexes.

next you need to declare end as volatile, otherwise it may be optimized out of existance, preventing the threads from ending.  (VERY likely.)

Finally  you should not close the mutex handle unil the other threads finish.  To do this you can wait on the thread handles with WaitForSingleObject().  The wait function will return when the thread ends.  So use this to wait until two the additional threads are done, then close the mutex handle

let me know if you have any quesitons.
0
 
LVL 22

Expert Comment

by:nietod
ID: 2646046
One more thing the threads that lock the mutex never release it.  You should have them release the mutex before then terminate.
0

Featured Post

 [eBook] Windows Nano Server

Download this FREE eBook and learn all you need to get started with Windows Nano Server, including deployment options, remote management
and troubleshooting tips and tricks

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

This article shows a few slightly more advanced techniques for Windows 7 gadget programming, including how to save and restore user settings for your gadget and how to populate the "details" panel that is displayed in the Windows 7 gadget gallery.  …
This article describes how to add a user-defined command button to the Windows 7 Explorer toolbar.  In the previous article (http://www.experts-exchange.com/A_2172.html), we saw how to put the Delete button back there where it belongs.  "Delete" is …
This Micro Tutorial will teach you how to add a cinematic look to any film or video out there. There are very few simple steps that you will follow to do so. This will be demonstrated using Adobe Premiere Pro CS6.
This is Part 3 in a 3-part series on Experts Exchange to discuss error handling in VBA code written for Excel. Part 1 of this series discussed basic error handling code using VBA. http://www.experts-exchange.com/videos/1478/Excel-Error-Handlin…

783 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question