diff options
author | David Benjamin <davidben@google.com> | 2018-08-10 08:42:41 -0500 |
---|---|---|
committer | Adam Langley <agl@google.com> | 2018-08-10 14:44:43 +0000 |
commit | ddedf6d455e33d6a0b3bc40581700f3d252029d7 (patch) | |
tree | 046ae8505a06ea167625c054f9222be84fa8ed2c /fuzz | |
parent | 17dc94e87466ac8928096c4f45954989d585cfe5 (diff) | |
download | boringssl-ddedf6d455e33d6a0b3bc40581700f3d252029d7.zip boringssl-ddedf6d455e33d6a0b3bc40581700f3d252029d7.tar.gz boringssl-ddedf6d455e33d6a0b3bc40581700f3d252029d7.tar.bz2 |
Fix SSL_CTX_set1_sigalgs_list fuzzer.
SSL_CTX_set1_sigalgs_list wants a NUL-terminated string, so we need to use
GetString to give it one.
Bug: oss-fuzz:9808
Change-Id: Id7f676aa514c36de9dea900763db3cbbf5c79a4c
Reviewed-on: https://boringssl-review.googlesource.com/30804
Reviewed-by: Adam Langley <agl@google.com>
Diffstat (limited to 'fuzz')
-rw-r--r-- | fuzz/ssl_ctx_api.cc | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/fuzz/ssl_ctx_api.cc b/fuzz/ssl_ctx_api.cc index c050770..4b47734 100644 --- a/fuzz/ssl_ctx_api.cc +++ b/fuzz/ssl_ctx_api.cc @@ -479,7 +479,11 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t *buf, size_t len) { SSL_CTX_set1_sigalgs(ctx, (const int *)CBS_data(cbs), CBS_len(cbs) / 2); }, [](SSL_CTX *ctx, CBS *cbs) { - SSL_CTX_set1_sigalgs_list(ctx, (const char *) CBS_data(cbs)); + std::string sigalgs; + if (!GetString(&sigalgs, cbs)) { + return; + } + SSL_CTX_set1_sigalgs_list(ctx, sigalgs.c_str()); }, }; |