From: Yingxin Cheng <yingxincheng@gmail.com>
Date: Fri, 1 Mar 2019 07:46:36 +0000 (+0800)
Subject: msg/async: fix banner_v1 check in ProtocolV2
X-Git-Tag: v14.1.1~90^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F26714%2Fhead;p=ceph.git

msg/async: fix banner_v1 check in ProtocolV2

Signed-off-by: Yingxin Cheng <yingxincheng@gmail.com>
---

diff --git a/src/msg/async/ProtocolV2.cc b/src/msg/async/ProtocolV2.cc
index cf700b601d701..8c2d8f239fc76 100644
--- a/src/msg/async/ProtocolV2.cc
+++ b/src/msg/async/ProtocolV2.cc
@@ -826,7 +826,7 @@ CtPtr ProtocolV2::_handle_peer_banner(char *buffer, int r) {
   unsigned banner_prefix_len = strlen(CEPH_BANNER_V2_PREFIX);
 
   if (memcmp(buffer, CEPH_BANNER_V2_PREFIX, banner_prefix_len)) {
-    if (memcmp(buffer, CEPH_BANNER, strlen(CEPH_BANNER))) {
+    if (memcmp(buffer, CEPH_BANNER, strlen(CEPH_BANNER)) == 0) {
       lderr(cct) << __func__ << " peer " << *connection->peer_addrs
                  << " is using msgr V1 protocol" << dendl;
       return _fault();