[openssl-commits] [openssl] master update

Dr. Stephen Henson steve at openssl.org
Mon May 2 21:48:43 UTC 2016


The branch master has been updated
       via  05aef4bbdbc18e7b9490512cdee41e8a608bcc0e (commit)
      from  5a1f853b9870fc32a2e8960763d62209108ff519 (commit)


- Log -----------------------------------------------------------------
commit 05aef4bbdbc18e7b9490512cdee41e8a608bcc0e
Author: Dr. Stephen Henson <steve at openssl.org>
Date:   Mon May 2 17:33:50 2016 +0100

    Fix i2d_X509_AUX: pp can be NULL.
    
    Reported by David Benjamin
    
    Reviewed-by: Emilia Käsper <emilia at openssl.org>

-----------------------------------------------------------------------

Summary of changes:
 crypto/x509/x_x509.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/crypto/x509/x_x509.c b/crypto/x509/x_x509.c
index 11e758b..043ab07 100644
--- a/crypto/x509/x_x509.c
+++ b/crypto/x509/x_x509.c
@@ -184,14 +184,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;


More information about the openssl-commits mailing list