New issue
Advanced search Search tips

Issue 784792 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: ----



Sign in to add a comment

browser_tests and interactive_ui_tests failing on MSan

Project Member Reported by jdoerrie@chromium.org, Nov 14 2017

Issue description

browser_tests and interactive_ui_tests failing on MSan

Builders failed on: 
- Linux ChromiumOS MSan Tests: 
  https://build.chromium.org/p/chromium.memory/builders/Linux%20ChromiumOS%20MSan%20Tests
- Linux MSan Tests: 
  https://build.chromium.org/p/chromium.memory/builders/Linux%20MSan%20Tests


Relevant line:
==26947==WARNING: MemorySanitizer: use-of-uninitialized-value
    #0 0xb5a2fc5 in BrowserProcessImpl::optimization_guide_service() chrome/browser/browser_process_impl.cc:951:7

Suspecting r513712 hence assigning to sophiechang@, but will attempt a fix myself.
 
Attempted Fix is in https://crrev.com/c/768381.
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 14 2017

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

commit bc6fe9d09c5c3286c8397ff3841a4aa994986739
Author: jdoerrie <jdoerrie@chromium.org>
Date: Tue Nov 14 10:46:28 2017

Fix uninitialized memory error in BrowserProcessImpl

TBR=sky@chromium.org

Bug:  784792 
Change-Id: Ib1cb7e3d38f887ce7e1411c06e40d4d58f677ea7
Reviewed-on: https://chromium-review.googlesource.com/768381
Reviewed-by: Jan Wilken Dörrie <jdoerrie@chromium.org>
Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org>
Cr-Commit-Position: refs/heads/master@{#516257}
[modify] https://crrev.com/bc6fe9d09c5c3286c8397ff3841a4aa994986739/chrome/browser/browser_process_impl.cc

Status: Fixed (was: Assigned)
Fix seems to have worked, will close this bug now.

Sign in to add a comment