diff options
author | Matt Caswell <matt@openssl.org> | 2018-09-07 15:17:34 +0100 |
---|---|---|
committer | Matt Caswell <matt@openssl.org> | 2018-09-07 18:24:59 +0100 |
commit | f01344cb5c6239af0d406f48d65362d0df9627b5 (patch) | |
tree | 54f51286ca17243df77c04cc82cb3ad5222a0228 /test/recipes | |
parent | 328a0547ad61d9e260fca73a280d2288714f2b92 (diff) |
Do not reset SNI data in SSL_do_handshake()
PR #3783 introduce coded to reset the server side SNI state in
SSL_do_handshake() to ensure any erroneous config time SNI changes are
cleared. Unfortunately SSL_do_handshake() can be called mid-handshake
multiple times so this is the wrong place to do this and can mean that
any SNI data is cleared later on in the handshake too.
Therefore move the code to a more appropriate place.
Fixes #7014
Reviewed-by: Tim Hudson <tjh@openssl.org>
Reviewed-by: Viktor Dukhovni <viktor@openssl.org>
Reviewed-by: Ben Kaduk <kaduk@mit.edu>
(Merged from https://github.com/openssl/openssl/pull/7149)
Diffstat (limited to 'test/recipes')
-rw-r--r-- | test/recipes/70-test_servername.t | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/test/recipes/70-test_servername.t b/test/recipes/70-test_servername.t index dae5d46baa..e6448aa88d 100644 --- a/test/recipes/70-test_servername.t +++ b/test/recipes/70-test_servername.t @@ -11,7 +11,7 @@ use strict; use warnings; use OpenSSL::Test::Simple; -use OpenSSL::Test; +use OpenSSL::Test qw/:DEFAULT srctop_file/; use OpenSSL::Test::Utils qw(alldisabled available_protocols); setup("test_servername"); @@ -19,4 +19,8 @@ setup("test_servername"); plan skip_all => "No TLS/SSL protocols are supported by this OpenSSL build" if alldisabled(grep { $_ ne "ssl3" } available_protocols("tls")); -simple_test("test_servername", "servername_test"); +plan tests => 1; + +ok(run(test(["servername_test", srctop_file("apps", "server.pem"), + srctop_file("apps", "server.pem")])), + "running servername_test"); |