id,summary,reporter,owner,description,type,status,milestone,component,version,severity,resolution,keywords,cc
5841,thread.cpp: thread_group: possible resource leak between create_thread/add_thread,noloader@…,viboes,"If add_thread finds the newly created thread in its list, it will not add the thread. However, the thread/pointer will not be properly disposed. In addition, create_thread will return a value which indicates success regardless of the result of add_thread.

{{{
thread* thread_group::create_thread(const function0<void>& threadfunc)
{
    std::auto_ptr<thread> thrd(new thread(threadfunc));
    add_thread(thrd.get());
    return thrd.release();
}

void thread_group::add_thread(thread* thrd)
{
    mutex::scoped_lock scoped_lock(m_mutex);

    // For now we'll simply ignore requests to add a thread object multiple
    // times. Should we consider this an error and either throw or return an
    // error value?
    std::list<thread*>::iterator it = std::find(m_threads.begin(),
        m_threads.end(), thrd);
    assert(it == m_threads.end());
    if (it == m_threads.end())
        m_threads.push_back(thrd);
}
}}}
",Bugs,closed,To Be Determined,thread,Boost 1.47.0,Problem,invalid,,viboes