March 2013 - Wine-commits - list.winehq.org

Alexandre Julliard : winedbg: Use socklen_t where appropriate.
by Alexandre Julliard 06 Mar '13

06 Mar '13

Module: wine Branch: master Commit: 51947acd5abb07209a8929ab639c1016a733e828 URL: http://source.winehq.org/git/wine.git/?a=commit;h=51947acd5abb07209a8929ab6… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Mar 6 14:09:24 2013 +0100 winedbg: Use socklen_t where appropriate. --- programs/winedbg/gdbproxy.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/programs/winedbg/gdbproxy.c b/programs/winedbg/gdbproxy.c index 580fb88..66ff6a8 100644 --- a/programs/winedbg/gdbproxy.c +++ b/programs/winedbg/gdbproxy.c @@ -476,6 +476,7 @@ static inline DWORD64 cpu_register(CONTEXT* ctx, unsigned idx) default: fprintf(stderr, "got unexpected size: %u\n", (unsigned)cpu_register_map[idx].ctx_length); assert(0); + return 0; } } @@ -2363,7 +2364,7 @@ static BOOL gdb_startup(struct gdb_context* gdbctx, DEBUG_EVENT* de, unsigned fl { int sock; struct sockaddr_in s_addrs; - unsigned int s_len = sizeof(s_addrs); + socklen_t s_len = sizeof(s_addrs); struct pollfd pollfd; IMAGEHLP_MODULE64 imh_mod; BOOL ret = FALSE;

1 0

Francois Gouget : inetcomm: Make mimebody_create() static.
by Alexandre Julliard 05 Mar '13

05 Mar '13

Module: wine Branch: master Commit: d5a4621bd6cfc002b978534ae490ba059d0cb319 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d5a4621bd6cfc002b978534ae… Author: Francois Gouget <fgouget(a)free.fr> Date: Tue Mar 5 12:15:44 2013 +0100 inetcomm: Make mimebody_create() static. --- dlls/inetcomm/mimeole.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/inetcomm/mimeole.c b/dlls/inetcomm/mimeole.c index 4c61f72..38acd49 100644 --- a/dlls/inetcomm/mimeole.c +++ b/dlls/inetcomm/mimeole.c @@ -1094,7 +1094,7 @@ static HRESULT MimeBody_set_offsets(MimeBody *body, const BODYOFFSETS *offsets) #define FIRST_CUSTOM_PROP_ID 0x100 -MimeBody *mimebody_create(void) +static MimeBody *mimebody_create(void) { MimeBody *This; BODYOFFSETS body_offsets;

1 0

Piotr Caban : ieframe: Pass BSTR to handle_navigation_error.
by Alexandre Julliard 05 Mar '13

05 Mar '13

Module: wine Branch: master Commit: 402b10013f7082d8578e55dedb19b5e5ecfc0e69 URL: http://source.winehq.org/git/wine.git/?a=commit;h=402b10013f7082d8578e55ded… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Tue Mar 5 16:17:57 2013 +0100 ieframe: Pass BSTR to handle_navigation_error. --- dlls/ieframe/navigate.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/ieframe/navigate.c b/dlls/ieframe/navigate.c index 2601ba8..3a35acf 100644 --- a/dlls/ieframe/navigate.c +++ b/dlls/ieframe/navigate.c @@ -45,7 +45,7 @@ typedef struct { DocHost *doc_host; IBinding *binding; - LPWSTR url; + BSTR url; HGLOBAL post_data; BSTR headers; ULONG post_data_len; @@ -206,7 +206,7 @@ static ULONG WINAPI BindStatusCallback_Release(IBindStatusCallback *iface) if(This->post_data) GlobalFree(This->post_data); SysFreeString(This->headers); - heap_free(This->url); + SysFreeString(This->url); heap_free(This); } @@ -543,7 +543,7 @@ static BindStatusCallback *create_callback(DocHost *doc_host, LPCWSTR url, PBYTE ret->IHttpSecurity_iface.lpVtbl = &HttpSecurityVtbl; ret->ref = 1; - ret->url = heap_strdupW(url); + ret->url = SysAllocString(url); ret->post_data = NULL; ret->post_data_len = post_data_len; ret->headers = headers ? SysAllocString(headers) : NULL;

