Simplify ChangeCipherSpec code in runner. Not sure why it was expanded out like that. Change-Id: I6899dbd23130ed7196c45c2784330b2a4fe9bdba Reviewed-on: https://boringssl-review.googlesource.com/17666 Commit-Queue: Steven Valdez <svaldez@google.com> Reviewed-by: Steven Valdez <svaldez@google.com>
diff --git a/ssl/test/runner/conn.go b/ssl/test/runner/conn.go index 61fc9d3..9e72084 100644 --- a/ssl/test/runner/conn.go +++ b/ssl/test/runner/conn.go
@@ -1141,12 +1141,7 @@ if typ == recordTypeChangeCipherSpec { err = c.out.changeCipherSpec(c.config) if err != nil { - // Cannot call sendAlert directly, - // because we already hold c.out.Mutex. - c.tmp[0] = alertLevelError - c.tmp[1] = byte(err.(alert)) - c.writeRecord(recordTypeAlert, c.tmp[0:2]) - return n, c.out.setErrorLocked(&net.OpError{Op: "local error", Err: err}) + return n, c.sendAlertLocked(alertLevelError, err.(alert)) } } return
diff --git a/ssl/test/runner/dtls.go b/ssl/test/runner/dtls.go index d46b247..87b9975 100644 --- a/ssl/test/runner/dtls.go +++ b/ssl/test/runner/dtls.go
@@ -159,12 +159,7 @@ if typ == recordTypeChangeCipherSpec { err = c.out.changeCipherSpec(c.config) if err != nil { - // Cannot call sendAlert directly, - // because we already hold c.out.Mutex. - c.tmp[0] = alertLevelError - c.tmp[1] = byte(err.(alert)) - c.writeRecord(recordTypeAlert, c.tmp[0:2]) - return n, c.out.setErrorLocked(&net.OpError{Op: "local error", Err: err}) + return n, c.sendAlertLocked(alertLevelError, err.(alert)) } } return