Commit 89688b473e for openssl.org
commit 89688b473e1769d0874a565d467cef4bb2642431
Author: Norbert Pocs <norbertp@openssl.org>
Date: Fri Aug 28 13:07:37 2026 +0200
fuzz/x509v3.c: drop a dead store of cert
Setting cert to NULL before assinging another X509_new() result
to it is unnecessary, as was pointed out by Coverity. Drop
the dead store.
Resolves: https://scan5.scan.coverity.com/#/project-view/65248/10222?selectedIssue=1700563
Complements: 9da88ee430a0 "Add X509V3_EXT_add_nconf fuzzer"
Signed-off-by: Norbert Pocs <norbertp@openssl.org>
Reviewed-by: Paul Dale <paul.dale@oracle.com>
Reviewed-by: Nikola Pajkovsky <nikolap@openssl.org>
Reviewed-by: Neil Horman <nhorman@openssl.org>
Reviewed-by: Eugene Syromiatnikov <esyr@openssl.org>
Merge-date: Mon Aug 31 05:19:12 2026
Merged-from: https://github.com/openssl/openssl/pull/32576
diff --git a/fuzz/x509v3.c b/fuzz/x509v3.c
index 83d30fef76..d44b958082 100644
--- a/fuzz/x509v3.c
+++ b/fuzz/x509v3.c
@@ -61,7 +61,6 @@ int FuzzerTestOneInput(const uint8_t *buf, size_t len)
X509V3_set_nconf(&ctx, conf);
X509V3_EXT_add_nconf(conf, &ctx, "default", cert);
X509_free(cert);
- cert = NULL;
}
cert = X509_new();