From: Sage Weil <sage@redhat.com>
Date: Thu, 13 Sep 2018 18:31:40 +0000 (-0500)
Subject: osd/osd_types: fix pg_t::contains() to check pool id too
X-Git-Tag: v13.2.3~99^2~3
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=a890baa50a12a6e57f8be0b5234efdffca108799;p=ceph.git

osd/osd_types: fix pg_t::contains() to check pool id too

This is used by bluestore fsck to ensure we're looking at the right
collection.  It needs to validate both that the pool id matches the
object (including the temp objects) and that the hash matches; we were
only checking the hash before.

Fixes: http://tracker.ceph.com/issues/32731
Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit 0bd2546eaca72ed0122a9c2648df4bef05b0d5d2)
---

diff --git a/src/common/hobject.h b/src/common/hobject.h
index 24eeb9754e2..d05d294abe1 100644
--- a/src/common/hobject.h
+++ b/src/common/hobject.h
@@ -42,6 +42,7 @@ private:
   bool max;
   uint32_t nibblewise_key_cache;
   uint32_t hash_reverse_bits;
+public:
   static const int64_t POOL_META = -1;
   static const int64_t POOL_TEMP_START = -2; // and then negative
   friend class spg_t;  // for POOL_TEMP_START
diff --git a/src/osd/osd_types.h b/src/osd/osd_types.h
index c211c58bbd1..95dbab04553 100644
--- a/src/osd/osd_types.h
+++ b/src/osd/osd_types.h
@@ -419,10 +419,16 @@ struct pg_t {
   unsigned get_split_bits(unsigned pg_num) const;
 
   bool contains(int bits, const ghobject_t& oid) {
-    return oid.match(bits, ps());
+    return
+      ((int64_t)m_pool == oid.hobj.pool ||
+       hobject_t::POOL_TEMP_START-(int64_t)m_pool == oid.hobj.pool) &&
+      oid.match(bits, ps());
   }
   bool contains(int bits, const hobject_t& oid) {
-    return oid.match(bits, ps());
+    return
+      ((int64_t)m_pool == oid.pool ||
+       hobject_t::POOL_TEMP_START-(int64_t)m_pool == oid.pool) &&
+      oid.match(bits, ps());
   }
 
   hobject_t get_hobj_start() const;