mapi32: initialize session pointer to zero in MAPILogonEx (2nd resend)

Louis. Lenders xerox_xerox2000 at yahoo.co.uk
Wed Oct 8 15:07:21 CDT 2008





diff --git a/dlls/mapi32/mapi32_main.c b/dlls/mapi32/mapi32_main.c
index 936d435..5aace9a 100644
--- a/dlls/mapi32/mapi32_main.c
+++ b/dlls/mapi32/mapi32_main.c
@@ -109,6 +109,7 @@ HRESULT WINAPI MAPILogonEx(ULONG_PTR uiparam,
LPWSTR profile,
{
     FIXME("(0x%08lx %s %p 0x%08x %p) Stub\n", uiparam,
           debugstr_w(profile), password, flags, session);
+    *session=0;
     return SUCCESS_SUCCESS;
}

>Are you guaranteed that session is not NULL? 

No, no garantueed, but i looked up some code-samples by google, and as
far as i can tell, the current stub for MAPILogonEx will pass back garbage anyway to the application, so my thought was this patch would be rather harmless anyway. 

>A test case would help.
I guess you're right about that, thanks anyway for the comments.
--Juan


      
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://www.winehq.org/pipermail/wine-devel/attachments/20081008/af8f408d/attachment.htm 


More information about the wine-devel mailing list