diff options
author | Matt Caswell <matt@openssl.org> | 2017-03-14 17:27:46 +0000 |
---|---|---|
committer | Matt Caswell <matt@openssl.org> | 2017-03-14 23:15:21 +0000 |
commit | 162e120711490cbd26f8608bf268a906c42e2027 (patch) | |
tree | b8fe6dae02829245db108ba4f2a0a8e662176b01 /test/ossl_shim | |
parent | e29d7cea332e58678640aaa84c6ddfaa0adce74f (diff) | |
download | openssl-new-162e120711490cbd26f8608bf268a906c42e2027.tar.gz |
SSL_get_peer_cert_chain() does not work after a resumption
After a resumption it is documented that SSL_get_peer_cert_chain() will
return NULL. In BoringSSL it still returns the chain. We don't support that
so we should update the shim to call SSL_get_peer_certificate() instead
when checking whether a peer certificate is available.
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/2942)
Diffstat (limited to 'test/ossl_shim')
-rw-r--r-- | test/ossl_shim/ossl_shim.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/ossl_shim/ossl_shim.cc b/test/ossl_shim/ossl_shim.cc index be589e36d1..9607e528d8 100644 --- a/test/ossl_shim/ossl_shim.cc +++ b/test/ossl_shim/ossl_shim.cc @@ -857,7 +857,7 @@ static bool CheckHandshakeProperties(SSL *ssl, bool is_resume) { return false; } } else if (!config->is_server || config->require_any_client_certificate) { - if (SSL_get_peer_cert_chain(ssl) == nullptr) { + if (SSL_get_peer_certificate(ssl) == nullptr) { fprintf(stderr, "Received no peer certificate but expected one.\n"); return false; } |