Juan Lang : wintrust: Return error directly from SOFTPUB_OpenFile.
Alexandre Julliard
julliard at winehq.org
Thu Jan 28 11:15:34 CST 2010
Module: wine
Branch: master
Commit: fc1fbf61074fed98b1423cf501580ecb57d5b995
URL: http://source.winehq.org/git/wine.git/?a=commit;h=fc1fbf61074fed98b1423cf501580ecb57d5b995
Author: Juan Lang <juan.lang at gmail.com>
Date: Wed Jan 27 10:27:49 2010 -0800
wintrust: Return error directly from SOFTPUB_OpenFile.
---
dlls/wintrust/softpub.c | 18 ++++++++----------
1 files changed, 8 insertions(+), 10 deletions(-)
diff --git a/dlls/wintrust/softpub.c b/dlls/wintrust/softpub.c
index 967bb1a..1ff0599 100644
--- a/dlls/wintrust/softpub.c
+++ b/dlls/wintrust/softpub.c
@@ -75,9 +75,9 @@ HRESULT WINAPI DriverFinalPolicy(CRYPT_PROVIDER_DATA *data)
/* Assumes data->pWintrustData->u.pFile exists. Makes sure a file handle is
* open for the file.
*/
-static BOOL SOFTPUB_OpenFile(CRYPT_PROVIDER_DATA *data)
+static DWORD SOFTPUB_OpenFile(CRYPT_PROVIDER_DATA *data)
{
- BOOL ret = TRUE;
+ DWORD err = ERROR_SUCCESS;
/* PSDK implies that all values should be initialized to NULL, so callers
* typically have hFile as NULL rather than INVALID_HANDLE_VALUE. Check
@@ -92,13 +92,13 @@ static BOOL SOFTPUB_OpenFile(CRYPT_PROVIDER_DATA *data)
if (data->pWintrustData->u.pFile->hFile != INVALID_HANDLE_VALUE)
data->fOpenedFile = TRUE;
else
- ret = FALSE;
+ err = GetLastError();
}
- if (ret)
+ if (!err)
GetFileTime(data->pWintrustData->u.pFile->hFile, &data->sftSystemTime,
NULL, NULL);
- TRACE("returning %d\n", ret);
- return ret;
+ TRACE("returning %d\n", err);
+ return err;
}
/* Assumes data->pWintrustData->u.pFile exists. Sets data->pPDSip->gSubject to
@@ -332,11 +332,9 @@ static DWORD SOFTPUB_LoadFileMessage(CRYPT_PROVIDER_DATA *data)
err = ERROR_INVALID_PARAMETER;
goto error;
}
- if (!SOFTPUB_OpenFile(data))
- {
- err = GetLastError();
+ err = SOFTPUB_OpenFile(data);
+ if (err)
goto error;
- }
if (!SOFTPUB_GetFileSubject(data))
{
err = GetLastError();
More information about the wine-cvs
mailing list