From 7e2b14c7b80fbb1cf0ea0e91dad0a2f50425f625 Mon Sep 17 00:00:00 2001
From: Casey Bodley <cbodley@redhat.com>
Date: Tue, 2 Mar 2021 09:31:08 -0500
Subject: [PATCH] valgrind: suppress operator delete[] for tcmalloc

suppresses a mismatched operator delete[] from the librados client's
destructor:

<error>
  <unique>0xeef7eb</unique>
  <tid>1</tid>
  <kind>MismatchedFree</kind>
  <what>Mismatched free() / delete / delete []</what>
  <stack>
    <frame>
      <ip>0x4C32EA0</ip>
      <obj>/usr/libexec/valgrind/vgpreload_memcheck-amd64-linux.so</obj>
      <fn>operator delete[](void*, unsigned long)</fn>
      <dir>/builddir/build/BUILD/valgrind-3.16.0/coregrind/m_replacemalloc</dir>
      <file>vg_replace_malloc.c</file>
      <line>660</line>
    </frame>
    <frame>
      <ip>0x5FBEB86</ip>
      <obj>/usr/lib64/librados.so.2.0.0</obj>
    </frame>
    <frame>
      <ip>0x5FBEF2C</ip>
      <obj>/usr/lib64/librados.so.2.0.0</obj>
    </frame>
    <frame>
      <ip>0x5F7F7C5</ip>
      <obj>/usr/lib64/librados.so.2.0.0</obj>
      <fn>librados::v14_2_0::Rados::shutdown()</fn>
    </frame>

Fixes: https://tracker.ceph.com/issues/49501

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit bf087c3f840216a704600bb2436c643159421ea2)
---
 qa/valgrind.supp | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/qa/valgrind.supp b/qa/valgrind.supp
index a7257faaf6d6..c284ecd43f6b 100644
--- a/qa/valgrind.supp
+++ b/qa/valgrind.supp
@@ -5,7 +5,13 @@
    fun:free
    ...
 }
-
+{
+   operator delete[] in Rados::shutdown
+   Memcheck:Free
+   fun:_ZdaPvm
+   ...
+   fun:_ZN8librados7v14_2_05Rados8shutdownEv
+}
 {
    older boost mersenne twister uses uninitialized memory for randomness
    Memcheck:Cond
-- 
2.47.3