OSDCaps: fix allow_all()

OSD_CAP_ANY is not a mask. Treating it as one made any allowance
equivalent to 'allow *'.

Fixes: #3228
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>

diff --git a/src/osd/OSDCap.h b/src/osd/OSDCap.h
index 7f2e9f76eb58113be0869b3032b9d1df023d0881..941428c3f18062bf45383fa9577266b6517db022 100644 (file)
--- a/src/osd/OSDCap.h
+++ b/src/osd/OSDCap.h
@@ -50,7 +50,7 @@ struct OSDCapSpec {
   OSDCapSpec(std::string n, std::string a) : allow(0), class_name(n), class_allow(a) {}
 
   bool allow_all() const {
-    return allow & OSD_CAP_ANY;
+    return allow == OSD_CAP_ANY;
   }
 };