Change cert_GetCertificateEmailAddresses to return NULL rather than

a pointer to an empty string when a cert has no email addresses. Partial fix for bug 211540. Modified certdb/alg1485.c
sailfishos-mirror · Jul 9, 2003 · e5eb091 · e5eb091
1 parent d81d3d3
commit e5eb091
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/security/nss/lib/certdb/alg1485.c b/security/nss/lib/certdb/alg1485.c
@@ -1029,11 +1029,13 @@ cert_GetCertificateEmailAddresses(CERTCertificate *cert)
     }
     /* now copy superstring to cert's arena */
     finalLen = (pBuf - addrBuf) + 1;
-    pBuf = PORT_ArenaAlloc(cert->arena, finalLen);
-    if (pBuf) {
-    	PORT_Memcpy(pBuf, addrBuf, finalLen);
+    pBuf = NULL;
+    if (finalLen > 1) {
+	pBuf = PORT_ArenaAlloc(cert->arena, finalLen);
+	if (pBuf) {
+	    PORT_Memcpy(pBuf, addrBuf, finalLen);
+	}
     }
-
 loser:
     if (tmpArena)
 	PORT_FreeArena(tmpArena, PR_FALSE);