Nikolay Sivov : wintrust/tests: Use proper return types when calling provider funcs (PVS-Studio).
Alexandre Julliard
julliard at wine.codeweavers.com
Tue Oct 20 11:28:30 CDT 2015
Module: wine
Branch: master
Commit: 9b9dd19297e959f369f483f0118c37d44ba6f335
URL: http://source.winehq.org/git/wine.git/?a=commit;h=9b9dd19297e959f369f483f0118c37d44ba6f335
Author: Nikolay Sivov <nsivov at codeweavers.com>
Date: Tue Oct 20 16:54:39 2015 +0300
wintrust/tests: Use proper return types when calling provider funcs (PVS-Studio).
Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>
---
dlls/wintrust/tests/softpub.c | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/dlls/wintrust/tests/softpub.c b/dlls/wintrust/tests/softpub.c
index b1ac8a3..436de47 100644
--- a/dlls/wintrust/tests/softpub.c
+++ b/dlls/wintrust/tests/softpub.c
@@ -544,6 +544,7 @@ static void testCertTrust(SAFE_PROVIDER_FUNCTIONS *funcs, GUID *actionID)
CRYPT_PROVIDER_DATA data = { 0 };
CRYPT_PROVIDER_SGNR sgnr = { sizeof(sgnr), { 0 } };
HRESULT ret;
+ BOOL b;
if (!CertFreeCertificateChain_p)
{
@@ -563,8 +564,8 @@ static void testCertTrust(SAFE_PROVIDER_FUNCTIONS *funcs, GUID *actionID)
ok(data.padwTrustStepErrors[TRUSTERROR_STEP_FINAL_CERTPROV] ==
TRUST_E_NOSIGNATURE, "Expected TRUST_E_NOSIGNATURE, got %08x\n",
data.padwTrustStepErrors[TRUSTERROR_STEP_FINAL_CERTPROV]);
- ret = funcs->pfnAddSgnr2Chain(&data, FALSE, 0, &sgnr);
- if (ret)
+ b = funcs->pfnAddSgnr2Chain(&data, FALSE, 0, &sgnr);
+ if (b)
{
PCCERT_CONTEXT cert;
@@ -577,8 +578,8 @@ static void testCertTrust(SAFE_PROVIDER_FUNCTIONS *funcs, GUID *actionID)
{
WINTRUST_DATA wintrust_data = { 0 };
- ret = funcs->pfnAddCert2Chain(&data, 0, FALSE, 0, cert);
- ok(ret == S_FALSE, "Expected S_FALSE, got %08x\n", ret);
+ b = funcs->pfnAddCert2Chain(&data, 0, FALSE, 0, cert);
+ ok(b == TRUE, "Expected TRUE, got %d\n", b);
/* If pWintrustData isn't set, crashes attempting to access
* pWintrustData->fdwRevocationChecks
More information about the wine-cvs
mailing list