New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 806634 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jan 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , iOS , Chrome
Pri: 2
Type: Bug-Regression



Sign in to add a comment

net_unittests fails on Linux/Linux ChromiumOS MSan Tests

Project Member Reported by sheriff-...@appspot.gserviceaccount.com, Jan 29 2018

Issue description

Filed by sheriff-o-matic@appspot.gserviceaccount.com on behalf of shimazu@chromium.org

These three tests are failing on MSan bots: 
GetPromisedUrlFromHeaderBlock.AuthorityWithPath
GetPromisedUrlFromHeaderBlock.InvalidScheme
GetPromisedUrlFromHeaderBlock.SchemeWithAuthority

Example failure: 
https://ci.chromium.org/buildbot/chromium.memory/Linux%20MSan%20Tests/7504

One of failure log:

[ RUN      ] GetPromisedUrlFromHeaderBlock.AuthorityWithPath
==12579==WARNING: MemorySanitizer: use-of-uninitialized-value
    #0 0xc598576 in net::QuicUrlUtilsImpl::GetPushPromiseUrl(base::BasicStringPiece<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > >, base::BasicStringPiece<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > >, base::BasicStringPiece<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > >) ./../../net/quic/platform/impl/quic_url_utils_impl.cc:54:7
    #1 0xc58f19b in net::QuicUrlUtils::GetPushPromiseUrl(base::BasicStringPiece<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > >, base::BasicStringPiece<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > >, base::BasicStringPiece<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > >) ./../../net/quic/platform/api/quic_url_utils.cc:25:10
    #2 0xc55f857 in net::SpdyUtils::GetPromisedUrlFromHeaderBlock(net::SpdyHeaderBlock const&) ./../../net/quic/core/spdy_utils.cc:181:10
    #3 0x68ad4e8 in net::test::GetPromisedUrlFromHeaderBlock_AuthorityWithPath_Test::TestBody() ./../../net/quic/core/spdy_utils_test.cc:397:3
    #4 0x92487f3 in testing::Test::Run() ./../../third_party/googletest/src/googletest/src/gtest-internal-inl.h:0:10
    #5 0x924c269 in testing::TestInfo::Run() ./../../third_party/googletest/src/googletest/src/gtest.cc:2651:11
    #6 0x924dc39 in testing::TestCase::Run() ./../../third_party/googletest/src/googletest/src/gtest.cc:2769:28
    #7 0x926f00e in testing::internal::UnitTestImpl::RunAllTests() ./../../third_party/googletest/src/googletest/src/gtest.cc:4665:43
    #8 0x926dcdf in testing::UnitTest::Run() ./../../third_party/googletest/src/googletest/src/gtest.cc:0:0
    #9 0xd99170b in RUN_ALL_TESTS ./../../third_party/googletest/src/googletest/include/gtest/gtest.h:2233:46
    #10 0xd99170b in base::TestSuite::Run() ./../../base/test/test_suite.cc:272:0
    #11 0xd99d31b in Run ./../../base/callback.h:94:12
    #12 0xd99d31b in base::(anonymous namespace)::LaunchUnitTestsInternal(base::RepeatingCallback<int ()> const&, unsigned long, int, bool, base::RepeatingCallback<void ()> const&) ./../../base/test/launcher/unit_test_launcher.cc:220:0
    #13 0xd99cb75 in base::LaunchUnitTests(int, char**, base::RepeatingCallback<int ()> const&) ./../../base/test/launcher/unit_test_launcher.cc:558:10
    #14 0x7d1c55e in main ./../../net/test/run_all_unittests.cc:64:10
    #15 0x7f10dc8ccf44 in __libc_start_main /build/eglibc-ripdx6/eglibc-2.19/csu/libc-start.c:287:0
    #16 0xb2be69 in _start ??:0:0
  Uninitialized value was created by a heap allocation
    #0 0xb9bd19 in operator new[](unsigned long) /b/build/slave/linux_upload_clang/build/src/third_party/llvm/compiler-rt/lib/msan/msan_new_delete.cc:47:37
    #1 0xbc599a7 in Block ./../../net/base/arena.cc:90:44
    #2 0xbc599a7 in net::UnsafeArena::AllocBlock(unsigned long) ./../../net/base/arena.cc:86:0
    #3 0xbc593fe in net::UnsafeArena::Memdup(char const*, unsigned long) ./../../net/base/arena.cc:76:5
    #4 0xc757cf6 in Write ./../../net/spdy/core/spdy_header_block.cc:64:35
    #5 0xc757cf6 in net::SpdyHeaderBlock::WriteKey(base::BasicStringPiece<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > >) ./../../net/spdy/core/spdy_header_block.cc:384:0
    #6 0xc75743f in net::SpdyHeaderBlock::operator[](base::BasicStringPiece<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > >) ./../../net/spdy/core/spdy_header_block.cc:319:15
    #7 0x68ad31c in net::test::GetPromisedUrlFromHeaderBlock_AuthorityWithPath_Test::TestBody() ./../../net/quic/core/spdy_utils_test.cc:393:3
    #8 0x92487f3 in testing::Test::Run() ./../../third_party/googletest/src/googletest/src/gtest-internal-inl.h:0:10
    #9 0x924c269 in testing::TestInfo::Run() ./../../third_party/googletest/src/googletest/src/gtest.cc:2651:11
    #10 0x924dc39 in testing::TestCase::Run() ./../../third_party/googletest/src/googletest/src/gtest.cc:2769:28
    #11 0x926f00e in testing::internal::UnitTestImpl::RunAllTests() ./../../third_party/googletest/src/googletest/src/gtest.cc:4665:43
    #12 0x926dcdf in testing::UnitTest::Run() ./../../third_party/googletest/src/googletest/src/gtest.cc:0:0
    #13 0xd99170b in RUN_ALL_TESTS ./../../third_party/googletest/src/googletest/include/gtest/gtest.h:2233:46
    #14 0xd99170b in base::TestSuite::Run() ./../../base/test/test_suite.cc:272:0
    #15 0xd99d31b in Run ./../../base/callback.h:94:12
    #16 0xd99d31b in base::(anonymous namespace)::LaunchUnitTestsInternal(base::RepeatingCallback<int ()> const&, unsigned long, int, bool, base::RepeatingCallback<void ()> const&) ./../../base/test/launcher/unit_test_launcher.cc:220:0
    #17 0xd99cb75 in base::LaunchUnitTests(int, char**, base::RepeatingCallback<int ()> const&) ./../../base/test/launcher/unit_test_launcher.cc:558:10
    #18 0x7d1c55e in main ./../../net/test/run_all_unittests.cc:64:10
    #19 0x7f10dc8ccf44 in __libc_start_main /build/eglibc-ripdx6/eglibc-2.19/csu/libc-start.c:287:0
SUMMARY: MemorySanitizer: use-of-uninitialized-value (/b/s/w/ir/out/Release/net_unittests+0xc598576)

 
Labels: OS-Chrome OS-iOS OS-Linux Type-Bug-Regression
Owner: wangyix@chromium.org
Status: Assigned (was: Available)
This failure is starting from https://ci.chromium.org/buildbot/chromium.memory/Linux%20MSan%20Tests/7504 .
I guess crrev.com/c/887970 causes this. I'm reverting it speculatively.
Assigned to the CL's owner. If it seems wrong, let me know.
Cc: -shimazu@chromium.org

Comment 3 by pkl@chromium.org, Jan 29 2018

Components: Internals>Network>QUIC

Comment 4 by rch@chromium.org, Jan 29 2018

Since that CL has been reverted, can this bug be closed? 
Status: Fixed (was: Assigned)

Sign in to add a comment