osd/OSD: add 'isvalid=false' when failed to parse caps

Signed-off-by: Yanhu Cao <gmayyyha@gmail.com>

diff --git a/src/osd/OSD.cc b/src/osd/OSD.cc
index 58245ecb49de3eb2ad53bcef1a0c7e8e8d77a933..b44b4986f624d980178978061c203d12cee24964 100644 (file)
--- a/src/osd/OSD.cc
+++ b/src/osd/OSD.cc
@@ -7057,12 +7057,15 @@ bool OSD::ms_verify_authorizer(Connection *con, int peer_type,
        ::decode(str, p);
       }
       catch (buffer::error& e) {
+        isvalid = false;
       }
       bool success = s->caps.parse(str);
       if (success)
        dout(10) << " session " << s << " " << s->entity_name << " has caps " << s->caps << " '" << str << "'" << dendl;
-      else
+      else {
        dout(10) << " session " << s << " " << s->entity_name << " failed to parse caps '" << str << "'" << dendl;
+        isvalid = false;
+      }
     }
 
     s->put();