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

Issue 616564 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
please use my google.com address
Closed: Sep 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 3
Type: Bug



Sign in to add a comment

DrMemory: uninitialized read in NamedPlatformChannelPair::PassClientHandleFromParentProcess

Project Member Reported by osh...@chromium.org, Jun 1 2016

Issue description

Started on:
https://build.chromium.org/p/chromium.memory.fyi/builders/Windows%20Content%20Browser%20%28DrMemory%20full%29%20%286%29/builds/3767

Probably r396766 (https://codereview.chromium.org/2013733002)? amistry@, can you look into this?

UNINITIALIZED READ: reading 0x03213340-0x03213348 8 byte(s) within 0x03213340-0x0321334c
# 0 system call NtFsControlFile FILE_PIPE_WAIT_FOR_BUFFER Timeout+NameLength
# 1 KERNELBASE.dll!WaitNamedPipeW                                             +0x2c6    (0x75b883cc <KERNELBASE.dll+0x83cc>)
# 2 mojo_system_impl.dll!mojo::edk::NamedPlatformChannelPair::PassClientHandleFromParentProcess [mojo\edk\embedder\named_platform_channel_pair_win.cc:90]
# 3 content.dll!content::`anonymous namespace'::InitializeMojoIPCChannel       [content\child\child_thread_impl.cc:240]
# 4 content.dll!content::ChildThreadImpl::Init                                 [content\child\child_thread_impl.cc:399]
# 5 content.dll!content::ChildThreadImpl::ChildThreadImpl                      [content\child\child_thread_impl.cc:340]
# 6 content.dll!content::UtilityThreadImpl::UtilityThreadImpl                  [content\utility\utility_thread_impl.cc:42]
# 7 content.dll!content::UtilityMain                                           [content\utility\utility_main.cc:40]
# 8 content.dll!content::RunNamedProcessTypeMain                               [content\app\content_main_runner.cc:420]
# 9 content.dll!content::ContentMainRunnerImpl::Run                            [content\app\content_main_runner.cc:787]
#10 content.dll!content::ContentMain                                           [content\app\content_main.cc:20]
#11 content::LaunchTests                                                       [content\public\test\test_launcher.cc:523]
#12 main                                                                       [content\test\content_test_launcher.cc:131]
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jun 1 2016

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

commit 50be322e3fd263e430bdbdc08fd8c51c8f00f88f
Author: oshima <oshima@chromium.org>
Date: Wed Jun 01 20:11:15 2016

Suppress UNINIT READ in NamedPlatformChannelPair::PassClientHandleFromParentProcess

BUG= 616564 
TBR=amistry@chromium.org

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

[modify] https://crrev.com/50be322e3fd263e430bdbdc08fd8c51c8f00f88f/tools/valgrind/drmemory/suppressions_full.txt

This looks like a DrMemory bug. The memory it's accessing is an internal windows data structure which I have no control over.
Cc: sa...@chromium.org roc...@chromium.org
Owner: ----
Status: Available (was: Assigned)
Cc: -roc...@chromium.org
Owner: roc...@chromium.org
Status: Assigned (was: Available)
Making sure this doesn't get lost without an owner

Comment 6 by roc...@chromium.org, Sep 27 2016

Status: WontFix (was: Assigned)
Given that it's a bug in DrMemory, the bug has been fixed, and we're probably going to stop running DrMemory, I don't think there's anything to do here.

Sign in to add a comment