Skip to content

Commit 36e1fa0

Browse files
committed
Preserve remote_endpoint_ across release() in reactor_stream_socket
On develop, reactor_stream_socket had no do_release_socket() override, so release went straight to the base which clears local_endpoint_ but leaves remote_endpoint_ intact. The refactor inadvertently added an override that also cleared remote_endpoint_, changing user-visible release semantics. Remove the override to restore the original behavior.
1 parent 647e610 commit 36e1fa0

File tree

1 file changed

+0
-12
lines changed

1 file changed

+0
-12
lines changed

include/boost/corosio/native/detail/reactor/reactor_stream_socket.hpp

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -230,18 +230,6 @@ class reactor_stream_socket
230230
remote_endpoint_ = Endpoint{};
231231
}
232232

233-
/** Release the socket without closing the fd.
234-
235-
Extends the base do_release_socket() to also reset
236-
the remote endpoint.
237-
*/
238-
native_handle_type do_release_socket() noexcept
239-
{
240-
auto fd = base_type::do_release_socket();
241-
remote_endpoint_ = Endpoint{};
242-
return fd;
243-
}
244-
245233
private:
246234
// CRTP callbacks for reactor_basic_socket cancel/close
247235

0 commit comments

Comments
 (0)