From: Dr. Stephen Henson <steve@openssl.org>
Date: Sun, 15 Feb 2009 12:10:39 +0000 (+0000)
Subject: PR: 1422
X-Git-Tag: OpenSSL_0_9_8k~38
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=9a6401acdf50a5c7948ced371d20d47dc4ea1226;p=openssl

PR: 1422

Fix return value of X509_NAME_cmp() so it works with qsort/bsearch again.
---

diff --git a/crypto/x509/x509_cmp.c b/crypto/x509/x509_cmp.c
index e4c682fc44..2faf92514a 100644
--- a/crypto/x509/x509_cmp.c
+++ b/crypto/x509/x509_cmp.c
@@ -288,7 +288,8 @@ int X509_NAME_cmp(const X509_NAME *a, const X509_NAME *b)
 			if (!(nabit & STR_TYPE_CMP) ||
 				!(nbbit & STR_TYPE_CMP))
 				return j;
-			j = asn1_string_memcmp(na->value, nb->value);
+			if (!asn1_string_memcmp(na->value, nb->value))
+				j = 0;
 			}
 		else if (na->value->type == V_ASN1_PRINTABLESTRING)
 			j=nocase_spacenorm_cmp(na->value, nb->value);