[PATCH] ntoskrnl.exe/tests: Don't call CertFreeCertificateContext on cleanup.

[Rémi Bernon]

It seems to crash the test on w864.

Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com>
---
 dlls/ntoskrnl.exe/tests/ntoskrnl.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/dlls/ntoskrnl.exe/tests/ntoskrnl.c b/dlls/ntoskrnl.exe/tests/ntoskrnl.c
index f9549b804bd..af26638e305 100644
--- a/dlls/ntoskrnl.exe/tests/ntoskrnl.c
+++ b/dlls/ntoskrnl.exe/tests/ntoskrnl.c
@@ -235,15 +235,11 @@ static void testsign_cleanup(struct testsign_context *ctx)
 
     ret = CertDeleteCertificateFromStore(ctx->root_cert);
     ok(ret, "Failed to remove certificate, error %u\n", GetLastError());
-    ret = CertFreeCertificateContext(ctx->root_cert);
-    ok(ret, "Failed to free certificate, error %u\n", GetLastError());
     ret = CertCloseStore(ctx->root_store, CERT_CLOSE_STORE_CHECK_FLAG);
     ok(ret, "Failed to close store, error %u\n", GetLastError());
 
     ret = CertDeleteCertificateFromStore(ctx->publisher_cert);
     ok(ret, "Failed to remove certificate, error %u\n", GetLastError());
-    ret = CertFreeCertificateContext(ctx->publisher_cert);
-    ok(ret, "Failed to free certificate, error %u\n", GetLastError());
     ret = CertCloseStore(ctx->publisher_store, CERT_CLOSE_STORE_CHECK_FLAG);
     ok(ret, "Failed to close store, error %u\n", GetLastError());
 
-- 
2.31.0

[Marvin]

Hi,

While running your changed tests, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?

Full results can be found at:
https://testbot.winehq.org/JobDetails.pl?Key=92576

Your paranoid android.


=== w8 (32 bit report) ===

ntoskrnl.exe:
driver.c:759: Test failed: got 0