1 0

Piotr Caban : msvcrt: Added __pxcptinfoptrs tests.
by Alexandre Julliard 05 Mar '13

05 Mar '13

Module: wine Branch: master Commit: d989e92c3e2ffd666b14a0aa779b88b66fcf7602 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d989e92c3e2ffd666b14a0aa7… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Tue Mar 5 11:05:57 2013 +0100 msvcrt: Added __pxcptinfoptrs tests. --- dlls/msvcrt/tests/signal.c | 32 ++++++++++++++++++++++++++++++++ 1 files changed, 32 insertions(+), 0 deletions(-) diff --git a/dlls/msvcrt/tests/signal.c b/dlls/msvcrt/tests/signal.c index d33498b..ad6ea6e 100644 --- a/dlls/msvcrt/tests/signal.c +++ b/dlls/msvcrt/tests/signal.c @@ -26,6 +26,13 @@ static int test_value = 0; static void __cdecl sighandler(int signum) { + void **ret = __pxcptinfoptrs(); + + ok(ret != NULL, "ret = NULL\n"); + if(signum == SIGABRT) + ok(*ret == (void*)0xdeadbeef, "*ret = %p\n", *ret); + else if(signum == SIGSEGV) + ok(*ret == NULL, "*ret = %p\n", *ret); ++test_value; } @@ -42,7 +49,32 @@ static void test_signal(void) ok(test_value == 1, "SIGBREAK handler not invoked\n"); } +static void test___pxcptinfoptrs(void) +{ + void **ret = __pxcptinfoptrs(); + int res; + + ok(ret != NULL, "ret == NULL\n"); + ok(*ret == NULL, "*ret != NULL\n"); + + test_value = 0; + + *ret = (void*)0xdeadbeef; + signal(SIGSEGV, sighandler); + res = raise(SIGSEGV); + ok(res == 0, "failed to raise SIGSEGV\n"); + ok(*ret == (void*)0xdeadbeef, "*ret = %p\n", *ret); + + signal(SIGABRT, sighandler); + res = raise(SIGABRT); + ok(res == 0, "failed to raise SIGBREAK\n"); + ok(*ret == (void*)0xdeadbeef, "*ret = %p\n", *ret); + + ok(test_value == 2, "test_value = %d\n", test_value); +} + START_TEST(signal) { test_signal(); + test___pxcptinfoptrs(); }

1 0

Piotr Caban : msvcr90: Check more fields in _getptd tests.
by Alexandre Julliard 05 Mar '13

05 Mar '13

