Merge pull request #6391 from SUSE/wip-13590-hammer

mon: should not set isvalid = true when cephx_verify_authorizer return false Reviewed-by: Sage Weil <sage@redhat.com>
ceph · Nov 17, 2015 · 779744a · 779744a
1 parent 8f8d1b8
commit 779744a
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/mon/Monitor.cc b/src/mon/Monitor.cc
@@ -4656,9 +4656,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 {