wininet: Fix a memory leak.

Hans Leidekker hans at codeweavers.com
Tue Feb 23 06:03:37 CST 2010


Found by Valgrind.
---
 dlls/wininet/http.c |    5 ++++-
 1 files changed, 4 insertions(+), 1 deletions(-)

diff --git a/dlls/wininet/http.c b/dlls/wininet/http.c
index 62c3953..ab7e1da 100644
--- a/dlls/wininet/http.c
+++ b/dlls/wininet/http.c
@@ -4431,7 +4431,7 @@ static INT HTTP_GetResponseHeaders(http_request_t *lpwhr, BOOL clear)
     char bufferA[MAX_REPLY_LEN];
     LPWSTR status_code = NULL, status_text = NULL;
     DWORD cchMaxRawHeaders = 1024;
-    LPWSTR lpszRawHeaders = HeapAlloc(GetProcessHeap(), 0, (cchMaxRawHeaders+1)*sizeof(WCHAR));
+    LPWSTR lpszRawHeaders = NULL;
     LPWSTR temp;
     DWORD cchRawHeaders = 0;
     BOOL codeHundred = FALSE;
@@ -4507,6 +4507,9 @@ static INT HTTP_GetResponseHeaders(http_request_t *lpwhr, BOOL clear)
     *(status_text-1) = ' ';
 
     /* regenerate raw headers */
+    lpszRawHeaders = HeapAlloc(GetProcessHeap(), 0, (cchMaxRawHeaders + 1) * sizeof(WCHAR));
+    if (!lpszRawHeaders) goto lend;
+
     while (cchRawHeaders + buflen + strlenW(szCrLf) > cchMaxRawHeaders)
         cchMaxRawHeaders *= 2;
     temp = HeapReAlloc(GetProcessHeap(), 0, lpszRawHeaders, (cchMaxRawHeaders+1)*sizeof(WCHAR));
-- 
1.6.3.3




More information about the wine-patches mailing list