Boost C++ Libraries: Ticket #5859: win32 shared_mutex constructor leaks on exceptions

owner, status changed; cc set

viboes — Fri, 02 Dec 2011 22:24:08 GMT

cc viboes added
owner changed from Anthony Williams to viboes
status new → assigned

Thanks for this report. I will try to solve it soon.

viboes — Sun, 04 Dec 2011 15:38:10 GMT

In order to avoid a chained try-catch a no_throw create_anonymous_semaphore_nothrow function can be used like in

            if (!semaphores[exclusive_sem])
            {
              detail::win32::release_semaphore(semaphores[unlock_sem],LONG_MAX);
              boost::throw_exception(thread_resource_error());
            }
            upgrade_sem=detail::win32::create_anonymous_semaphore_nothrow(0,LONG_MAX);
            if (!upgrade_sem)
            {
              detail::win32::release_semaphore(semaphores[unlock_sem],LONG_MAX);
              detail::win32::release_semaphore(semaphores[exclusive_sem],LONG_MAX);
              boost::throw_exception(thread_resource_error());
            }

While this is not too elegant it solves the issue. A RAI handle would be also appreciated.

status, milestone changed; resolution set

viboes — Sun, 04 Dec 2011 23:07:16 GMT

status assigned → closed
resolution → fixed
milestone To Be Determined → Boost 1.49.0

Committed in trunk At revision: [75806]

status changed; resolution deleted

viboes — Sun, 04 Dec 2011 23:17:45 GMT

status closed → reopened
resolution fixed

Let close the ticket when merged to release

status changed; resolution set

viboes — Sat, 07 Jan 2012 22:49:57 GMT

status reopened → closed
resolution → fixed

Committed in release at [76291]