wintrust: Don't dereference a potentially NULL pointer.

ba78b548 · Juan Lang · Alexandre Julliard · e6a4a866 · ba78b548
Commit ba78b548 authored Dec 11, 2011 by Juan Lang Committed by Alexandre Julliard Dec 12, 2011
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

softpub.c dlls/wintrust/softpub.c +2 -1

No files found.
--- 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;