Allocate correct number of pointers for szAcceptTypes, since we need

it NULL terminated. HeapReAlloc will modify the pointer, so store it there.

Allocate correct number of pointers for szAcceptTypes, since we need
8624ef4a · Marcus Meissner · Alexandre Julliard · 33d9a97b · 8624ef4a
Commit 8624ef4a authored Mar 22, 2004 by Marcus Meissner Committed by Alexandre Julliard Mar 22, 2004
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

http.c dlls/wininet/http.c +3 -1

No files found.
--- a/dlls/wininet/http.c
+++ b/dlls/wininet/http.c
@@ -338,7 +338,7 @@ HINTERNET WINAPI HttpOpenRequestW(HINTERNET hHttpSession,
    if (lpszAcceptTypes)
    {
        while (lpszAcceptTypes[acceptTypesCount]) { acceptTypesCount++; } /* find out how many there are */
-        szAcceptTypes = HeapAlloc(GetProcessHeap(), 0, sizeof(CHAR *) * acceptTypesCount);
+        szAcceptTypes = HeapAlloc(GetProcessHeap(), 0, sizeof(CHAR*)*(acceptTypesCount+1));
        acceptTypesCount = 0;
        while (lpszAcceptTypes[acceptTypesCount])
        {
@@ -351,6 +351,7 @@ HINTERNET WINAPI HttpOpenRequestW(HINTERNET hHttpSession,
                                -1, szAcceptTypes[acceptTypesCount], len, NULL, NULL);
            acceptTypesCount++;
        }
+	szAcceptTypes[acceptTypesCount] = NULL;
    }
    else szAcceptTypes = 0;

@@ -2164,6 +2165,7 @@ BOOL HTTP_ProcessHeader(LPWININETHTTPREQA lpwhr, LPCSTR field, LPCSTR value, DWO
            lpsztmp = HeapReAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY,  lphttpHdr->lpszValue, len+1);
            if (lpsztmp)
            {
+		lphttpHdr->lpszValue = lpsztmp;
 		/* FIXME: Increment lphttpHdr->wCount. Perhaps lpszValue should be an array */
                if (ch > 0)
                {