HTTP: Fix OCSP error path

If addition of a peer issuer certificate fails, the certs pointer would
be NULL when being passed to sk_X509_push() for peer issuer's issuer.
Fix this by skipping addition of issuer's issue if issuer addition
fails.

Signed-off-by: Jouni Malinen <jouni@qca.qualcomm.com>

diff --git a/src/utils/http_curl.c b/src/utils/http_curl.c
index 0c18269da519909e9336b2b5c2fc906e3ccd5884..3a042f9324a4688f35d96346b8b4645365f95261 100644 (file)
--- a/src/utils/http_curl.c
+++ b/src/utils/http_curl.c
@@ -1098,7 +1098,7 @@ static int ocsp_resp_cb(SSL *s, void *arg)
                                sk_X509_free(certs);
                                certs = NULL;
                        }
-                       if (ctx->peer_issuer_issuer) {
+                       if (certs && ctx->peer_issuer_issuer) {
                                cert = X509_dup(ctx->peer_issuer_issuer);
                                if (cert && !sk_X509_push(certs, cert)) {
                                        tls_show_errors(