The new ASN1 code automatically allocates
authorDr. Stephen Henson <steve@openssl.org>
Thu, 13 Jun 2002 00:43:59 +0000 (00:43 +0000)
committerDr. Stephen Henson <steve@openssl.org>
Thu, 13 Jun 2002 00:43:59 +0000 (00:43 +0000)
structures for fields that are not OPTIONAL.

However in the AUTHORITY_INFO_ACCESS case
the 'location' field was set to NULL in
the old code.

So in 0.9.7+ we should free up the field before
overwriting it in v2i_AUTHORITY_INFO_ACCESS.

crypto/x509v3/v3_info.c

index 7f17f3231d11402455f66ffb116ec38e015f3efa..e1cf01a9b46830ad2137809a6b0a36d6522437f5 100644 (file)
@@ -158,6 +158,7 @@ static AUTHORITY_INFO_ACCESS *v2i_AUTHORITY_INFO_ACCESS(X509V3_EXT_METHOD *metho
                objlen = ptmp - cnf->name;
                ctmp.name = ptmp + 1;
                ctmp.value = cnf->value;
+               GENERAL_NAME_free(acc->location);
                if(!(acc->location = v2i_GENERAL_NAME(method, ctx, &ctmp)))
                                                                 goto err; 
                if(!(objtmp = OPENSSL_malloc(objlen + 1))) {