New issue
Advanced search Search tips

Issue 921763 link

Starred by 1 user
Status: Fixed
Owner:
est...@chromium.org
Closed: Jan 14
Cc:
jbroman@chromium.org
kmarshall@chromium.org
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----



Sign in to add a comment

base_unittests failing on chromium.memory/Linux MSan Tests

Project Member Reported by sheriff-...@appspot.gserviceaccount.com, Jan 14 
Filed by sheriff-o-matic@appspot.gserviceaccount.com on behalf of kmarshall@chromium.org

base_unittests failing on chromium.memory/Linux MSan Tests

Builders failed on: 
- Linux MSan Tests: 
  https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Linux%20MSan%20Tests

Comment 1 by kmarshall@chromium.org, Jan 14

Cc: jbroman@chromium.org
Labels: -Sheriff-Chromium
Owner: est...@chromium.org
Status: Assigned (was: Available)
Looks like the code is reading an uninitialized value. Revert CL is here. https://chromium-review.googlesource.com/c/chromium/src/+/1409815


[ RUN      ] TimezoneTest.CountryCodeForTimezones
==6383==WARNING: MemorySanitizer: use-of-uninitialized-value
    #0 0x5649ff0500c1 in icu_63::TZEnumeration::snext(UErrorCode&) ./../../third_party/icu/source/i18n/timezone.cpp:931:31
    #1 0x5649fb95cc45 in base::(anonymous namespace)::TimezoneTest_CountryCodeForTimezones_Test::TestBody() ./../../base/i18n/timezone_unittest.cc:20:58
    #2 0x5649fe5819a0 in HandleExceptionsInMethodIfSupported<testing::Test, void> ./../../third_party/googletest/src/googletest/src/gtest.cc:0:0
    #3 0x5649fe5819a0 in testing::Test::Run() ./../../third_party/googletest/src/googletest/src/gtest.cc:2522:0
    #4 0x5649fe5854d3 in testing::TestInfo::Run() ./../../third_party/googletest/src/googletest/src/gtest.cc:2703:11
    #5 0x5649fe5870e9 in testing::TestCase::Run() ./../../third_party/googletest/src/googletest/src/gtest.cc:2825:28
    #6 0x5649fe5c19d9 in testing::internal::UnitTestImpl::RunAllTests() ./../../third_party/googletest/src/googletest/src/gtest.cc:5227:43
    #7 0x5649fe5c02c8 in HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool> ./../../third_party/googletest/src/googletest/src/gtest.cc:0:0
    #8 0x5649fe5c02c8 in testing::UnitTest::Run() ./../../third_party/googletest/src/googletest/src/gtest.cc:4835:0
    #9 0x5649fec2242a in RUN_ALL_TESTS ./../../third_party/googletest/src/googletest/include/gtest/gtest.h:2369:46
    #10 0x5649fec2242a in base::TestSuite::Run() ./../../base/test/test_suite.cc:294:0
    #11 0x5649fec4cd49 in Run ./../../base/callback.h:99:12
    #12 0x5649fec4cd49 in base::(anonymous namespace)::LaunchUnitTestsInternal(base::OnceCallback<int ()>, unsigned long, int, bool, base::OnceCallback<void ()>) ./../../base/test/launcher/unit_test_launcher.cc:225:0
    #13 0x5649fec4c4a7 in base::LaunchUnitTests(int, char**, base::OnceCallback<int ()>) ./../../base/test/launcher/unit_test_launcher.cc:575:10
    #14 0x5649febe52a8 in main ./../../base/test/run_all_base_unittests.cc:12:10
    #15 0x7f5f95134f44 in __libc_start_main ??:0:0
    #16 0x5649fad20029 in _start ??:0:0

Comment 2 by est...@chromium.org, Jan 14

Status: Fixed (was: Assigned)
change was reveretd
►

Sign in to add a comment