[ntuple] add assert()s in RPage(Pool) logic

root-project · Nov 15, 2024 · 0bc26ca · 0bc26ca
1 parent 77ae704
commit 0bc26ca
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/tree/ntuple/v7/inc/ROOT/RPage.hxx b/tree/ntuple/v7/inc/ROOT/RPage.hxx
@@ -18,6 +18,7 @@
 
 #include <ROOT/RNTupleUtil.hxx>
 
+#include <cassert>
 #include <cstddef>
 #include <cstdint>
 #include <memory>
@@ -152,6 +153,7 @@ public:
    /// return value is ignored.
    void *GrowUnchecked(ClusterSize_t::ValueType nElements)
    {
+      assert(fNElements + nElements <= fMaxElements);
       auto offset = GetNBytes();
       fNElements += nElements;
       return static_cast<unsigned char *>(fBuffer) + offset;

diff --git a/tree/ntuple/v7/src/RPagePool.cxx b/tree/ntuple/v7/src/RPagePool.cxx
@@ -55,6 +55,7 @@ void ROOT::Experimental::Internal::RPagePool::ReleasePage(const RPage &page)
    const auto idx = itrLookup->second;
    const auto N = fEntries.size();
 
+   assert(fEntries[idx].fRefCounter >= 1);
    if (--fEntries[idx].fRefCounter == 0) {
       fLookupByBuffer.erase(itrLookup);