diff options
| author | relyea%netscape.com <devnull@localhost> | 2002-05-10 18:10:26 +0000 |
|---|---|---|
| committer | relyea%netscape.com <devnull@localhost> | 2002-05-10 18:10:26 +0000 |
| commit | f54c62bc9ebee6b72368efa750c673db0e5f5d4a (patch) | |
| tree | 4d5c06b344fc57ad4469975ae759bc4335c20c55 | |
| parent | 93f92b7a1054172174a2b17d06458294fe74d94c (diff) | |
| download | nss-hg-f54c62bc9ebee6b72368efa750c673db0e5f5d4a.tar.gz | |
When we force, we should preserve the NULL nickname semantics.
| -rw-r--r-- | security/nss/lib/pki/pki3hack.c | 24 |
1 files changed, 14 insertions, 10 deletions
diff --git a/security/nss/lib/pki/pki3hack.c b/security/nss/lib/pki/pki3hack.c index 779bed836..93d840506 100644 --- a/security/nss/lib/pki/pki3hack.c +++ b/security/nss/lib/pki/pki3hack.c @@ -586,7 +586,6 @@ fill_CERTCertificateFields(NSSCertificate *c, CERTCertificate *cc, PRBool forced int nicklen, tokenlen, len; NSSUTF8 *tokenName = NULL; char *nick; - nicklen = nssUTF8_Size(stanNick, &nssrv); if (instance && !PK11_IsInternal(instance->token->pk11slot)) { tokenName = nssToken_GetName(instance->token); tokenlen = nssUTF8_Size(tokenName, &nssrv); @@ -594,16 +593,21 @@ fill_CERTCertificateFields(NSSCertificate *c, CERTCertificate *cc, PRBool forced /* don't use token name for internal slot; 3.3 didn't */ tokenlen = 0; } - len = tokenlen + nicklen; - cc->nickname = PORT_ArenaAlloc(cc->arena, len); - nick = cc->nickname; - if (tokenName) { - memcpy(nick, tokenName, tokenlen-1); - nick += tokenlen-1; - *nick++ = ':'; + if (stanNick) { + nicklen = nssUTF8_Size(stanNick, &nssrv); + len = tokenlen + nicklen; + cc->nickname = PORT_ArenaAlloc(cc->arena, len); + nick = cc->nickname; + if (tokenName) { + memcpy(nick, tokenName, tokenlen-1); + nick += tokenlen-1; + *nick++ = ':'; + } + memcpy(nick, stanNick, nicklen-1); + cc->nickname[len-1] = '\0'; + } else { + cc->nickname = NULL; } - memcpy(nick, stanNick, nicklen-1); - cc->nickname[len-1] = '\0'; } if (context) { /* trust */ |