diff options
author | tangyiqun <tangyiqun@uniontech.com> | 2022-03-09 18:06:41 +0800 |
---|---|---|
committer | Tomas Mraz <tomas@openssl.org> | 2022-03-16 11:20:39 +0100 |
commit | 494ea32909f0a7f7ee0a6711b45b1c551c0ef9b1 (patch) | |
tree | b4d3d1e536fa4c2d7672b2d2468a91939e2f33a8 /fuzz | |
parent | b0aa42d0506b76e9b65828a41c9197032d6b0d4a (diff) |
check return value of functions that call BIO_new()
Reviewed-by: Todd Short <todd.short@me.com>
Reviewed-by: Paul Dale <pauli@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/17850)
(cherry picked from commit edba19760fa682ed095ca26ba89ba95530003bfe)
Diffstat (limited to 'fuzz')
-rw-r--r-- | fuzz/client.c | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/fuzz/client.c b/fuzz/client.c index 698ff0f669..1f8933eb22 100644 --- a/fuzz/client.c +++ b/fuzz/client.c @@ -55,7 +55,7 @@ int FuzzerInitialize(int *argc, char ***argv) int FuzzerTestOneInput(const uint8_t *buf, size_t len) { - SSL *client; + SSL *client = NULL; BIO *in; BIO *out; SSL_CTX *ctx; @@ -65,13 +65,23 @@ int FuzzerTestOneInput(const uint8_t *buf, size_t len) /* This only fuzzes the initial flow from the client so far. */ ctx = SSL_CTX_new(SSLv23_method()); + if (ctx == NULL) + goto end; client = SSL_new(ctx); + if (client == NULL) + goto end; OPENSSL_assert(SSL_set_min_proto_version(client, 0) == 1); OPENSSL_assert(SSL_set_cipher_list(client, "ALL:eNULL:@SECLEVEL=0") == 1); SSL_set_tlsext_host_name(client, "localhost"); in = BIO_new(BIO_s_mem()); + if (in == NULL) + goto end; out = BIO_new(BIO_s_mem()); + if (out == NULL) { + BIO_free(in); + goto end; + } SSL_set_bio(client, in, out); SSL_set_connect_state(client); OPENSSL_assert((size_t)BIO_write(in, buf, len) == len); @@ -84,6 +94,7 @@ int FuzzerTestOneInput(const uint8_t *buf, size_t len) } } } + end: SSL_free(client); ERR_clear_error(); SSL_CTX_free(ctx); |