From 418cdc4df42b0ba60ae2cf4a124204c1d67676a4 Mon Sep 17 00:00:00 2001 From: David Benjamin Date: Wed, 9 May 2018 13:56:09 -0400 Subject: [PATCH] Use the right alert for bad CA lists. Bug: 245 Change-Id: I6bfaf2dbe4996219773742a88c401d6cfffe3a3d Reviewed-on: https://boringssl-review.googlesource.com/28284 Commit-Queue: David Benjamin Commit-Queue: Steven Valdez Reviewed-by: Steven Valdez CQ-Verified: CQ bot account: commit-bot@chromium.org --- ssl/ssl_cert.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ssl/ssl_cert.cc b/ssl/ssl_cert.cc index 730280c4..8b569556 100644 --- a/ssl/ssl_cert.cc +++ b/ssl/ssl_cert.cc @@ -663,7 +663,7 @@ UniquePtr ssl_parse_client_CA_list(SSL *ssl, } if (!ssl->ctx->x509_method->check_client_CA_list(ret.get())) { - *out_alert = SSL_AD_INTERNAL_ERROR; + *out_alert = SSL_AD_DECODE_ERROR; OPENSSL_PUT_ERROR(SSL, SSL_R_DECODE_ERROR); return nullptr; }