diff options
| author | Pauli <paul.dale@oracle.com> | 2017-07-14 10:08:38 +1000 |
|---|---|---|
| committer | Pauli <paul.dale@oracle.com> | 2017-07-17 06:59:45 +1000 |
| commit | 20e237c025acd9edb58df87f21c47753ac02b68f (patch) | |
| tree | 91ae1d270dca7b48a539938d05ee196d4aef192e /test/handshake_helper.c | |
| parent | d212b98b36d0874eb39c5ef08a18a43b9b006142 (diff) | |
| download | openssl-20e237c025acd9edb58df87f21c47753ac02b68f.zip openssl-20e237c025acd9edb58df87f21c47753ac02b68f.tar.gz openssl-20e237c025acd9edb58df87f21c47753ac02b68f.tar.bz2 | |
Address Coverity issues.
This patch addresses the use of uninitialised data raised in Coverity
issues 1414881 and 1414882.
Reviewed-by: Andy Polyakov <appro@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/3929)
Diffstat (limited to 'test/handshake_helper.c')
| -rw-r--r-- | test/handshake_helper.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/test/handshake_helper.c b/test/handshake_helper.c index 67facdc..6584e4d 100644 --- a/test/handshake_helper.c +++ b/test/handshake_helper.c @@ -1319,6 +1319,8 @@ static HANDSHAKE_RESULT *do_handshake_internal( memset(&client_ctx_data, 0, sizeof(client_ctx_data)); memset(&server, 0, sizeof(server)); memset(&client, 0, sizeof(client)); + memset(&server_ex_data, 0, sizeof(server_ex_data)); + memset(&client_ex_data, 0, sizeof(client_ex_data)); if (!configure_handshake_ctx(server_ctx, server2_ctx, client_ctx, test_ctx, extra, &server_ctx_data, @@ -1348,9 +1350,6 @@ static HANDSHAKE_RESULT *do_handshake_internal( goto err; } - memset(&server_ex_data, 0, sizeof(server_ex_data)); - memset(&client_ex_data, 0, sizeof(client_ex_data)); - ret->result = SSL_TEST_INTERNAL_ERROR; if (test_ctx->use_sctp) { |