fix lock leak

src/lib/homestore/heap_chunk_selector.cpp

@@ -49,6 +49,7 @@ csharedChunk HeapChunkSelector::select_chunk(homestore::blk_count_t count, const
     it->second->first.lock();
     if(it->second->second.empty()) {
         LOGINFO("No more available chunks found for pdev {}", pdevID);
+        it->second->first.unlock();
         return nullptr;
     }
 

src/lib/homestore/heap_chunk_selector.h

@@ -44,14 +44,15 @@ class HeapChunkSelector : public homestore::ChunkSelector {
     csharedChunk select_chunk([[maybe_unused]]homestore::blk_count_t nblks, const homestore::blk_alloc_hints& hints);
 
 private:
+    //this holds all the chunks for each pdev, they are all unselected.
     folly::ConcurrentHashMap< uint32_t, std::shared_ptr< std::pair< std::mutex, VChunkHeap > > > m_pdev_heap_map;
+
     //for now, uint32_t is enough for the sum of all the available blocks of a pdev.
     //if necessary , we can change this to uint64_t to hold a larger sum.
     std::unordered_map< uint32_t, uint32_t> m_pdev_avalable_blk_map;
 
     //hold all the chunks , selected or not
     std::unordered_set< VChunk > m_chunks;
     std::mutex lock;
-
 };
 } // namespace homeobject