diff options
author | alexei.volkov.bugs%sun.com <devnull@localhost> | 2006-05-13 00:33:14 +0000 |
---|---|---|
committer | alexei.volkov.bugs%sun.com <devnull@localhost> | 2006-05-13 00:33:14 +0000 |
commit | bfbd75e9d7da377cf7a34f5d171b414d53685c0e (patch) | |
tree | 28132e07a65dbce982a61fea057a29d985079edb | |
parent | 2f06a6ad6b50fc7cb1a5874a2a14b85dc342ef44 (diff) | |
download | nss-hg-bfbd75e9d7da377cf7a34f5d171b414d53685c0e.tar.gz |
Patch contributed by timeless@bemail.org
[Bug 336466] oom crash [@ CERT_DecodeAuthKeyID]. r=nelson
-rw-r--r-- | security/nss/lib/certdb/xauthkid.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/security/nss/lib/certdb/xauthkid.c b/security/nss/lib/certdb/xauthkid.c index 8d9df2123..8fb5a0122 100644 --- a/security/nss/lib/certdb/xauthkid.c +++ b/security/nss/lib/certdb/xauthkid.c @@ -119,10 +119,10 @@ CERT_DecodeAuthKeyID (PRArenaPool *arena, SECItem *encodedValue) do { mark = PORT_ArenaMark (arena); - value = (CERTAuthKeyID*)PORT_ArenaZAlloc (arena, sizeof (*value)); - value->DERAuthCertIssuer = NULL; + value = (CERTAuthKeyID*)PORT_ArenaZAlloc (arena, sizeof (*value)); if (value == NULL) break; + value->DERAuthCertIssuer = NULL; /* copy the DER into the arena, since Quick DER returns data that points into the DER input, which may get freed by the caller */ rv = SECITEM_CopyItem(arena, &newEncodedValue, encodedValue); |