[openssl-commits] Still Failing: openssl/openssl#10584 (master - bf846a6)

[Travis CI]

Build Update for openssl/openssl
-------------------------------------

Build: #10584
Status: Still Failing

Duration: 19 minutes and 55 seconds
Commit: bf846a6 (master)
Author: Matt Caswell
Message: Don't overwrite the alert value if there is no alert to send

The function tls_early_post_process_client_hello() was overwriting the
passed "al" parameter even if it was successful. The caller of that
function, tls_post_process_client_hello(), sets "al" to a sensible default
(HANDSHAKE_FAILURE), but this was being overwritten to be INTERNAL_ERROR.
The result is a "no shared cipher" error (and probably other similar errors)
were being reported back to the client with an incorrect INTERNAL_ERROR
alert.

Reviewed-by: Rich Salz <rsalz at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/3314)

View the changeset: https://github.com/openssl/openssl/compare/d91b7423af79...bf846a6d47a0

View the full build log and details: https://travis-ci.org/openssl/openssl/builds/226003159

--

You can configure recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mta.openssl.org/pipermail/openssl-commits/attachments/20170426/a1034e2e/attachment.html>