New issue
Advanced search Search tips

Issue 594657 link

Starred by 1 user

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

Uninitialized read in courgette::adjustment_method_2::AssignmentProblem::Declassify

Project Member Reported by reillyg@chromium.org, Mar 14 2016

Issue description

Dr. Memory has detected a family of uninitialized read errors in courgette::adjustment_method_2::AssignmentProblem::Declassify:

https://build.chromium.org/p/chromium.memory.fyi/builders/Windows%20Unit%20%28DrMemory%20full%29%20%285%29/builds/6817

UNINITIALIZED READ: reading register edx
# 0 std::_Tree<>::_Eqrange<>                                                   [c:\b\depot_tools\win_toolchain\vs_files\391bbf1220d3edcd3cc3fccdb56224181e3b13a7\vc\include\xtree:2037]
# 1 courgette::adjustment_method_2::AssignmentProblem::Declassify              [courgette\adjustment_method_2.cc:903]
# 2 courgette::adjustment_method_2::AssignmentProblem::AssignOne               [courgette\adjustment_method_2.cc:1134]
# 3 courgette::adjustment_method_2::AssignmentProblem::FindAndAssignBestLeader [courgette\adjustment_method_2.cc:1194]
# 4 courgette::adjustment_method_2::AssignmentProblem::Solve                   [courgette\adjustment_method_2.cc:825]
# 5 courgette::adjustment_method_2::Adjuster::Solve                            [courgette\adjustment_method_2.cc:1275]
# 6 courgette::adjustment_method_2::Adjuster::Finish                           [courgette\adjustment_method_2.cc:1245]
# 7 courgette::adjustment_method_2::Adjuster::Adjust                           [courgette\adjustment_method_2.cc:1234]
# 8 courgette::Adjust                                                          [courgette\adjustment_method.cc:687]
# 9 courgette::PatchGeneratorX86_32::Transform                                 [courgette\patch_generator_x86_32.h:100]
#10 courgette::GenerateEnsemblePatch                                           [courgette\ensemble_create.cc:345]
#11 EnsembleTest::TestEnsemble                                                 [courgette\ensemble_unittest.cc:43]
#12 EnsembleTest::Elf32Ensemble                                                [courgette\ensemble_unittest.cc:74]
#13 testing::internal::HandleExceptionsInMethodIfSupported<>                   [testing\gtest\src\gtest.cc:2458]
Note: @0:06:40.010 in thread 4360
Note: instruction: mov    0x0c(%edx) -> %ecx
Suppression (error hash=#0B556FD9A1F3BE7A#):
For more info on using suppressions see http://dev.chromium.org/developers/how-tos/using-drmemory#TOC-Suppressing-error-reports-from-the-
{
UNINITIALIZED READ
name=<insert_a_suppression_name_here>
*!std::_Tree<>::_Eqrange<>
*!courgette::adjustment_method_2::AssignmentProblem::Declassify
*!courgette::adjustment_method_2::AssignmentProblem::AssignOne
*!courgette::adjustment_method_2::AssignmentProblem::FindAndAssignBestLeader
*!courgette::adjustment_method_2::AssignmentProblem::Solve
*!courgette::adjustment_method_2::Adjuster::Solve
*!courgette::adjustment_method_2::Adjuster::Finish
*!courgette::adjustment_method_2::Adjuster::Adjust
*!courgette::Adjust
*!courgette::PatchGeneratorX86_32::Transform
*!courgette::GenerateEnsemblePatch
*!EnsembleTest::TestEnsemble
*!EnsembleTest::Elf32Ensemble
*!testing::internal::HandleExceptionsInMethodIfSupported<>
}
 
Project Member

Comment 1 by bugdroid1@chromium.org, Mar 14 2016

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

commit 5f3968f92ff9f809b089a6889349e223fed6a3ad
Author: reillyg <reillyg@chromium.org>
Date: Mon Mar 14 20:02:44 2016

Add suppression for Courgette issues under Dr. Memory.

BUG=594657
TBR=zhaoqin@chromium.org
NOTRY=true

Review URL: https://codereview.chromium.org/1799833002

Cr-Commit-Position: refs/heads/master@{#381051}

[modify] https://crrev.com/5f3968f92ff9f809b089a6889349e223fed6a3ad/tools/valgrind/drmemory/suppressions_full.txt

Sign in to add a comment