diff --git a/keymaster/3.0/vts/functional/keymaster_hidl_hal_test.cpp b/keymaster/3.0/vts/functional/keymaster_hidl_hal_test.cpp index 3a181a96b7..7cdf253be8 100644 --- a/keymaster/3.0/vts/functional/keymaster_hidl_hal_test.cpp +++ b/keymaster/3.0/vts/functional/keymaster_hidl_hal_test.cpp @@ -20,6 +20,7 @@ #include #include +#include #include #include @@ -322,11 +323,11 @@ bool verify_chain(const hidl_vec>& chain) { char* cert_sub = X509_NAME_oneline(X509_get_subject_name(key_cert.get()), nullptr, 0); EXPECT_STREQ("/CN=Android Keystore Key", cert_sub) << "Cert " << i << " has wrong subject. (Possibly b/38394614)"; - free(cert_sub); + OPENSSL_free(cert_sub); } - free(cert_issuer); - free(signer_subj); + OPENSSL_free(cert_issuer); + OPENSSL_free(signer_subj); if (dump_Attestations) std::cout << bin2hex(chain[i]) << std::endl; } diff --git a/keymaster/4.0/vts/functional/keymaster_hidl_hal_test.cpp b/keymaster/4.0/vts/functional/keymaster_hidl_hal_test.cpp index 854c716881..450b3eb4e5 100644 --- a/keymaster/4.0/vts/functional/keymaster_hidl_hal_test.cpp +++ b/keymaster/4.0/vts/functional/keymaster_hidl_hal_test.cpp @@ -20,6 +20,7 @@ #include #include +#include #include #include @@ -208,11 +209,11 @@ bool verify_chain(const hidl_vec>& chain) { char* cert_sub = X509_NAME_oneline(X509_get_subject_name(key_cert.get()), nullptr, 0); EXPECT_STREQ("/CN=Android Keystore Key", cert_sub) << "Cert " << i << " has wrong subject."; - free(cert_sub); + OPENSSL_free(cert_sub); } - free(cert_issuer); - free(signer_subj); + OPENSSL_free(cert_issuer); + OPENSSL_free(signer_subj); if (dump_Attestations) std::cout << bin2hex(chain[i]) << std::endl; }