mirror of
git://source.winehq.org/git/wine.git
synced 2024-10-31 12:19:49 +00:00
wintrust/tests: Use proper return types when calling provider funcs (PVS-Studio).
Signed-off-by: Nikolay Sivov <nsivov@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>
This commit is contained in:
parent
186bed3d16
commit
9b9dd19297
1 changed files with 5 additions and 4 deletions
|
@ -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
|
||||
|
|
Loading…
Reference in a new issue