Module: wine Branch: master Commit: d36c6452ce011e3ac2d0c71464907388323bf45b URL: http://source.winehq.org/git/wine.git/?a=commit;h=d36c6452ce011e3ac2d0c7146… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Tue Mar 5 11:06:08 2013 +0100 msvcr90: Check more fields in _getptd tests. --- dlls/msvcr90/tests/msvcr90.c | 6 +++++- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/dlls/msvcr90/tests/msvcr90.c b/dlls/msvcr90/tests/msvcr90.c index f1ac539..fe3fa58 100644 --- a/dlls/msvcr90/tests/msvcr90.c +++ b/dlls/msvcr90/tests/msvcr90.c @@ -117,6 +117,7 @@ static int (__cdecl *p_feof)(FILE*); static int (__cdecl *p_ferror)(FILE*); static int (__cdecl *p_flsbuf)(int, FILE*); static unsigned long (__cdecl *p_byteswap_ulong)(unsigned long); +static void** (__cdecl *p__pxcptinfoptrs)(void); /* make sure we use the correct errno */ #undef errno @@ -302,6 +303,7 @@ static BOOL init(void) SET(p_ferror, "ferror"); SET(p_flsbuf, "_flsbuf"); SET(p_byteswap_ulong, "_byteswap_ulong"); + SET(p__pxcptinfoptrs, "__pxcptinfoptrs"); if (sizeof(void *) == 8) { SET(p_type_info_name_internal_method, "?_name_internal_method(a)type_info@@QEBAPEBDPEAU__type_info_node@@@Z"); @@ -1015,7 +1017,8 @@ struct __thread_data { struct tm *time_buffer; char *efcvt_buffer; int unk3[2]; - void *unk4[4]; + void *unk4[3]; + EXCEPTION_POINTERS *xcptinfo; int fpecode; pthreadmbcinfo mbcinfo; pthreadlocinfo locinfo; @@ -1060,6 +1063,7 @@ static void test_getptd(void) ok(p_wasctime(&time) == ptd->wasctime_buffer, "ptd->wasctime_buffer is incorrect\n"); ok(p_localtime64(&secs) == ptd->time_buffer, "ptd->time_buffer is incorrect\n"); ok(p_ecvt(3.12, 1, &dec, &sign) == ptd->efcvt_buffer, "ptd->efcvt_buffer is incorrect\n"); + ok(p__pxcptinfoptrs() == (void**)&ptd->xcptinfo, "ptd->xcptinfo is incorrect\n"); ok(p_fpecode() == &ptd->fpecode, "ptd->fpecode is incorrect\n"); mbcinfo = ptd->mbcinfo; locinfo = ptd->locinfo;

1 0

Hans Leidekker : winhttp: Use symbolic constants for HTTP status codes.
by Alexandre Julliard 05 Mar '13

05 Mar '13

