> On July 27, 2016, 8:43 a.m., haosdent huang wrote: > > 3rdparty/libprocess/include/process/socket.hpp, line 111 > > <https://reviews.apache.org/r/50477/diff/1/?file=1454469#file1454469line111> > > > > May we show `close.error()` here?
The `CHECK_SOME` macro will include the error message in the output. See: https://github.com/apache/mesos/blob/6d63c6236cec99a6ff79afd690c2287ccf3063eb/3rdparty/stout/include/stout/check.hpp#L159 - Joseph ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50477/#review143747 ----------------------------------------------------------- On July 26, 2016, 7:03 p.m., Benjamin Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50477/ > ----------------------------------------------------------- > > (Updated July 26, 2016, 7:03 p.m.) > > > Review request for mesos, Joris Van Remoortere and Joseph Wu. > > > Bugs: MESOS-5913 > https://issues.apache.org/jira/browse/MESOS-5913 > > > Repository: mesos > > > Description > ------- > > The deferred call to SSL_shutdown within ~LibeventSSLSocketImpl > can occur after the socket fd has been closed by Socket::~Impl. > > This can lead to a TLS Alert message being sent on any fd if > it the fd is re-used between the close and the SSL_shutdown! > > Thanks to Jan-Philip Gehrcke for reporting the issue. > > > Diffs > ----- > > 3rdparty/libprocess/include/process/socket.hpp > 881b44b987e5894cac838dae046ab7dbad20b000 > 3rdparty/libprocess/src/libevent_ssl_socket.hpp > 4d376d8b7c1b29105de69bed2e4077f8c94fed0b > 3rdparty/libprocess/src/libevent_ssl_socket.cpp > f4c0b0b97960400b0282837979bf0ed17f56a068 > > Diff: https://reviews.apache.org/r/50477/diff/ > > > Testing > ------- > > make check > > Ran my repro steps (issue HTTP requests while hammering the master with HTTPS > requests). Issue disappears after this fix. > > > Thanks, > > Benjamin Mahler > >
