Changeset 64885 in vbox for trunk

Timestamp:

Dec 15, 2016 3:31:24 PM (8 years ago)

Author:

vboxsync

Message:

IPRT/ASN.1: Refactored array handling (SET OF, SEQUENCE OF) to use a pointer array instead of an object instance array. The old approach would move objects around in memory after they'd be initialized/decoded, making certain core optimziations involving pointers to object members impossible, as well as causing potentially causing trouble when modifying structures that takes down pointers after decoding. Fixed validation bug in rtCrX509Name_CheckSanityExtra where it didn't check that the RDNs had subitems but instead checked the parent twice (slight risk).

File:

• trunk/src/VBox/Main/src-server/CertificateImpl.cpp (modified) (2 diffs)

trunk/src/VBox/Main/src-server/CertificateImpl.cpp

@@ -222 +222 @@
     for (uint32_t i = 0; i < pName->cItems; i++)
     {
-        PCRTCRX509RELATIVEDISTINGUISHEDNAME pRdn = &pName->paItems[i];
+        PCRTCRX509RELATIVEDISTINGUISHEDNAME pRdn = pName->papItems[i];
         for (uint32_t j = 0; j < pRdn->cItems; j++)
         {
-            PCRTCRX509ATTRIBUTETYPEANDVALUE pComponent = &pRdn->paItems[j];
+            PCRTCRX509ATTRIBUTETYPEANDVALUE pComponent = pRdn->papItems[j];
             AssertContinue(pComponent->Value.enmType == RTASN1TYPE_STRING);
 
@@ -470 +470 @@
         for (uint32_t i = 0; i < a_pName->cItems; i++)
         {
-            PCRTCRX509RELATIVEDISTINGUISHEDNAME pRdn = &a_pName->paItems[i];
+            PCRTCRX509RELATIVEDISTINGUISHEDNAME pRdn = a_pName->papItems[i];
             for (uint32_t j = 0; j < pRdn->cItems; j++)
             {
-                PCRTCRX509ATTRIBUTETYPEANDVALUE pComponent = &pRdn->paItems[j];
+                PCRTCRX509ATTRIBUTETYPEANDVALUE pComponent = pRdn->papItems[j];
 
                 AssertReturn(pComponent->Value.enmType == RTASN1TYPE_STRING,