id	summary	reporter	owner	description	type	status	milestone	component	version	severity	resolution	keywords	cc
5840	thread.cpp: race condition in thread_proxy's thread state	noloader@…	viboes	"Using BOOST_HAS_PTHREADS to clarify the code, but all paths appear to suffer.


{{{
extern ""C"" {
  static void* thread_proxy(void* param)
  {
    // try
    {
      thread_param* p = static_cast<thread_param*>(param);
      boost::function0<void> threadfunc = p->m_threadfunc;
      p->started();
      threadfunc();
    }
    ...
  }
}}}

p->started() is called *before* the actual thread function. started() will change state even if an exception occurs such that the thread is not actually started:

{{{
void started()
{
  boost::mutex::scoped_lock scoped_lock(m_mutex);
  m_started = true;
  m_condition.notify_one();
}
}}}
"	Bugs	closed	To Be Determined	thread	Boost 1.47.0	Problem	invalid		viboes