diff options
author | nelsonb%netscape.com <devnull@localhost> | 2003-11-15 00:10:01 +0000 |
---|---|---|
committer | nelsonb%netscape.com <devnull@localhost> | 2003-11-15 00:10:01 +0000 |
commit | 7bc73da2b2f7e8138f9b831fc7f65ffd7de5632a (patch) | |
tree | a43a3ccb52ff89fcfba2ce8f49739f13061ce782 /security/nss/lib/pkcs7 | |
parent | c49e8312b2a9e29dd5c556301720a702206c0b93 (diff) | |
download | nss-hg-7bc73da2b2f7e8138f9b831fc7f65ffd7de5632a.tar.gz |
Detect empty emailAddr strings in CERTCertificates. Bugzilla bug 211540.
Modified Files:
cmd/dbck/dbck.c cmd/signtool/util.c lib/certdb/certdb.c
lib/certdb/stanpcertdb.c lib/pkcs7/p7decode.c lib/pki/certificate.c
lib/pki/pki3hack.c lib/smime/cmssiginfo.c lib/softoken/pkcs11u.c
Diffstat (limited to 'security/nss/lib/pkcs7')
-rw-r--r-- | security/nss/lib/pkcs7/p7decode.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/security/nss/lib/pkcs7/p7decode.c b/security/nss/lib/pkcs7/p7decode.c index 447502176..cba3ac9ef 100644 --- a/security/nss/lib/pkcs7/p7decode.c +++ b/security/nss/lib/pkcs7/p7decode.c @@ -1835,7 +1835,7 @@ savecert: * Only save the smime profile if we are checking an email message and * the cert has an email address in it. */ - if ( ( cert->emailAddr != NULL ) && + if ( cert->emailAddr && cert->emailAddr[0] && ( ( certusage == certUsageEmailSigner ) || ( certusage == certUsageEmailRecipient ) ) ) { SECItem *profile = NULL; @@ -2005,7 +2005,7 @@ sec_pkcs7_get_signer_cert_info(SEC_PKCS7ContentInfo *cinfo, int selector) container = CERT_GetCommonName (&signercert->subject); break; case sec_email_address: - if(signercert->emailAddr) { + if(signercert->emailAddr && signercert->emailAddr[0]) { container = PORT_Strdup(signercert->emailAddr); } else { container = NULL; |