kris/boringssl
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Don't swallow tool output on failure.

Browse Source 
If the test failed due to non-ASan reasons but ASan also had errors,
output those too.

Change-Id: Id908fe2a823c59255c6a9585dfaa894a4fcd9f59
Reviewed-on: https://boringssl-review.googlesource.com/11127
Reviewed-by: Adam Langley <agl@google.com>
This commit is contained in:
David Benjamin 2016-09-20 19:36:53 -04:00 committed by Adam Langley
parent 7a4aaa4ce7
commit 9aafb64849
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ssl/test/runner/runner.go
View File

@ -988,7 +988,7 @@ func runTest(test *testCase, shimPath string, mallocNumToFail int64) error {
panic("internal error") panic("internal error")
} }
return fmt.Errorf("%s: local error '%s', child error '%s', stdout:\n%s\nstderr:\n%s", msg, localError, childError, stdout, stderr) return fmt.Errorf("%s: local error '%s', child error '%s', stdout:\n%s\nstderr:\n%s\n%s", msg, localError, childError, stdout, stderr, extraStderr)
} }
if !*useValgrind && (len(extraStderr) > 0 || (!failed && len(stderr) > 0)) { if !*useValgrind && (len(extraStderr) > 0 || (!failed && len(stderr) > 0)) {