Juan Lang : wintrust: Don't dereference a potentially NULL pointer.

Alexandre Julliard julliard at winehq.org
Mon Dec 12 12:25:53 CST 2011


Module: wine
Branch: master
Commit: ba78b54820b5a5cdb4cdcfb5556b35781a825fe6
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=ba78b54820b5a5cdb4cdcfb5556b35781a825fe6

Author: Juan Lang <juan.lang at gmail.com>
Date:   Sun Dec 11 08:36:55 2011 -0800

wintrust: Don't dereference a potentially NULL pointer.

---

 dlls/wintrust/softpub.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/dlls/wintrust/softpub.c b/dlls/wintrust/softpub.c
index 0f54c0b..358ba01 100644
--- a/dlls/wintrust/softpub.c
+++ b/dlls/wintrust/softpub.c
@@ -1207,7 +1207,8 @@ HRESULT WINAPI SoftpubCleanup(CRYPT_PROVIDER_DATA *data)
     CryptMsgClose(data->hMsg);
 
     if (data->fOpenedFile &&
-     data->pWintrustData->dwUnionChoice == WTD_CHOICE_FILE)
+     data->pWintrustData->dwUnionChoice == WTD_CHOICE_FILE &&
+     data->pWintrustData->u.pFile)
         CloseHandle(data->pWintrustData->u.pFile->hFile);
 
     return S_OK;




More information about the wine-cvs mailing list