commit c9cf4bc815e08ee53e84da4b4c8300dad1d4d178
author Alex Gaynor <alex.gaynor@gmail.com> Sat Apr 08 09:33:17 2017 -0400
committer Rich Salz <rsalz@openssl.org> Mon May 22 08:03:22 2017 -0400
tree a07033e1ba83e554c99698818ab4b37da3ba3f8c
parent 2d8e9dbd2c62f29f777dd3002a39419d45a891dd

Update the test to assert that the SCT is from an X.509 extension

Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/3149)

test/ct_test.c

diff --git a/test/ct_test.c b/test/ct_test.c
index 583db44..6b36a43 100644
--- a/test/ct_test.c
+++ b/test/ct_test.c
@@ -262,6 +262,7 @@
 
     if (fixture.certificate_file != NULL) {
         int sct_extension_index;
+        int i;
         X509_EXTENSION *sct_extension = NULL;
 
         if (!TEST_ptr(cert = load_pem_cert(fixture.certs_dir,
@@ -289,18 +290,16 @@
                                                expected_sct_text))
                     goto end;
 
-            if (fixture.test_validity) {
-                int i;
+            scts = X509V3_EXT_d2i(sct_extension);
+            for (i = 0; i < sk_SCT_num(scts); ++i) {
+                SCT *sct_i = sk_SCT_value(scts, i);
 
-                scts = X509V3_EXT_d2i(sct_extension);
-                for (i = 0; i < sk_SCT_num(scts); ++i) {
-                    SCT *sct_i = sk_SCT_value(scts, i);
-
-                    if (!TEST_true(SCT_set_source(sct_i,
-                                                  SCT_SOURCE_X509V3_EXTENSION)))
-                        goto end;
+                if (!TEST_int_eq(SCT_get_source(sct_i), SCT_SOURCE_X509V3_EXTENSION)) {
+                    goto end;
                 }
+            }
 
+            if (fixture.test_validity) {
                 if (!assert_validity(fixture, scts, ct_policy_ctx))
                     goto end;
             }