[Boost-bugs] [Boost C++ Libraries] #8913: boost/asio/detail/posix_static_mutex.hpp: Ignores all failures from pthread_* functions

Subject: [Boost-bugs] [Boost C++ Libraries] #8913: boost/asio/detail/posix_static_mutex.hpp: Ignores all failures from pthread_* functions
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2013-07-25 04:25:37

#8913: boost/asio/detail/posix_static_mutex.hpp: Ignores all failures from
pthread_* functions
--------------------------------------------+----------------------------
 Reporter:  Jeffrey Walton <noloader@…>     |      Owner:  chris_kohlhoff
     Type:  Bugs                            |     Status:  new
Milestone:  To Be Determined                |  Component:  asio
  Version:  Boost 1.54.0                    |   Severity:  Problem
 Keywords:  asio mutex pthread API failure  |
--------------------------------------------+----------------------------
 boost/asio/detail/posix_static_mutex.hpp ignores nearly all failures from
 pthread_* functions. Functions include pthread_mutex_lock and
 pthread_mutex_unlock.

 A lock failure is usually a bad thing, and I can't come up with scenarios
 where a silent failure is desired. It will make a bad problem worse by
 corrupting data or terminating the program.

 At minimum (as a user), I would expect for Boost to use BOOST_ASSERT in
 debugging and diagnostic builds; and BOOST_VERIFY with an appropriate
 exception for release or production builds.

 Perhaps it would be a good idea to use boost/thread/pthread/mutex.hpp. It
 appears to be more mature and have a bit more stability. In addition, it
 throws lock exceptions where appropriate.

-- 
Ticket URL: <https://svn.boost.org/trac/boost/ticket/8913>
Boost C++ Libraries <http://www.boost.org/>
Boost provides free peer-reviewed portable C++ source libraries.