Fix i2d_X509_AUX: pp can be NULL.
authorDr. Stephen Henson <steve@openssl.org>
Mon, 2 May 2016 16:33:50 +0000 (17:33 +0100)
committerDr. Stephen Henson <steve@openssl.org>
Mon, 2 May 2016 21:49:32 +0000 (22:49 +0100)
Reported by David Benjamin

Reviewed-by: Emilia Käsper <emilia@openssl.org>
(cherry picked from commit 05aef4bbdbc18e7b9490512cdee41e8a608bcc0e)

crypto/asn1/x_x509.c

index ccdf6dfe139a136d827e5e26affb2a9f970a3bc9..e31e1e750d9db0325074763d888773e3a5a541ee 100644 (file)
@@ -202,14 +202,15 @@ X509 *d2i_X509_AUX(X509 **a, const unsigned char **pp, long length)
 int i2d_X509_AUX(X509 *a, unsigned char **pp)
 {
     int length, tmplen;
-    unsigned char *start = *pp;
+    unsigned char *start = pp != NULL ? *pp : NULL;
     length = i2d_X509(a, pp);
     if (length < 0 || a == NULL)
         return length;
 
     tmplen = i2d_X509_CERT_AUX(a->aux, pp);
     if (tmplen < 0) {
-        *pp = start;
+        if (start != NULL)
+            *pp = start;
         return tmplen;
     }
     length += tmplen;