If I call GetExitCodeThread for a thread that I know for sure has exited, why does it still say STILL_ACTIVE?

Raymond Chen

Raymond

A customer reported that when they took the thread handle returned by the _beginthread function and passed it to Get­Exit­Code­Thread, the function reported that the thread was STILL_ACTIVE, even though the thread is known to have already exited.

The customer found this text in the documentation for _beginthread and wants more information about the case where the function can return an invalid handle if the thread exits quickly.

If the thread that’s generated by _beginthread exits quickly, the handle that’s returned to the caller of _beginthread might be invalid or point to another thread.

The point is that the _begin­thread function returns a handle to a thread, but that handle is valid only as long as the thread is running. Once the thread exits, the handle spontaneously becomes invalid, at which point it becomes eligible for recycling.

The upshot of this is that the handle returned by _begin­thread is useless, because it can go invalid for reasons outside your control.

The source code for the C runtime library is included with Visual Studio. You can read the source code for the _begin­thread function if you want to understand what’s going on.

The customer asked whether this behavior of _begin­thread would explain the problem they are seeing.

Yes, it is exactly the problem you are seeing. Calling Get­Exit­Code­Thread with the handle returned by _begin­thread is useless.

  • If the thread is running, then you will get STILL_ACTIVE.
  • If the thread is not running, then you are using an invalid parameter and you will get garbage. The value STILL_ACTIVE is one possible manifestation of garbage.

In general, the return value from _begin­thread is not useful and should not be used for anything other than determining whether the thread was started successfully.

We recommend that if the customer wants to use the thread handle, they should switch to _begin­thread­ex and remember to close the handle when they are done.

The customer explained that their application was originally developed by another company. They had considered switching from _begin­thread to _begin­thread­ex, but didn’t want to do so unless absolutely necessary, because they would have to justify the time and money required to fix the problem to their management.

Yes, switching from _begin­thread to _begin­thread­ex will fix the problem. As noted, the handle returned by _begin­thread is useless, and any code that tries to do anything beyond test it for success is like to run into problems exactly like the one you describe.

We learned about some people who want documentation that a bad idea is a bad idea. But that documentation is already there in MSDN. They already have the paperwork they need. I couldn’t quite figure out what the customer was looking for. Did they just want a personalized version of the documentation customized just for them?

If that’s what you need, you can copy/paste the following paragraph.

To whom it may concern,

It is my recommendation that the return value from the _begin­thread function should be used only to determine whether the function succeeded. In particular, the handle returned by the function is of indeterminate lifetime and cannot be reliably used.

Sincerely,

Raymond Chen

Raymond Chen
Raymond Chen

Follow Raymond   

0 comments

Comments are closed.