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.

This archive was generated by hypermail 2.1.7 : 2017-02-16 18:50:13 UTC