commit 019842119d4e1d350a1b1d34da02a130f528d9a5
author Przemek Stekiel <przemyslaw.stekiel@mobica.com> Thu Feb 09 09:29:34 2023 +0100
committer Przemek Stekiel <przemyslaw.stekiel@mobica.com> Tue Apr 04 17:48:28 2023 +0200
tree bac810f0e9a78b1ead4580e41575c05d198875b3
parent 4f3e7b934e45b6a9db6c1f8e0cb66ea875c838c9

Adapt test for authority_key_id (parsing subject alt name)

Signed-off-by: Przemek Stekiel <przemyslaw.stekiel@mobica.com>

tests/suites/test_suite_x509parse.function

diff --git a/tests/suites/test_suite_x509parse.function b/tests/suites/test_suite_x509parse.function
index 6346a6c..c7d341d 100644
--- a/tests/suites/test_suite_x509parse.function
+++ b/tests/suites/test_suite_x509parse.function
@@ -1506,6 +1506,8 @@
     int bufferCounter = 0;
     size_t issuerCounter = 0;
     unsigned int result = 0;
+    mbedtls_x509_subject_alternative_name san = { 0 };
+    mbedtls_x509_name *pname = NULL;
 
     mbedtls_x509_crt_init(&crt);
 
@@ -1518,14 +1520,18 @@
 
         /* Issuer test */
         mbedtls_x509_sequence *issuerPtr = &crt.authority_key_id.authorityCertIssuer;
-        while (issuerPtr != NULL) {
-            /* First 9 bytes are always ASN1 coding related information that does not matter right now. Only the values are asserted */
-            for (issuerCounter = 9u; issuerCounter < issuerPtr->buf.len; issuerCounter++) {
+
+        TEST_ASSERT(mbedtls_x509_parse_subject_alt_name(&issuerPtr->buf, &san) == 0);
+
+        pname = &san.san.directory_name;
+
+        while (pname != NULL) {
+            for (issuerCounter = 0; issuerCounter < pname->val.len; issuerCounter++) {
                 result |=
-                    (authorityKeyId_issuer[bufferCounter++] != issuerPtr->buf.p[issuerCounter]);
+                    (authorityKeyId_issuer[bufferCounter++] != pname->val.p[issuerCounter]);
             }
             bufferCounter++; /* Skipping the slash */
-            issuerPtr = issuerPtr->next;
+            pname = pname->next;
         }
         TEST_ASSERT(result == 0);