Clear one more timeout when using gdb.

https://boringssl-review.googlesource.com/18605 got the other ones.

Change-Id: If00487a4dd8508496a31a0a565c965559e12879c
Reviewed-on: https://boringssl-review.googlesource.com/21264
Commit-Queue: David Benjamin <davidben@google.com>
Commit-Queue: Adam Langley <agl@google.com>
Reviewed-by: Adam Langley <agl@google.com>
CQ-Verified: CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org>
This commit is contained in:
David Benjamin 2017-10-06 13:10:45 -04:00 committed by CQ bot account: commit-bot@chromium.org
parent 4519a5a063
commit 861f28a624

View File

@ -1145,6 +1145,10 @@ func runTest(test *testCase, shimPath string, mallocNumToFail int64) error {
listener.Close()
listener = nil
var childErr error
if *useGDB {
childErr = <-waitChan
} else {
var shimKilledLock sync.Mutex
var shimKilled bool
waitTimeout := time.AfterFunc(*idleTimeout, func() {
@ -1153,13 +1157,15 @@ func runTest(test *testCase, shimPath string, mallocNumToFail int64) error {
shimKilledLock.Unlock()
shim.Process.Kill()
})
childErr := <-waitChan
childErr = <-waitChan
waitTimeout.Stop()
shimKilledLock.Lock()
if shimKilled && err == nil {
err = errors.New("timeout waiting for the shim to exit.")
}
shimKilledLock.Unlock()
}
var isValgrindError bool
if exitError, ok := childErr.(*exec.ExitError); ok {
switch exitError.Sys().(syscall.WaitStatus).ExitStatus() {