From c7f75b8f7c0a773148ec16141941efd00ee76626 Mon Sep 17 00:00:00 2001 From: yangruifeng Date: Mon, 19 Oct 2015 08:08:12 -0400 Subject: [PATCH] mon: should not set isvalid = true when cephx_verify_authorizer return false Fixes: #13525 Signed-off-by: Ruifeng Yang --- src/mon/Monitor.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mon/Monitor.cc b/src/mon/Monitor.cc index dcfd5121e85de..58b13cf9294f5 100644 --- a/src/mon/Monitor.cc +++ b/src/mon/Monitor.cc @@ -5004,9 +5004,9 @@ bool Monitor::ms_verify_authorizer(Connection *con, int peer_type, CephXServiceTicketInfo auth_ticket_info; if (authorizer_data.length()) { - int ret = cephx_verify_authorizer(g_ceph_context, &keyring, iter, + bool ret = cephx_verify_authorizer(g_ceph_context, &keyring, iter, auth_ticket_info, authorizer_reply); - if (ret >= 0) { + if (ret) { session_key = auth_ticket_info.session_key; isvalid = true; } else { -- 2.47.3