Module: wine Branch: master Commit: aa16c89b29a8ab0b53b31a2ccc8fa4e456918998 URL: http://source.winehq.org/git/wine.git/?a=commit;h=aa16c89b29a8ab0b53b31a2ccc...
Author: Marcus Meissner marcus@jet.franken.de Date: Fri Dec 31 12:00:22 2010 +0100
crypt32: NULL ptr could leak into function (Coverity).
---
dlls/crypt32/encode.c | 7 ++++++- 1 files changed, 6 insertions(+), 1 deletions(-)
diff --git a/dlls/crypt32/encode.c b/dlls/crypt32/encode.c index 5d6affe..f10c92e 100644 --- a/dlls/crypt32/encode.c +++ b/dlls/crypt32/encode.c @@ -4628,8 +4628,13 @@ BOOL WINAPI CryptEncodeObjectEx(DWORD dwCertEncodingType, LPCSTR lpszStructType, }
SetLastError(NOERROR); - if (dwFlags & CRYPT_ENCODE_ALLOC_FLAG && pvEncoded) + if (dwFlags & CRYPT_ENCODE_ALLOC_FLAG) { + if (!pvEncoded) { + SetLastError(ERROR_INVALID_PARAMETER); + return FALSE; + } *(BYTE **)pvEncoded = NULL; + } encodeFunc = CRYPT_GetBuiltinEncoder(dwCertEncodingType, lpszStructType); if (!encodeFunc) {