Bommels05/Mozilla
Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix for 232377 - assertion in vfychain with DSA cert. r=wtc, nelsonb

Browse Source 
git-svn-id: svn://10.0.0.236/trunk@151990 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
jpierre%netscape.com 2004-01-28 23:25:07 +00:00
parent a9d9ed95da
commit 13b5f81d25
1 changed files with 12 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
mozilla/security/nss/lib/cryptohi/seckey.c
View File

@ -514,10 +514,12 @@ seckey_UpdateCertPQGChain(CERTCertificate * subjectCert, int count)
(tag != SEC_OID_BOGUS_DSA_SIGNATURE_WITH_SHA1_DIGEST) &&
(tag != SEC_OID_SDN702_DSA_SIGNATURE) &&
(tag != SEC_OID_ANSIX962_EC_PUBLIC_KEY) ) {
return SECFailure;
rv = SECFailure;
goto loser;
}
} else {
return SECFailure; /* return failure if oid is NULL */
rv = SECFailure; /* return failure if oid is NULL */
goto loser;
}
@ -526,7 +528,10 @@ seckey_UpdateCertPQGChain(CERTCertificate * subjectCert, int count)
* pqg parameters with a recursive call to this same function. */
rv = seckey_UpdateCertPQGChain(issuerCert, count);
if (rv != SECSuccess) return rv;
if (rv != SECSuccess) {
rv = SECFailure;
goto loser;
}
/* ensure issuer has pqg parameters */
@ -544,6 +549,10 @@ seckey_UpdateCertPQGChain(CERTCertificate * subjectCert, int count)
&issuerSpki->algorithm.parameters);
}
loser:
if (issuerCert) {
CERT_DestroyCertificate(issuerCert);
}
return rv;
}