[crypt32 1/5] Added some input validation

Paul Vriens Paul.Vriens at xs4all.nl
Fri Sep 8 06:18:46 CDT 2006


Hi,

first of 5. Tests will follow as well.

Changelog
   Added some input validation

Cheers,

Paul.
---
 dlls/crypt32/main.c |   11 ++++++++++-
 1 files changed, 10 insertions(+), 1 deletions(-)

diff --git a/dlls/crypt32/main.c b/dlls/crypt32/main.c
index 28f63c8..86b9cf4 100644
--- a/dlls/crypt32/main.c
+++ b/dlls/crypt32/main.c
@@ -181,8 +181,17 @@ BOOL WINAPI CryptSIPAddProvider(SIP_ADD_
 
     TRACE("%p\n", psNewProv);
 
-    if( !psNewProv )
+    if (!psNewProv ||
+        psNewProv->cbStruct != sizeof(SIP_ADD_NEWPROVIDER) ||
+        !psNewProv->pwszGetFuncName ||
+        !psNewProv->pwszPutFuncName ||
+        !psNewProv->pwszCreateFuncName ||
+        !psNewProv->pwszVerifyFuncName ||
+        !psNewProv->pwszRemoveFuncName)
+    {
+        SetLastError(ERROR_INVALID_PARAMETER);
         return FALSE;
+    }
 
     TRACE("%s %s %s %s\n",
           debugstr_guid( psNewProv->pgSubject ),
-- 
1.4.2




More information about the wine-patches mailing list