From: Rich Salz <rsalz@openssl.org>
Date: Wed, 10 Sep 2014 15:43:45 +0000 (-0400)
Subject: RT2560: missing NULL check in ocsp_req_find_signer
X-Git-Tag: OpenSSL_1_0_2-beta3~34
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=a9d928a8b60c13d6a6475294deb56ba92fc6fbac;p=openssl

RT2560: missing NULL check in ocsp_req_find_signer

If we don't find a signer in the internal list, then fall
through and look at the internal list; don't just return NULL.

Reviewed-by: Dr. Stephen Henson <steve@openssl.org>
(cherry picked from commit b2aa38a980e9fbf158aafe487fb729c492b241fb)
---

diff --git a/crypto/ocsp/ocsp_vfy.c b/crypto/ocsp/ocsp_vfy.c
index 276718304d..fc0d4cc0f5 100644
--- a/crypto/ocsp/ocsp_vfy.c
+++ b/crypto/ocsp/ocsp_vfy.c
@@ -436,8 +436,11 @@ static int ocsp_req_find_signer(X509 **psigner, OCSP_REQUEST *req, X509_NAME *nm
 	if(!(flags & OCSP_NOINTERN))
 		{
 		signer = X509_find_by_subject(req->optionalSignature->certs, nm);
-		*psigner = signer;
-		return 1;
+		if (signer)
+			{
+			*psigner = signer;
+			return 1;
+			}
 		}
 
 	signer = X509_find_by_subject(certs, nm);