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 <svaldez@google.com> Reviewed-by: David Benjamin <davidben@google.com>
This commit is contained in:
parent
1b0c438e1a
commit
154c2f2b37
@ -178,12 +178,14 @@ bool ParseConfig(int argc, char **argv, TestConfig *out_config) {
|
|||||||
size_t len;
|
size_t len;
|
||||||
if (!EVP_DecodedLength(&len, strlen(argv[i]))) {
|
if (!EVP_DecodedLength(&len, strlen(argv[i]))) {
|
||||||
fprintf(stderr, "Invalid base64: %s\n", argv[i]);
|
fprintf(stderr, "Invalid base64: %s\n", argv[i]);
|
||||||
|
return false;
|
||||||
}
|
}
|
||||||
std::unique_ptr<uint8_t[]> decoded(new uint8_t[len]);
|
std::unique_ptr<uint8_t[]> decoded(new uint8_t[len]);
|
||||||
if (!EVP_DecodeBase64(decoded.get(), &len, len,
|
if (!EVP_DecodeBase64(decoded.get(), &len, len,
|
||||||
reinterpret_cast<const uint8_t *>(argv[i]),
|
reinterpret_cast<const uint8_t *>(argv[i]),
|
||||||
strlen(argv[i]))) {
|
strlen(argv[i]))) {
|
||||||
fprintf(stderr, "Invalid base64: %s\n", argv[i]);
|
fprintf(stderr, "Invalid base64: %s\n", argv[i]);
|
||||||
|
return false;
|
||||||
}
|
}
|
||||||
base64_field->assign(reinterpret_cast<const char *>(decoded.get()), len);
|
base64_field->assign(reinterpret_cast<const char *>(decoded.get()), len);
|
||||||
continue;
|
continue;
|
||||||
|
Loading…
Reference in New Issue
Block a user