Module: wine Branch: master Commit: 58e7d39c7fae38766013453eb690fa5c6456bfe7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=58e7d39c7fae38766013453eb… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Tue Mar 5 12:14:25 2013 +0100 winhttp: Use symbolic constants for HTTP status codes. --- dlls/winhttp/request.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/winhttp/request.c b/dlls/winhttp/request.c index 301478d..9915a43 100644 --- a/dlls/winhttp/request.c +++ b/dlls/winhttp/request.c @@ -1422,12 +1422,12 @@ static BOOL handle_authorization( request_t *request, DWORD status ) switch (status) { - case 401: + case HTTP_STATUS_DENIED: target = WINHTTP_AUTH_TARGET_SERVER; level = WINHTTP_QUERY_WWW_AUTHENTICATE; break; - case 407: + case HTTP_STATUS_PROXY_AUTH_REQ: target = WINHTTP_AUTH_TARGET_PROXY; level = WINHTTP_QUERY_PROXY_AUTHENTICATE; break; @@ -1886,7 +1886,7 @@ static BOOL receive_response( request_t *request, BOOL async ) send_request( request, NULL, 0, NULL, 0, 0, 0, FALSE ); /* recurse synchronously */ continue; } - else if (status == 401 || status == 407) + else if (status == HTTP_STATUS_DENIED || status == HTTP_STATUS_PROXY_AUTH_REQ) { if (request->hdr.disable_flags & WINHTTP_DISABLE_AUTHENTICATION) break;

1 0

Hans Leidekker : winhttp: Add a helper to retrieve the redirect URL.
by Alexandre Julliard 05 Mar '13

05 Mar '13

Module: wine Branch: master Commit: e56cb72cd336044088309dd9b4c1803acfc5244e URL: http://source.winehq.org/git/wine.git/?a=commit;h=e56cb72cd336044088309dd9b… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Tue Mar 5 12:14:50 2013 +0100 winhttp: Add a helper to retrieve the redirect URL. --- dlls/winhttp/request.c | 27 +++++++++++++++++++-------- 1 files changed, 19 insertions(+), 8 deletions(-) diff --git a/dlls/winhttp/request.c b/dlls/winhttp/request.c index 9915a43..12b1f25 100644 --- a/dlls/winhttp/request.c +++ b/dlls/winhttp/request.c @@ -1574,28 +1574,39 @@ end: return TRUE; } +static WCHAR *get_redirect_url( request_t *request, DWORD *len ) +{ + DWORD size; + WCHAR *ret; + + query_headers( request, WINHTTP_QUERY_LOCATION, NULL, NULL, &size, NULL ); + if (get_last_error() != ERROR_INSUFFICIENT_BUFFER) return FALSE; + if (!(ret = heap_alloc( size ))) return NULL; + *len = size / sizeof(WCHAR); + if (query_headers( request, WINHTTP_QUERY_LOCATION, NULL, ret, &size, NULL )) return ret; + heap_free( ret ); + return NULL; +} + static BOOL handle_redirect( request_t *request, DWORD status ) { BOOL ret = FALSE; - DWORD size, len; + DWORD len; URL_COMPONENTS uc; connect_t *connect = request->connect; INTERNET_PORT port; - WCHAR *hostname = NULL, *location = NULL; + WCHAR *hostname = NULL, *location; int index; - size = 0; - query_headers( request, WINHTTP_QUERY_LOCATION, NULL, NULL, &size, NULL ); - if (!(location = heap_alloc( size ))) return FALSE; - if (!query_headers( request, WINHTTP_QUERY_LOCATION, NULL, location, &size, NULL )) goto end; + if (!(location = get_redirect_url( request, &len ))) return FALSE; - send_callback( &request->hdr, WINHTTP_CALLBACK_STATUS_REDIRECT, location, size / sizeof(WCHAR) + 1 ); + send_callback( &request->hdr, WINHTTP_CALLBACK_STATUS_REDIRECT, location, len + 1 ); memset( &uc, 0, sizeof(uc) ); uc.dwStructSize = sizeof(uc); uc.dwSchemeLength = uc.dwHostNameLength = uc.dwUrlPathLength = uc.dwExtraInfoLength = ~0u; - if (!WinHttpCrackUrl( location, size / sizeof(WCHAR), 0, &uc )) /* assume relative redirect */ + if (!WinHttpCrackUrl( location, len, 0, &uc )) /* assume relative redirect */ { WCHAR *path, *p;

1 0

Hans Leidekker : winhttp: Implement WINHTTP_OPTION_REDIRECT_POLICY_DISALLOW_HTTPS_TO_HTTP.
by Alexandre Julliard 05 Mar '13

05 Mar '13

Module: wine Branch: master Commit: 418d2ce5232a48b909084a0a93b2daeff8e8ebfc URL: http://source.winehq.org/git/wine.git/?a=commit;h=418d2ce5232a48b909084a0a9… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Tue Mar 5 12:15:22 2013 +0100 winhttp: Implement WINHTTP_OPTION_REDIRECT_POLICY_DISALLOW_HTTPS_TO_HTTP. --- dlls/winhttp/request.c | 218 ++++++++++++++++++++++++----------------------- 1 files changed, 111 insertions(+), 107 deletions(-) diff --git a/dlls/winhttp/request.c b/dlls/winhttp/request.c index 12b1f25..c07c613 100644 --- a/dlls/winhttp/request.c +++ b/dlls/winhttp/request.c @@ -1574,112 +1574,6 @@ end: return TRUE; } -static WCHAR *get_redirect_url( request_t *request, DWORD *len ) -{ - DWORD size; - WCHAR *ret; - - query_headers( request, WINHTTP_QUERY_LOCATION, NULL, NULL, &size, NULL ); - if (get_last_error() != ERROR_INSUFFICIENT_BUFFER) return FALSE; - if (!(ret = heap_alloc( size ))) return NULL; - *len = size / sizeof(WCHAR); - if (query_headers( request, WINHTTP_QUERY_LOCATION, NULL, ret, &size, NULL )) return ret; - heap_free( ret ); - return NULL; -} - -static BOOL handle_redirect( request_t *request, DWORD status ) -{ - BOOL ret = FALSE; - DWORD len; - URL_COMPONENTS uc; - connect_t *connect = request->connect; - INTERNET_PORT port; - WCHAR *hostname = NULL, *location; - int index; - - if (!(location = get_redirect_url( request, &len ))) return FALSE; - - send_callback( &request->hdr, WINHTTP_CALLBACK_STATUS_REDIRECT, location, len + 1 ); - - memset( &uc, 0, sizeof(uc) ); - uc.dwStructSize = sizeof(uc); - uc.dwSchemeLength = uc.dwHostNameLength = uc.dwUrlPathLength = uc.dwExtraInfoLength = ~0u; - - if (!WinHttpCrackUrl( location, len, 0, &uc )) /* assume relative redirect */ - { - WCHAR *path, *p; - - len = strlenW( location ) + 1; - if (location[0] != '/') len++; - if (!(p = path = heap_alloc( len * sizeof(WCHAR) ))) goto end; - - if (location[0] != '/') *p++ = '/'; - strcpyW( p, location ); - - heap_free( request->path ); - request->path = path; - } - else - { - if (uc.nScheme == INTERNET_SCHEME_HTTP && request->hdr.flags & WINHTTP_FLAG_SECURE) - { - TRACE("redirect from secure page to non-secure page\n"); - request->hdr.flags &= ~WINHTTP_FLAG_SECURE; - } - else if (uc.nScheme == INTERNET_SCHEME_HTTPS && !(request->hdr.flags & WINHTTP_FLAG_SECURE)) - { - TRACE("redirect from non-secure page to secure page\n"); - request->hdr.flags |= WINHTTP_FLAG_SECURE; - } - - len = uc.dwHostNameLength; - if (!(hostname = heap_alloc( (len + 1) * sizeof(WCHAR) ))) goto end; - memcpy( hostname, uc.lpszHostName, len * sizeof(WCHAR) ); - hostname[len] = 0; - - port = uc.nPort ? uc.nPort : (uc.nScheme == INTERNET_SCHEME_HTTPS ? 443 : 80); - if (strcmpiW( connect->hostname, hostname ) || connect->serverport != port) - { - heap_free( connect->hostname ); - connect->hostname = hostname; - connect->hostport = port; - if (!(ret = set_server_for_hostname( connect, hostname, port ))) goto end; - - netconn_close( &request->netconn ); - if (!(ret = netconn_init( &request->netconn ))) goto end; - } - if (!(ret = add_host_header( request, WINHTTP_ADDREQ_FLAG_REPLACE ))) goto end; - if (!(ret = open_connection( request ))) goto end; - - heap_free( request->path ); - request->path = NULL; - if (uc.dwUrlPathLength) - { - len = uc.dwUrlPathLength + uc.dwExtraInfoLength; - if (!(request->path = heap_alloc( (len + 1) * sizeof(WCHAR) ))) goto end; - strcpyW( request->path, uc.lpszUrlPath ); - } - else request->path = strdupW( slashW ); - } - - /* remove content-type/length headers */ - if ((index = get_header_index( request, attr_content_type, 0, TRUE )) >= 0) delete_header( request, index ); - if ((index = get_header_index( request, attr_content_length, 0, TRUE )) >= 0 ) delete_header( request, index ); - - if (status != HTTP_STATUS_REDIRECT_KEEP_VERB && !strcmpW( request->verb, postW )) - { - heap_free( request->verb ); - request->verb = strdupW( getW ); - } - ret = TRUE; - -end: - if (!ret) heap_free( hostname ); - heap_free( location ); - return ret; -} - static BOOL receive_data( request_t *request, void *buffer, DWORD size, DWORD *read, BOOL async ) { DWORD to_read; @@ -1862,6 +1756,117 @@ static void record_cookies( request_t *request ) } } +static WCHAR *get_redirect_url( request_t *request, DWORD *len ) +{ + DWORD size; + WCHAR *ret; + + query_headers( request, WINHTTP_QUERY_LOCATION, NULL, NULL, &size, NULL ); + if (get_last_error() != ERROR_INSUFFICIENT_BUFFER) return FALSE; + if (!(ret = heap_alloc( size ))) return NULL; + *len = size / sizeof(WCHAR); + if (query_headers( request, WINHTTP_QUERY_LOCATION, NULL, ret, &size, NULL )) return ret; + heap_free( ret ); + return NULL; +} + +static BOOL handle_redirect( request_t *request, DWORD status ) +{ + BOOL ret = FALSE; + DWORD len, len_url; + URL_COMPONENTS uc; + connect_t *connect = request->connect; + INTERNET_PORT port; + WCHAR *hostname = NULL, *location; + int index; + + if (!(location = get_redirect_url( request, &len_url ))) return FALSE; + + memset( &uc, 0, sizeof(uc) ); + uc.dwStructSize = sizeof(uc); + uc.dwSchemeLength = uc.dwHostNameLength = uc.dwUrlPathLength = uc.dwExtraInfoLength = ~0u; + + if (!WinHttpCrackUrl( location, len_url, 0, &uc )) /* assume relative redirect */ + { + WCHAR *path, *p; + + len = strlenW( location ) + 1; + if (location[0] != '/') len++; + if (!(p = path = heap_alloc( len * sizeof(WCHAR) ))) goto end; + + if (location[0] != '/') *p++ = '/'; + strcpyW( p, location ); + + heap_free( request->path ); + request->path = path; + + drain_content( request ); + send_callback( &request->hdr, WINHTTP_CALLBACK_STATUS_REDIRECT, location, len_url + 1 ); + } + else + { + if (uc.nScheme == INTERNET_SCHEME_HTTP && request->hdr.flags & WINHTTP_FLAG_SECURE) + { + if (request->hdr.redirect_policy == WINHTTP_OPTION_REDIRECT_POLICY_DISALLOW_HTTPS_TO_HTTP) goto end; + TRACE("redirect from secure page to non-secure page\n"); + request->hdr.flags &= ~WINHTTP_FLAG_SECURE; + } + else if (uc.nScheme == INTERNET_SCHEME_HTTPS && !(request->hdr.flags & WINHTTP_FLAG_SECURE)) + { + TRACE("redirect from non-secure page to secure page\n"); + request->hdr.flags |= WINHTTP_FLAG_SECURE; + } + + drain_content( request ); + send_callback( &request->hdr, WINHTTP_CALLBACK_STATUS_REDIRECT, location, len_url + 1 ); + + len = uc.dwHostNameLength; + if (!(hostname = heap_alloc( (len + 1) * sizeof(WCHAR) ))) goto end; + memcpy( hostname, uc.lpszHostName, len * sizeof(WCHAR) ); + hostname[len] = 0; + + port = uc.nPort ? uc.nPort : (uc.nScheme == INTERNET_SCHEME_HTTPS ? 443 : 80); + if (strcmpiW( connect->hostname, hostname ) || connect->serverport != port) + { + heap_free( connect->hostname ); + connect->hostname = hostname; + connect->hostport = port; + if (!(ret = set_server_for_hostname( connect, hostname, port ))) goto end; + + netconn_close( &request->netconn ); + if (!(ret = netconn_init( &request->netconn ))) goto end; + } + if (!(ret = add_host_header( request, WINHTTP_ADDREQ_FLAG_REPLACE ))) goto end; + if (!(ret = open_connection( request ))) goto end; + + heap_free( request->path ); + request->path = NULL; + if (uc.dwUrlPathLength) + { + len = uc.dwUrlPathLength + uc.dwExtraInfoLength; + if (!(request->path = heap_alloc( (len + 1) * sizeof(WCHAR) ))) goto end; + strcpyW( request->path, uc.lpszUrlPath ); + } + else request->path = strdupW( slashW ); + } + + /* remove content-type/length headers */ + if ((index = get_header_index( request, attr_content_type, 0, TRUE )) >= 0) delete_header( request, index ); + if ((index = get_header_index( request, attr_content_length, 0, TRUE )) >= 0 ) delete_header( request, index ); + + if (status != HTTP_STATUS_REDIRECT_KEEP_VERB && !strcmpW( request->verb, postW )) + { + heap_free( request->verb ); + request->verb = strdupW( getW ); + } + ret = TRUE; + +end: + if (!ret) heap_free( hostname ); + heap_free( location ); + return ret; +} + static BOOL receive_response( request_t *request, BOOL async ) { BOOL ret; @@ -1890,7 +1895,6 @@ static BOOL receive_response( request_t *request, BOOL async ) if (request->hdr.disable_flags & WINHTTP_DISABLE_REDIRECTS || request->hdr.redirect_policy == WINHTTP_OPTION_REDIRECT_POLICY_NEVER) break; - drain_content( request ); if (!(ret = handle_redirect( request, status ))) break; clear_response_headers( request );

1 0

Hans Leidekker : winhttp: Make sure to clear response headers from a previous request.
by Alexandre Julliard 05 Mar '13

05 Mar '13

Module: wine Branch: master Commit: 71874a38aac99d992c0569a4e45d5570de8d3599 URL: http://source.winehq.org/git/wine.git/?a=commit;h=71874a38aac99d992c0569a4e… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Tue Mar 5 12:15:54 2013 +0100 winhttp: Make sure to clear response headers from a previous request. --- dlls/winhttp/request.c | 32 ++++++++++++++++---------------- 1 files changed, 16 insertions(+), 16 deletions(-) diff --git a/dlls/winhttp/request.c b/dlls/winhttp/request.c index c07c613..cd583e7 100644 --- a/dlls/winhttp/request.c +++ b/dlls/winhttp/request.c @@ -1038,6 +1038,20 @@ static BOOL add_host_header( request_t *request, DWORD modifier ) return ret; } +static void clear_response_headers( request_t *request ) +{ + unsigned int i; + + for (i = 0; i < request->num_headers; i++) + { + if (!request->headers[i].field) continue; + if (!request->headers[i].value) continue; + if (request->headers[i].is_request) continue; + delete_header( request, i ); + i--; + } +} + static BOOL send_request( request_t *request, LPCWSTR headers, DWORD headers_len, LPVOID optional, DWORD optional_len, DWORD total_len, DWORD_PTR context, BOOL async ) { @@ -1053,6 +1067,8 @@ static BOOL send_request( request_t *request, LPCWSTR headers, DWORD headers_len int bytes_sent; DWORD len, i, flags; + clear_response_headers( request ); + flags = WINHTTP_ADDREQ_FLAG_ADD|WINHTTP_ADDREQ_FLAG_COALESCE_WITH_COMMA; for (i = 0; i < request->num_accept_types; i++) { @@ -1457,20 +1473,6 @@ static BOOL handle_authorization( request_t *request, DWORD status ) return FALSE; } -static void clear_response_headers( request_t *request ) -{ - unsigned int i; - - for (i = 0; i < request->num_headers; i++) - { - if (!request->headers[i].field) continue; - if (!request->headers[i].value) continue; - if (request->headers[i].is_request) continue; - delete_header( request, i ); - i--; - } -} - #define MAX_REPLY_LEN 1460 #define INITIAL_HEADER_BUFFER_LEN 512 @@ -1897,7 +1899,6 @@ static BOOL receive_response( request_t *request, BOOL async ) if (!(ret = handle_redirect( request, status ))) break; - clear_response_headers( request ); send_request( request, NULL, 0, NULL, 0, 0, 0, FALSE ); /* recurse synchronously */ continue; } @@ -1911,7 +1912,6 @@ static BOOL receive_response( request_t *request, BOOL async ) ret = TRUE; break; } - clear_response_headers( request ); send_request( request, NULL, 0, NULL, 0, 0, 0, FALSE ); continue; }

1 0

Tijl Coosemans : winebuild: Don' t register cleanup function when --save-temps is passed after --output.
by Alexandre Julliard 05 Mar '13

05 Mar '13

Module: wine Branch: master Commit: afd7dc5947abe6d8387d2e4a38f397366c81a8ac URL: http://source.winehq.org/git/wine.git/?a=commit;h=afd7dc5947abe6d8387d2e4a3… Author: Tijl Coosemans <tijl(a)coosemans.org> Date: Mon Mar 4 17:21:16 2013 +0100 winebuild: Don't register cleanup function when --save-temps is passed after --output. --- tools/winebuild/build.h | 2 +- tools/winebuild/main.c | 4 +++- tools/winebuild/utils.c | 4 +--- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/tools/winebuild/build.h b/tools/winebuild/build.h index 3fbf89d..359428f 100644 --- a/tools/winebuild/build.h +++ b/tools/winebuild/build.h @@ -250,6 +250,7 @@ extern char *find_tool( const char *name, const char * const *names ); extern struct strarray *get_as_command(void); extern struct strarray *get_ld_command(void); extern const char *get_nm_command(void); +extern void cleanup_tmp_files(void); extern char *get_temp_file_name( const char *prefix, const char *suffix ); extern void output_standard_file_header(void); extern FILE *open_input_file( const char *srcdir, const char *name ); @@ -347,7 +348,6 @@ extern int nb_errors; extern int display_warnings; extern int kill_at; extern int verbose; -extern int save_temps; extern int link_ext_symbols; extern int force_pointer_size; extern int unwind_tables; diff --git a/tools/winebuild/main.c b/tools/winebuild/main.c index 594b31b..39dbe62 100644 --- a/tools/winebuild/main.c +++ b/tools/winebuild/main.c @@ -44,7 +44,6 @@ int nb_errors = 0; int display_warnings = 0; int kill_at = 0; int verbose = 0; -int save_temps = 0; int link_ext_symbols = 0; int force_pointer_size = 0; int unwind_tables = 0; @@ -358,6 +357,7 @@ static char **parse_options( int argc, char **argv, DLLSPEC *spec ) { char *p; int optc; + int save_temps = 0; while ((optc = getopt_long( argc, argv, short_options, long_options, NULL )) != -1) { @@ -515,6 +515,8 @@ static char **parse_options( int argc, char **argv, DLLSPEC *spec ) } } + if (!save_temps) atexit( cleanup_tmp_files ); + if (spec->file_name && !strchr( spec->file_name, '.' )) strcat( spec->file_name, exec_mode == MODE_EXE ? ".exe" : ".dll" ); init_dll_name( spec ); diff --git a/tools/winebuild/utils.c b/tools/winebuild/utils.c index 262ff3a..bab2b85 100644 --- a/tools/winebuild/utils.c +++ b/tools/winebuild/utils.c @@ -64,7 +64,7 @@ static const struct }; /* atexit handler to clean tmp files */ -static void cleanup_tmp_files(void) +void cleanup_tmp_files(void) { unsigned int i; for (i = 0; i < nb_tmp_files; i++) if (tmp_files[i]) unlink( tmp_files[i] ); @@ -461,8 +461,6 @@ char *get_temp_file_name( const char *prefix, const char *suffix ) const char *ext, *basename; int fd; - if (!nb_tmp_files && !save_temps) atexit( cleanup_tmp_files ); - if (!prefix || !prefix[0]) prefix = "winebuild"; if (!suffix) suffix = ""; if ((basename = strrchr( prefix, '/' ))) basename++;

1 0