From 154c2f2b37b1acff8f033787072fe7f7bed82fa5 Mon Sep 17 00:00:00 2001 From: David Benjamin Date: Sat, 5 Mar 2016 11:57:44 -0500 Subject: [PATCH] Add some missing return false lines to test_config.cc. Change-Id: I9540c931b6cdd4d65fa9ebfc52e1770d2174abd2 Reviewed-on: https://boringssl-review.googlesource.com/7330 Reviewed-by: Steven Valdez Reviewed-by: David Benjamin --- ssl/test/test_config.cc | 2 ++ 1 file changed, 2 insertions(+) diff --git a/ssl/test/test_config.cc b/ssl/test/test_config.cc index 1cf316d5..95c85a8d 100644 --- a/ssl/test/test_config.cc +++ b/ssl/test/test_config.cc @@ -178,12 +178,14 @@ bool ParseConfig(int argc, char **argv, TestConfig *out_config) { size_t len; if (!EVP_DecodedLength(&len, strlen(argv[i]))) { fprintf(stderr, "Invalid base64: %s\n", argv[i]); + return false; } std::unique_ptr decoded(new uint8_t[len]); if (!EVP_DecodeBase64(decoded.get(), &len, len, reinterpret_cast(argv[i]), strlen(argv[i]))) { fprintf(stderr, "Invalid base64: %s\n", argv[i]); + return false; } base64_field->assign(reinterpret_cast(decoded.get()), len); continue;