wininet: Do not omit mandatory argument to WriteFile.

Thomas Faber thomas.faber at reactos.org
Sat Jul 12 04:19:34 CDT 2014


From a67623c1a6f7ab5251fd2b5b80706acde17855be Mon Sep 17 00:00:00 2001
From: Thomas Faber <thomas.faber at reactos.org>
Date: Sat, 12 Jul 2014 10:30:29 +0200
Subject: wininet: Do not omit mandatory argument to WriteFile.

---
 dlls/wininet/cookie.c | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/dlls/wininet/cookie.c b/dlls/wininet/cookie.c
index 8bf8953..7b49387 100644
--- a/dlls/wininet/cookie.c
+++ b/dlls/wininet/cookie.c
@@ -251,6 +251,7 @@ static BOOL save_persistent_cookie(cookie_domain *domain)
     BOOL do_save = FALSE;
     char buf[64], *dyn_buf;
     FILETIME time;
+    DWORD bytes_written;
 
     if (!create_cookie_url(domain->lpCookieDomain, domain->lpCookiePath, cookie_url, sizeof(cookie_url)/sizeof(cookie_url[0])))
         return FALSE;
@@ -289,31 +290,31 @@ static BOOL save_persistent_cookie(cookie_domain *domain)
             continue;
 
         dyn_buf = heap_strdupWtoA(cookie_container->lpCookieName);
-        if(!dyn_buf || !WriteFile(cookie_handle, dyn_buf, strlen(dyn_buf), NULL, NULL)) {
+        if(!dyn_buf || !WriteFile(cookie_handle, dyn_buf, strlen(dyn_buf), &bytes_written, NULL)) {
             heap_free(dyn_buf);
             do_save = FALSE;
             break;
         }
         heap_free(dyn_buf);
-        if(!WriteFile(cookie_handle, "\n", 1, NULL, NULL)) {
+        if(!WriteFile(cookie_handle, "\n", 1, &bytes_written, NULL)) {
             do_save = FALSE;
             break;
         }
 
         dyn_buf = heap_strdupWtoA(cookie_container->lpCookieData);
-        if(!dyn_buf || !WriteFile(cookie_handle, dyn_buf, strlen(dyn_buf), NULL, NULL)) {
+        if(!dyn_buf || !WriteFile(cookie_handle, dyn_buf, strlen(dyn_buf), &bytes_written, NULL)) {
             heap_free(dyn_buf);
             do_save = FALSE;
             break;
         }
         heap_free(dyn_buf);
-        if(!WriteFile(cookie_handle, "\n", 1, NULL, NULL)) {
+        if(!WriteFile(cookie_handle, "\n", 1, &bytes_written, NULL)) {
             do_save = FALSE;
             break;
         }
 
         dyn_buf = heap_strdupWtoA(domain->lpCookieDomain);
-        if(!dyn_buf || !WriteFile(cookie_handle, dyn_buf, strlen(dyn_buf), NULL, NULL)) {
+        if(!dyn_buf || !WriteFile(cookie_handle, dyn_buf, strlen(dyn_buf), &bytes_written, NULL)) {
             heap_free(dyn_buf);
             do_save = FALSE;
             break;
@@ -321,7 +322,7 @@ static BOOL save_persistent_cookie(cookie_domain *domain)
         heap_free(dyn_buf);
 
         dyn_buf = heap_strdupWtoA(domain->lpCookiePath);
-        if(!dyn_buf || !WriteFile(cookie_handle, dyn_buf, strlen(dyn_buf), NULL, NULL)) {
+        if(!dyn_buf || !WriteFile(cookie_handle, dyn_buf, strlen(dyn_buf), &bytes_written, NULL)) {
             heap_free(dyn_buf);
             do_save = FALSE;
             break;
@@ -331,7 +332,7 @@ static BOOL save_persistent_cookie(cookie_domain *domain)
         sprintf(buf, "\n%u\n%u\n%u\n%u\n%u\n*\n", cookie_container->flags,
                 cookie_container->expiry.dwLowDateTime, cookie_container->expiry.dwHighDateTime,
                 cookie_container->create.dwLowDateTime, cookie_container->create.dwHighDateTime);
-        if(!WriteFile(cookie_handle, buf, strlen(buf), NULL, NULL)) {
+        if(!WriteFile(cookie_handle, buf, strlen(buf), &bytes_written, NULL)) {
             do_save = FALSE;
             break;
         }
-- 
1.9.0.msysgit.0



More information about the wine-patches mailing list