update sources to v12.1.2

[ceph.git] / ceph / src / auth / cephx / CephxSessionHandler.cc

diff --git a/ceph/src/auth/cephx/CephxSessionHandler.cc b/ceph/src/auth/cephx/CephxSessionHandler.cc
index 087d6c54aa2eb07d25815efd702834ce0a3c5453..5694a22c0766eac87103ce0b157391b70578f5a2 100644 (file)
--- a/ceph/src/auth/cephx/CephxSessionHandler.cc
+++ b/ceph/src/auth/cephx/CephxSessionHandler.cc
@@ -81,7 +81,6 @@ int CephxSessionHandler::sign_message(Message *m)
   ceph_msg_footer& f = m->get_footer();
   f.sig = sig;
   f.flags = (unsigned)f.flags | CEPH_MSG_FOOTER_SIGNED;
-  messages_signed++;
   ldout(cct, 20) << "Putting signature in client message(seq # " << m->get_seq()
                 << "): sig = " << sig << dendl;
   return 0;
@@ -103,8 +102,6 @@ int CephxSessionHandler::check_message_signature(Message *m)
   if (r < 0)
     return r;
 
-  signatures_checked++;
-
   if (sig != m->get_footer().sig) {
     // Should have been signed, but signature check failed.  PLR
     if (!(m->get_footer().flags & CEPH_MSG_FOOTER_SIGNED)) {
@@ -122,14 +119,10 @@ int CephxSessionHandler::check_message_signature(Message *m)
     // security failure, particularly when there are large numbers of
     // them, since the latter is a potential sign of an attack.  PLR
 
-    signatures_failed++;
     ldout(cct, 0) << "Signature failed." << dendl;
     return (SESSION_SIGNATURE_FAILURE);
   }
 
-  // If we get here, the signature checked.  PLR
-  signatures_matched++;
-
   return 0;
 }