From d8b797fe876bd3f2182169908db5a1a5172569d2 Mon Sep 17 00:00:00 2001 From: Radoslaw Zarzynski Date: Fri, 8 Mar 2019 13:14:52 +0100 Subject: [PATCH] msg/async, v2: drop temp_buffer and limitations driven by it. Signed-off-by: Radoslaw Zarzynski --- src/msg/async/ProtocolV2.cc | 6 ------ src/msg/async/ProtocolV2.h | 1 - 2 files changed, 7 deletions(-) diff --git a/src/msg/async/ProtocolV2.cc b/src/msg/async/ProtocolV2.cc index aa3b80865b2..919ec79124f 100644 --- a/src/msg/async/ProtocolV2.cc +++ b/src/msg/async/ProtocolV2.cc @@ -80,7 +80,6 @@ if(connection->interceptor) { \ ProtocolV2::ProtocolV2(AsyncConnection *connection) : Protocol(2, connection), - temp_buffer(nullptr), state(NONE), peer_required_features(0), client_cookie(0), @@ -96,11 +95,9 @@ ProtocolV2::ProtocolV2(AsyncConnection *connection) next_payload_len(0), next_tag(static_cast(0)), keepalive(false) { - temp_buffer = new char[4096]; } ProtocolV2::~ProtocolV2() { - delete[] temp_buffer; } void ProtocolV2::connect() { @@ -809,9 +806,6 @@ CtPtr ProtocolV2::_handle_peer_banner(rx_buffer_t &&buffer, int r) { return _fault(); } - ceph_assert(payload_len <= 4096); // if we need more then we need to increase - // temp_buffer size as well - next_payload_len = payload_len; INTERCEPT(state == BANNER_CONNECTING ? 5 : 6); diff --git a/src/msg/async/ProtocolV2.h b/src/msg/async/ProtocolV2.h index f3e42368a10..52fa967d2cb 100644 --- a/src/msg/async/ProtocolV2.h +++ b/src/msg/async/ProtocolV2.h @@ -68,7 +68,6 @@ public: ceph::crypto::onwire::rxtx_t session_stream_handlers; private: entity_name_t peer_name; - char *temp_buffer; State state; uint64_t peer_required_features; -- 2.39.5