ReceiveTempSMS
/
Sweden-Number
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

wininet: Always set returned size in InternetGetCookieExA. 

Signed-off-by: Jacek Caban <jacek@codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard@winehq.org>
This commit is contained in:
Jacek Caban 2016-07-06 13:44:01 +02:00 committed by Alexandre Julliard
parent 8332d80665
commit b53b717dad
2 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
dlls/wininet/cookie.c
View File

@ -808,7 +808,7 @@ BOOL WINAPI InternetGetCookieExA(LPCSTR lpszUrl, LPCSTR lpszCookieName,
LPSTR lpCookieData, LPDWORD lpdwSize, DWORD flags, void *reserved) LPSTR lpCookieData, LPDWORD lpdwSize, DWORD flags, void *reserved)
{ {
WCHAR *url, *name; WCHAR *url, *name;
DWORD len, size; DWORD len, size = 0;
BOOL r; BOOL r;
TRACE("(%s %s %p %p(%u) %x %p)\n", debugstr_a(lpszUrl), debugstr_a(lpszCookieName), TRACE("(%s %s %p %p(%u) %x %p)\n", debugstr_a(lpszUrl), debugstr_a(lpszCookieName),
@ -841,12 +841,12 @@ BOOL WINAPI InternetGetCookieExA(LPCSTR lpszUrl, LPCSTR lpszCookieName,
r = FALSE; r = FALSE;
} }
} }
*lpdwSize = size;
} }
heap_free( szCookieData ); heap_free( szCookieData );
} }
} }
*lpdwSize = size;
heap_free( name ); heap_free( name );
heap_free( url ); heap_free( url );
return r; return r;

2
dlls/wininet/tests/internet.c
View File

@ -360,11 +360,13 @@ static void test_get_cookie(void)
DWORD len; DWORD len;
BOOL ret; BOOL ret;
len = 1024;
SetLastError(0xdeadbeef); SetLastError(0xdeadbeef);
ret = InternetGetCookieA("http://www.example.com", NULL, NULL, &len); ret = InternetGetCookieA("http://www.example.com", NULL, NULL, &len);
ok(!ret && GetLastError() == ERROR_NO_MORE_ITEMS, ok(!ret && GetLastError() == ERROR_NO_MORE_ITEMS,
"InternetGetCookie should have failed with %s and error %d\n", "InternetGetCookie should have failed with %s and error %d\n",
ret ? "TRUE" : "FALSE", GetLastError()); ret ? "TRUE" : "FALSE", GetLastError());
ok(!len, "len = %u\n", len);
} }