pitrou opened a new issue, #45270:
URL: https://github.com/apache/arrow/issues/45270

   ### Describe the bug, including details regarding any error messages, 
version, and platform.
   
   The Valgrind CI job is sometimes polluted by mimalloc-related Valgrind 
errors, such as:
   
https://github.com/ursacomputing/crossbow/actions/runs/12791589715/job/35660053477#step:7:3050
   ```
   [ RUN      ] TestPrimitiveWriter/6.OptionalSpaced
   ==15960== Conditional jump or move depends on uninitialised value(s)
   ==15960==    at 0x64C895D: mi_check_is_double_free (alloc.c:197)
   ==15960==    by 0x64C95FA: mi_free (alloc.c:497)
   ==15960==    by 0x640C95F: arrow::(anonymous 
namespace)::MimallocAllocator::DeallocateAligned(unsigned char*, long, long) 
(memory_pool.cc:430)
   ==15960==    by 0x640D544: arrow::BaseMemoryPoolImpl<arrow::(anonymous 
namespace)::MimallocAllocator>::Free(unsigned char*, long, long) 
(memory_pool.cc:508)
   ==15960==    by 0x6408C21: arrow::PoolBuffer::~PoolBuffer() 
(memory_pool.cc:851)
   ==15960==    by 0x6408C50: arrow::PoolBuffer::~PoolBuffer() 
(memory_pool.cc:853)
   ==15960==    by 0x496B651: operator() (unique_ptr.h:99)
   ==15960==    by 0x496B651: std::unique_ptr<arrow::Buffer, 
std::default_delete<arrow::Buffer> >::~unique_ptr() (unique_ptr.h:404)
   
   [etc.]
   ```
   
   Since Valgrind is probably not mimalloc-aware, we should simply disable 
mimalloc on Valgrind builds so that the system allocator is used.
   
   ### Component(s)
   
   C++, Continuous Integration


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@arrow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to