New issue
Advanced search Search tips

Issue 697172 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2017
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug

Blocking:
issue 697171



Sign in to add a comment

ProcessMemoryDumpTest.TakeAllDumpsFrom failing under msan

Project Member Reported by thakis@chromium.org, Feb 28 2017

Issue description

https://build.chromium.org/p/chromium.memory.full/builders/Linux%20MSan%20Tests/builds/6001/steps/base_unittests%20on%20Ubuntu-14.04/logs/ProcessMemoryDumpTest.TakeAllDumpsFrom

ProcessMemoryDumpTest.TakeAllDumpsFrom (run #1):
[ RUN      ] ProcessMemoryDumpTest.TakeAllDumpsFrom
==19772==WARNING: MemorySanitizer: use-of-uninitialized-value
    #0 0x291dd88 in BreakDownBy base/trace_event/heap_profiler_heap_dump_writer.cc:150:9
    #1 0x291830b in BreakDown base/trace_event/heap_profiler_heap_dump_writer.cc:224:23
    #2 0x2917ea7 in Summarize base/trace_event/heap_profiler_heap_dump_writer.cc:262:3
    #3 0x291a136 in ExportHeapDump base/trace_event/heap_profiler_heap_dump_writer.cc:318:27
    #4 0x294c5cd in DumpHeapUsage base/trace_event/process_memory_dump.cc:248:46
    #5 0x21ecdff in TestBody base/trace_event/process_memory_dump_unittest.cc:108:9
    #6 0x2b2e7cc in HandleExceptionsInMethodIfSupported\u003Ctesting::Test, void> testing/gtest/src/gtest.cc:2458:12
    #7 0x2b2e7cc in Run testing/gtest/src/gtest.cc:2474:0
    #8 0x2b31151 in Run testing/gtest/src/gtest.cc:2656:11
    #9 0x2b326b9 in Run testing/gtest/src/gtest.cc:2774:28
    #10 0x2b52bcd in RunAllTests testing/gtest/src/gtest.cc:4647:43
    #11 0x2b51a3e in HandleExceptionsInMethodIfSupported\u003Ctesting::internal::UnitTestImpl, bool> testing/gtest/src/gtest.cc:2458:12
    #12 0x2b51a3e in Run testing/gtest/src/gtest.cc:4255:0
    #13 0x2a4ece0 in RUN_ALL_TESTS testing/gtest/include/gtest/gtest.h:2237:46
    #14 0x2a4ece0 in Run base/test/test_suite.cc:271:0
    #15 0x2a7ceba in Run base/callback.h:85:12
    #16 0x2a7ceba in LaunchUnitTestsInternal base/test/launcher/unit_test_launcher.cc:211:0
    #17 0x2a7c6dd in LaunchUnitTests base/test/launcher/unit_test_launcher.cc:453:10
    #18 0x2a19673 in main base/test/run_all_base_unittests.cc:22:10
    #19 0x7f402d143f44 in __libc_start_main /build/eglibc-oGUzwX/eglibc-2.19/csu/libc-start.c:287:0
    #20 0x59958c in _start ??:?

  Uninitialized value was stored to memory at
    #0 0x291a20c in HeapDumpWriter base/trace_event/heap_profiler_heap_dump_writer.cc:187:7
    #1 0x291a20c in ExportHeapDump base/trace_event/heap_profiler_heap_dump_writer.cc:314:0
    #2 0x294c5cd in DumpHeapUsage base/trace_event/process_memory_dump.cc:248:46
    #3 0x21ecdff in TestBody base/trace_event/process_memory_dump_unittest.cc:108:9
    #4 0x2b2e7cc in HandleExceptionsInMethodIfSupported\u003Ctesting::Test, void> testing/gtest/src/gtest.cc:2458:12
    #5 0x2b2e7cc in Run testing/gtest/src/gtest.cc:2474:0
    #6 0x2b31151 in Run testing/gtest/src/gtest.cc:2656:11
    #7 0x2b326b9 in Run testing/gtest/src/gtest.cc:2774:28
    #8 0x2b52bcd in RunAllTests testing/gtest/src/gtest.cc:4647:43
    #9 0x2b51a3e in HandleExceptionsInMethodIfSupported\u003Ctesting::internal::UnitTestImpl, bool> testing/gtest/src/gtest.cc:2458:12
    #10 0x2b51a3e in Run testing/gtest/src/gtest.cc:4255:0
    #11 0x2a4ece0 in RUN_ALL_TESTS testing/gtest/include/gtest/gtest.h:2237:46
    #12 0x2a4ece0 in Run base/test/test_suite.cc:271:0
    #13 0x2a7ceba in Run base/callback.h:85:12
    #14 0x2a7ceba in LaunchUnitTestsInternal base/test/launcher/unit_test_launcher.cc:211:0
    #15 0x2a7c6dd in LaunchUnitTests base/test/launcher/unit_test_launcher.cc:453:10
    #16 0x2a19673 in main base/test/run_all_base_unittests.cc:22:10
    #17 0x7f402d143f44 in __libc_start_main /build/eglibc-oGUzwX/eglibc-2.19/csu/libc-start.c:287:0

  Uninitialized value was created by a heap allocation
    #0 0x609452 in operator new(unsigned long) ??:?
    #1 0x21ec860 in TestBody base/trace_event/process_memory_dump_unittest.cc:98:7
    #2 0x2b2e7cc in HandleExceptionsInMethodIfSupported\u003Ctesting::Test, void> testing/gtest/src/gtest.cc:2458:12
    #3 0x2b2e7cc in Run testing/gtest/src/gtest.cc:2474:0
    #4 0x2b31151 in Run testing/gtest/src/gtest.cc:2656:11
    #5 0x2b326b9 in Run testing/gtest/src/gtest.cc:2774:28
    #6 0x2b52bcd in RunAllTests testing/gtest/src/gtest.cc:4647:43
    #7 0x2b51a3e in HandleExceptionsInMethodIfSupported\u003Ctesting::internal::UnitTestImpl, bool> testing/gtest/src/gtest.cc:2458:12
    #8 0x2b51a3e in Run testing/gtest/src/gtest.cc:4255:0
    #9 0x2a4ece0 in RUN_ALL_TESTS testing/gtest/include/gtest/gtest.h:2237:46
    #10 0x2a4ece0 in Run base/test/test_suite.cc:271:0
    #11 0x2a7ceba in Run base/callback.h:85:12
    #12 0x2a7ceba in LaunchUnitTestsInternal base/test/launcher/unit_test_launcher.cc:211:0
    #13 0x2a7c6dd in LaunchUnitTests base/test/launcher/unit_test_launcher.cc:453:10
    #14 0x2a19673 in main base/test/run_all_base_unittests.cc:22:10
    #15 0x7f402d143f44 in __libc_start_main /build/eglibc-oGUzwX/eglibc-2.19/csu/libc-start.c:287:0

SUMMARY: MemorySanitizer: use-of-uninitialized-value (/b/s/w/irAUiAI5/out/Release/base_unittests+0x291dd88)
 

Comment 1 by thakis@chromium.org, Feb 28 2017

Labels: Stability-Memory-MemorySanitizer

Comment 2 by thakis@chromium.org, Feb 28 2017

Owner: thakis@chromium.org
Status: Started (was: Untriaged)
Project Member

Comment 4 by bugdroid1@chromium.org, Feb 28 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d6c9b97e0d5402918f030e48e0528f9fe5649861

commit d6c9b97e0d5402918f030e48e0528f9fe5649861
Author: thakis <thakis@chromium.org>
Date: Tue Feb 28 22:11:19 2017

Fix ProcessMemoryDumpTest.TakeAllDumpsFrom under msan.

https://codereview.chromium.org/2582453002 broke this.

BUG= 697172 

Review-Url: https://codereview.chromium.org/2720093004
Cr-Commit-Position: refs/heads/master@{#453721}

[modify] https://crrev.com/d6c9b97e0d5402918f030e48e0528f9fe5649861/base/trace_event/memory_dump_session_state.cc

Status: Fixed (was: Started)
Nice!

Sign in to add a comment