mon: should not set isvalid = true when cephx_verify_authorizer retur…

…n false Fixes: ceph#13525 Signed-off-by: Ruifeng Yang <yangruifeng.09209@h3c.com>
SUSE · Oct 19, 2015 · c7f75b8 · c7f75b8
1 parent c55f8d4
commit c7f75b8
Showing 1 changed file with 2 additions and 2 deletions.
diff --git 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 {