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

Issue 599545 link

Starred by 1 user
Status: Fixed
Owner:
wfh@chromium.org
Closed: Apr 2016
Cc:
█ penny...@chromium.org
wfh@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug

Blocking:
issue 597137



Sign in to add a comment

ProcessMitigationsTest.CheckWin10ImageLoadNoLowLabel failing on Win10 10586

Project Member Reported by wfh@chromium.org, Mar 31 2016 
Version: efb0407ede1da3162449ba51c7743a5b5a477589
OS: Windows 10.0.10586

https://build.chromium.org/p/chromium.fyi/builders/Chromium%20Win%2010/builds/2742/steps/sbox_integration_tests/logs/stdio

[ RUN      ] ProcessMitigationsTest.CheckWin10ImageLoadNoLowLabelSuccess
processed file: C:\Users\CHROME~1\AppData\Local\Temp\scoped_dir7312_23507\lowIL_calc.exe
Successfully processed 1 files; Failed processing 0 files
e:\b\build\slave\chromium_win_10\build\src\sandbox\win\src\process_mitigations_test.cc(200): error: Value of: runner.RunTest(test.c_str())
  Actual: -441384953
Expected: (is_success_test ? sandbox::SBOX_TEST_SUCCEEDED : sandbox::SBOX_TEST_FAILED)
Which is: 632356865
[  FAILED  ] ProcessMitigationsTest.CheckWin10ImageLoadNoLowLabelSuccess (19 ms)
[107/108] ProcessMitigationsTest.CheckWin10ImageLoadNoLowLabelSuccess (19 ms)
[ RUN      ] ProcessMitigationsTest.CheckWin10ImageLoadNoLowLabelFailure
processed file: C:\Users\CHROME~1\AppData\Local\Temp\scoped_dir6276_3392\lowIL_calc.exe
Successfully processed 1 files; Failed processing 0 files
e:\b\build\slave\chromium_win_10\build\src\sandbox\win\src\process_mitigations_test.cc(200): error: Value of: runner.RunTest(test.c_str())
  Actual: -441384953
Expected: (is_success_test ? sandbox::SBOX_TEST_SUCCEEDED : sandbox::SBOX_TEST_FAILED)
Which is: -441384949
[  FAILED  ] ProcessMitigationsTest.CheckWin10ImageLoadNoLowLabelFailure (19 ms)

seems to have been broken since https://codereview.chromium.org/1826223004 so probably my fault.

Passes fine on the 10240 ESX bots e.g. look at https://chromium-swarm.appspot.com/user/task/2de604b0c25ac910

Comment 1 by wfh@chromium.org, Apr 1 2016

Blocking: 597137

Comment 2 by roberto....@gmail.com, Apr 1 2016 

[image: Mic Drop]
Project Member

Comment 3 by bugdroid1@chromium.org, Apr 1 2016 

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

commit f8a5a575c7e5109a353dcd71240c53fa484537aa
Author: wfh <wfh@chromium.org>
Date: Fri Apr 01 20:06:16 2016

Make second param for TestChildProcess optional.

This fixes a regression intoduced in crrev.com/1826223004 as the
TestChildProcess function is used by several other tests.

BUG= 599545 
TEST=sbox_integration_tests on Windows 10.0.10586

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

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

[modify] https://crrev.com/f8a5a575c7e5109a353dcd71240c53fa484537aa/sandbox/win/src/process_mitigations_test.cc

Comment 4 by wfh@chromium.org, Apr 1 2016

Status: Fixed (was: Started)
►

Sign in to add a comment