diff options
author | Benjamin Kaduk <bkaduk@akamai.com> | 2017-12-07 14:23:35 -0600 |
---|---|---|
committer | Ben Kaduk <kaduk@mit.edu> | 2017-12-08 09:16:36 -0600 |
commit | 88e09fe79b909fbfe266e87a2731f96c197726fb (patch) | |
tree | 07d4825f81712f2cd052e60ff14209f3623484e0 /test/handshake_helper.c | |
parent | b6306d8049b04dca7fa738a86c892c43ba6a5fc4 (diff) |
Fix coverity nit in handshake_helper.c
There's no reason to wrap this call in TEST_true() if we're not
checking the return value of TEST_true() -- all of the surrounding
similar calls do not have the macro wrapping them.
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/4873)
Diffstat (limited to 'test/handshake_helper.c')
-rw-r--r-- | test/handshake_helper.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/handshake_helper.c b/test/handshake_helper.c index 78eaa012e2..0add973e50 100644 --- a/test/handshake_helper.c +++ b/test/handshake_helper.c @@ -497,8 +497,8 @@ static int configure_handshake_ctx(SSL_CTX *server_ctx, SSL_CTX *server2_ctx, case TLSEXT_max_fragment_length_2048: case TLSEXT_max_fragment_length_4096: case TLSEXT_max_fragment_length_DISABLED: - TEST_true(SSL_CTX_set_tlsext_max_fragment_length( - client_ctx, extra->client.max_fragment_len_mode)); + SSL_CTX_set_tlsext_max_fragment_length( + client_ctx, extra->client.max_fragment_len_mode); break; } |