fixup! Implement wolfSSL_X509_STORE_set_default_paths

wolfSSL · Jan 3, 2025 · cd73893 · cd73893
1 parent 341ed32
commit cd73893
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/src/x509_str.c b/src/x509_str.c
@@ -1553,6 +1553,8 @@ static int X509StoreLoadFile(WOLFSSL_X509_STORE *str,
     static_buffer_init(&content, stackBuffer, FILE_BUFFER_SIZE);
 #endif
 
+    WOLFSSL_MSG_EX("X509StoreLoadFile: Loading file: %s", fname);
+
     ret = X509StoreReadFile(fname, &content, &contentLen, &type);
     if (ret != WOLFSSL_SUCCESS) {
         WOLFSSL_MSG("Failed to load file");
@@ -1682,8 +1684,10 @@ int wolfSSL_X509_STORE_set_default_paths(WOLFSSL_X509_STORE *str)
     char* certDir = NULL;
     char* certFile = NULL;
 
-    certDir = wc_strdup_ex(XGETENV("SSL_CERT_DIR"), DYNAMIC_TYPE_TMP_BUFFER);
+    WOLFSSL_ENTER("wolfSSL_X509_STORE_set_default_paths");
+
     certFile = wc_strdup_ex(XGETENV("SSL_CERT_FILE"), DYNAMIC_TYPE_TMP_BUFFER);
+    certDir = wc_strdup_ex(XGETENV("SSL_CERT_DIR"), DYNAMIC_TYPE_TMP_BUFFER);
 
     ret = wolfSSL_X509_STORE_load_locations(str, certFile, certDir);
 

diff --git a/tests/api.c b/tests/api.c
@@ -62369,6 +62369,14 @@ static int test_wolfSSL_X509_STORE_load_locations(void)
     ExpectIntEQ(X509_STORE_load_locations(store, NULL, certs_path),
         WOLFSSL_SUCCESS);
 
+#if defined(XGETENV) && !defined(NO_GETENV)
+    ExpectIntEQ(wolfSSL_CTX_UnloadCAs(ctx), WOLFSSL_SUCCESS);
+    /* Test with env vars */
+    ExpectIntEQ(setenv("SSL_CERT_FILE", client_pem_file, 1), 0);
+    ExpectIntEQ(setenv("SSL_CERT_DIR", certs_path, 1), 0);
+    ExpectIntEQ(X509_STORE_set_default_paths(store), WOLFSSL_SUCCESS);
+#endif
+
 #if defined(OPENSSL_EXTRA) || defined(DEBUG_WOLFSSL_VERBOSE)
     /* Clear nodes */
     ERR_clear_error();