Skip to content

Commit ede266a

Browse files
committed
Fixes from review
1 parent 0f50c22 commit ede266a

1 file changed

Lines changed: 6 additions & 5 deletions

File tree

tests/api/test_ossl_x509.c

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1662,7 +1662,8 @@ int test_wolfssl_local_IsValidFQDN(void) {
16621662
test_cases[i].is_FQDN);
16631663
if (! EXPECT_SUCCESS()) {
16641664
fprintf(stderr, "wolfssl_local_IsValidFQDN() wrong result for "
1665-
"case %d \"%s\"\n", i, test_cases[i].str);
1665+
"case %d \"%s\"\n", i,
1666+
test_cases[i].str ? test_cases[i].str : "(null)");
16661667
break;
16671668
}
16681669
}
@@ -1709,12 +1710,12 @@ int test_wolfssl_local_IsValidFQDN(void) {
17091710
/* Verify that MatchDomainName() refuses to expand wildcards across IDNA
17101711
* A-labels (xn-- prefix) per RFC 6125 sec. 6.4.3 / RFC 9525 sec. 6.3.
17111712
*
1712-
* MatchDomainName() is WOLFSSL_LOCAL but visible to the test binary because
1713-
* tests link against the in-tree library. */
1713+
* MatchDomainName() is exposed for testing via the visibility mechanism
1714+
* declared in wolfssl/internal.h. */
17141715
int test_wolfSSL_MatchDomainName_idn(void)
17151716
{
17161717
EXPECT_DECLS;
1717-
#if !defined(NO_CERTS)
1718+
#if !defined(NO_ASN) && !defined(WOLFCRYPT_ONLY) && !defined(NO_CERTS)
17181719
static const struct {
17191720
const char* pattern;
17201721
const char* host;
@@ -1790,7 +1791,7 @@ int test_wolfSSL_MatchDomainName_idn(void)
17901791
break;
17911792
}
17921793
}
1793-
#endif /* !NO_CERTS */
1794+
#endif /* !NO_ASN && !WOLFCRYPT_ONLY && !NO_CERTS */
17941795
return EXPECT_RESULT();
17951796
}
17961797

0 commit comments

Comments
 (0)