Re: [Boost-bugs] [Boost C++ Libraries] #8613: [Windows] boost::asio::ip::tcp::socket::async_write(boost::asio::null_buffers(), ...) WriteHandler gets a boost::system::error_code with a NULL category pointer on success

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #8613: [Windows] boost::asio::ip::tcp::socket::async_write(boost::asio::null_buffers(), ...) WriteHandler gets a boost::system::error_code with a NULL category pointer on success
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2013-05-24 11:00:09

#8613: [Windows]
boost::asio::ip::tcp::socket::async_write(boost::asio::null_buffers(), ...)
WriteHandler gets a boost::system::error_code with a NULL category pointer
on success
-----------------------------------------------+----------------------------
  Reporter:  Segev Finer <segev208@…>          |       Owner:  chris_kohlhoff
      Type:  Bugs                              |      Status:  reopened      
 Milestone:  To Be Determined                  |   Component:  asio          
   Version:  Boost 1.53.0                      |    Severity:  Problem       
Resolution:                                    |    Keywords:                
-----------------------------------------------+----------------------------

Comment (by Segev Finer <segev208@…>):

 The fix you posted does seem to work.

 Altough I think it's possible that this issue exists in the first place,
 since some other part of the code depended on `overlapped_contains_result`
 being passed and as such any code that uses this functions should be
 reviewed if it requires it or not. a quick search revelead that there
 aren't that many (Well at least ones that are actually relevant to
 win_iocp)

 For example: `win_iocp_overlapped_ptr::release()` calls `on_pending()`,
 and maybe it needs `overlapped_contains_result` and maybe it doesn't, I
 can't really tell from just reading the code.
 But maybe you already did this?...

 Setting the correct argument there is the best fix for this, anyhow IMHO.

 Thanks.

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