[ARROW-16273] [C++] Valgrind error in arrow-compute-scalar-test

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Duplicate
• Component/s: C++, Continuous Integration
• External issue URL: https://github.com/apache/arrow/issues/31667
• Assignee: Unassigned
• Reporter: Weston Pace
• Created: 21/Apr/22 20:34
• Updated: 11/Jan/23 11:42
• Resolved: 25/Apr/22 09:27

Description

Currently valgrind is failing earlier on the tpch-node-test and hash-join-node-test. Once we fix those tests it seems the next error is this:

[ RUN      ] TestStringKernels/0.Strptime
==9928== Conditional jump or move depends on uninitialised value(s)
==9928==    at 0x411AEA2: arrow::TestInitialized(arrow::ArrayData const&) (gtest_util.cc:682)
==9928==    by 0xAE1C79: arrow::compute::(anonymous namespace)::ValidateOutput(arrow::ArrayData const&) (test_util.cc:287)
==9928==    by 0xAE23FC: arrow::compute::ValidateOutput(arrow::Datum const&) (test_util.cc:320)
==9928==    by 0xAE4946: arrow::compute::CheckScalarNonRecursive(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::vector<arrow::Datum, std::allocator<arrow::Datum> > const&, arrow::Datum const&, arrow::compute::FunctionOptions const*) (test_util.cc:80)
==9928==    by 0xAE63A4: arrow::compute::CheckScalar(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::vector<arrow::Datum, std::allocator<arrow::Datum> > const&, arrow::Datum, arrow::compute::FunctionOptions const*) (test_util.cc:108)
==9928==    by 0xAE7E28: arrow::compute::CheckScalarUnary(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, arrow::Datum, arrow::Datum, arrow::compute::FunctionOptions const*) (test_util.cc:254)
==9928==    by 0xAE80D3: arrow::compute::CheckScalarUnary(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::shared_ptr<arrow::DataType>, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::shared_ptr<arrow::DataType>, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, arrow::compute::FunctionOptions const*) (test_util.cc:260)
==9928==    by 0x9F783F: arrow::compute::BaseTestStringKernels<arrow::StringType>::CheckUnary(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::shared_ptr<arrow::DataType>, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, arrow::compute::FunctionOptions const*) (scalar_string_test.cc:56)
==9928==    by 0xA2A62D: arrow::compute::TestStringKernels_Strptime_Test<arrow::StringType>::TestBody() (scalar_string_test.cc:1855)
==9928==    by 0x64974DC: void testing::internal::HandleSehExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::*)(), char const*) (gtest.cc:2607)
==9928==    by 0x648E90C: void testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::*)(), char const*) (gtest.cc:2643)
==9928==    by 0x6469CDC: testing::Test::Run() (gtest.cc:2682)
==9928==    by 0x646A6FE: testing::TestInfo::Run() (gtest.cc:2861)
==9928==    by 0x646B0BD: testing::TestSuite::Run() (gtest.cc:3015)
==9928==    by 0x647B1DB: testing::internal::UnitTestImpl::RunAllTests() (gtest.cc:5855)
==9928==    by 0x6498497: bool testing::internal::HandleSehExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool>(testing::internal::UnitTestImpl*, bool (testing::internal::UnitTestImpl::*)(), char const*) (gtest.cc:2607)
==9928==    by 0x648FAF9: bool testing::internal::HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool>(testing::internal::UnitTestImpl*, bool (testing::internal::UnitTestImpl::*)(), char const*) (gtest.cc:2643)
==9928==    by 0x64796A8: testing::UnitTest::Run() (gtest.cc:5438)
==9928==    by 0x4204918: RUN_ALL_TESTS() (gtest.h:2490)
==9928==    by 0x420495B: main (gtest_main.cc:52)
==9928== 
{
   <insert_a_suppression_name_here>
   Memcheck:Cond
   fun:_ZN5arrow15TestInitializedERKNS_9ArrayDataE
   fun:_ZN5arrow7compute12_GLOBAL__N_114ValidateOutputERKNS_9ArrayDataE
   fun:_ZN5arrow7compute14ValidateOutputERKNS_5DatumE
   fun:_ZN5arrow7compute23CheckScalarNonRecursiveERKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEERKSt6vectorINS_5DatumESaISA_EERKSA_PKNS0_15FunctionOptionsE
   fun:_ZN5arrow7compute11CheckScalarENSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEERKSt6vectorINS_5DatumESaIS8_EES8_PKNS0_15FunctionOptionsE
   fun:_ZN5arrow7compute16CheckScalarUnaryENSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEENS_5DatumES7_PKNS0_15FunctionOptionsE
   fun:_ZN5arrow7compute16CheckScalarUnaryENSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEESt10shared_ptrINS_8DataTypeEES6_S9_S6_PKNS0_15FunctionOptionsE
   fun:_ZN5arrow7compute21BaseTestStringKernelsINS_10StringTypeEE10CheckUnaryENSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEES9_St10shared_ptrINS_8DataTypeEES9_PKNS0_15FunctionOptionsE
   fun:_ZN5arrow7compute31TestStringKernels_Strptime_TestINS_10StringTypeEE8TestBodyEv
   fun:_ZN7testing8internal38HandleSehExceptionsInMethodIfSupportedINS_4TestEvEET0_PT_MS4_FS3_vEPKc
   fun:_ZN7testing8internal35HandleExceptionsInMethodIfSupportedINS_4TestEvEET0_PT_MS4_FS3_vEPKc
   fun:_ZN7testing4Test3RunEv
   fun:_ZN7testing8TestInfo3RunEv
   fun:_ZN7testing9TestSuite3RunEv
   fun:_ZN7testing8internal12UnitTestImpl11RunAllTestsEv
   fun:_ZN7testing8internal38HandleSehExceptionsInMethodIfSupportedINS0_12UnitTestImplEbEET0_PT_MS4_FS3_vEPKc
   fun:_ZN7testing8internal35HandleExceptionsInMethodIfSupportedINS0_12UnitTestImplEbEET0_PT_MS4_FS3_vEPKc
   fun:_ZN7testing8UnitTest3RunEv
   fun:_Z13RUN_ALL_TESTSv
   fun:main
}
==9928== Conditional jump or move depends on uninitialised value(s)
==9928==    at 0x411AEA2: arrow::TestInitialized(arrow::ArrayData const&) (gtest_util.cc:682)
==9928==    by 0x411AEF3: arrow::TestInitialized(arrow::Array const&) (gtest_util.cc:663)
==9928==    by 0xAE1E13: arrow::compute::(anonymous namespace)::ValidateOutput(arrow::ChunkedArray const&) (test_util.cc:293)
==9928==    by 0xAE240E: arrow::compute::ValidateOutput(arrow::Datum const&) (test_util.cc:323)
==9928==    by 0xAE753A: arrow::compute::CheckScalar(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::vector<arrow::Datum, std::allocator<arrow::Datum> > const&, arrow::Datum, arrow::compute::FunctionOptions const*) (test_util.cc:167)
==9928==    by 0xAE7E28: arrow::compute::CheckScalarUnary(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, arrow::Datum, arrow::Datum, arrow::compute::FunctionOptions const*) (test_util.cc:254)
==9928==    by 0xAE80D3: arrow::compute::CheckScalarUnary(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::shared_ptr<arrow::DataType>, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::shared_ptr<arrow::DataType>, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, arrow::compute::FunctionOptions const*) (test_util.cc:260)
==9928==    by 0x9F783F: arrow::compute::BaseTestStringKernels<arrow::StringType>::CheckUnary(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::shared_ptr<arrow::DataType>, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, arrow::compute::FunctionOptions const*) (scalar_string_test.cc:56)
==9928==    by 0xA2A62D: arrow::compute::TestStringKernels_Strptime_Test<arrow::StringType>::TestBody() (scalar_string_test.cc:1855)
==9928==    by 0x64974DC: void testing::internal::HandleSehExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::*)(), char const*) (gtest.cc:2607)
==9928==    by 0x648E90C: void testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::*)(), char const*) (gtest.cc:2643)
==9928==    by 0x6469CDC: testing::Test::Run() (gtest.cc:2682)
==9928==    by 0x646A6FE: testing::TestInfo::Run() (gtest.cc:2861)
==9928==    by 0x646B0BD: testing::TestSuite::Run() (gtest.cc:3015)
==9928==    by 0x647B1DB: testing::internal::UnitTestImpl::RunAllTests() (gtest.cc:5855)
==9928==    by 0x6498497: bool testing::internal::HandleSehExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool>(testing::internal::UnitTestImpl*, bool (testing::internal::UnitTestImpl::*)(), char const*) (gtest.cc:2607)
==9928==    by 0x648FAF9: bool testing::internal::HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool>(testing::internal::UnitTestImpl*, bool (testing::internal::UnitTestImpl::*)(), char const*) (gtest.cc:2643)
==9928==    by 0x64796A8: testing::UnitTest::Run() (gtest.cc:5438)
==9928==    by 0x4204918: RUN_ALL_TESTS() (gtest.h:2490)
==9928==    by 0x420495B: main (gtest_main.cc:52)
==9928== 
{
   <insert_a_suppression_name_here>
   Memcheck:Cond
   fun:_ZN5arrow15TestInitializedERKNS_9ArrayDataE
   fun:_ZN5arrow15TestInitializedERKNS_5ArrayE
   fun:_ZN5arrow7compute12_GLOBAL__N_114ValidateOutputERKNS_12ChunkedArrayE
   fun:_ZN5arrow7compute14ValidateOutputERKNS_5DatumE
   fun:_ZN5arrow7compute11CheckScalarENSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEERKSt6vectorINS_5DatumESaIS8_EES8_PKNS0_15FunctionOptionsE
   fun:_ZN5arrow7compute16CheckScalarUnaryENSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEENS_5DatumES7_PKNS0_15FunctionOptionsE
   fun:_ZN5arrow7compute16CheckScalarUnaryENSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEESt10shared_ptrINS_8DataTypeEES6_S9_S6_PKNS0_15FunctionOptionsE
   fun:_ZN5arrow7compute21BaseTestStringKernelsINS_10StringTypeEE10CheckUnaryENSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEES9_St10shared_ptrINS_8DataTypeEES9_PKNS0_15FunctionOptionsE
   fun:_ZN5arrow7compute31TestStringKernels_Strptime_TestINS_10StringTypeEE8TestBodyEv
   fun:_ZN7testing8internal38HandleSehExceptionsInMethodIfSupportedINS_4TestEvEET0_PT_MS4_FS3_vEPKc
   fun:_ZN7testing8internal35HandleExceptionsInMethodIfSupportedINS_4TestEvEET0_PT_MS4_FS3_vEPKc
   fun:_ZN7testing4Test3RunEv
   fun:_ZN7testing8TestInfo3RunEv
   fun:_ZN7testing9TestSuite3RunEv
   fun:_ZN7testing8internal12UnitTestImpl11RunAllTestsEv
   fun:_ZN7testing8internal38HandleSehExceptionsInMethodIfSupportedINS0_12UnitTestImplEbEET0_PT_MS4_FS3_vEPKc
   fun:_ZN7testing8internal35HandleExceptionsInMethodIfSupportedINS0_12UnitTestImplEbEET0_PT_MS4_FS3_vEPKc
   fun:_ZN7testing8UnitTest3RunEv
   fun:_Z13RUN_ALL_TESTSv
   fun:main
}
[       OK ] TestStringKernels/0.Strptime (45 ms)

Issue Links

• is superceded by: ARROW-16185 [C++][CI] Valgrind failure in strptime kernel tests