wine-commits May 2016

wine-commits@winehq.org

4 participants
542 discussions

Henri Verbeet : wined3d: Rename MAX_CONST_B to WINED3D_MAX_CONSTS_B.
by Alexandre Julliard 18 May '16

18 May '16

Module: wine Branch: master Commit: 076697e31ab75f97b8b992debc8e7fc4e95a296b URL: http://source.winehq.org/git/wine.git/?a=commit;h=076697e31ab75f97b8b992deb… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue May 17 19:41:56 2016 +0200 wined3d: Rename MAX_CONST_B to WINED3D_MAX_CONSTS_B. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/arb_program_shader.c | 4 ++-- dlls/wined3d/device.c | 16 ++++++++-------- dlls/wined3d/glsl_shader.c | 10 +++++----- dlls/wined3d/stateblock.c | 4 ++-- dlls/wined3d/wined3d_private.h | 18 +++++++++--------- 5 files changed, 26 insertions(+), 26 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=076697e31ab75f97b8b99…

1 0

Jacek Caban : wininet: Get rid of INTERNET_INVALID_PORT_NUMBER fixups after InternetCrackUrl calls.
by Alexandre Julliard 18 May '16

18 May '16

Module: wine Branch: master Commit: 228c06086a45d84602f0b4fe3e45b3dd160a1459 URL: http://source.winehq.org/git/wine.git/?a=commit;h=228c06086a45d84602f0b4fe3… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue May 17 19:20:21 2016 +0200 wininet: Get rid of INTERNET_INVALID_PORT_NUMBER fixups after InternetCrackUrl calls. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wininet/http.c | 17 +++-------------- dlls/wininet/internet.c | 9 +-------- 2 files changed, 4 insertions(+), 22 deletions(-) diff --git a/dlls/wininet/http.c b/dlls/wininet/http.c index 96f481e..21f6ff2 100644 --- a/dlls/wininet/http.c +++ b/dlls/wininet/http.c @@ -1792,7 +1792,6 @@ static BOOL HTTP_DealWithProxy(appinfo_t *hIC, http_session_t *session, http_req URL_COMPONENTSW UrlComponents = { sizeof(UrlComponents) }; server_t *new_server = NULL; WCHAR *proxy; - BOOL is_https; proxy = INTERNET_FindProxyForProtocol(hIC->proxy, protoHttp); if(!proxy) @@ -1813,12 +1812,8 @@ static BOOL HTTP_DealWithProxy(appinfo_t *hIC, http_session_t *session, http_req if( !request->path ) request->path = szNul; - is_https = (UrlComponents.nScheme == INTERNET_SCHEME_HTTPS); - if (is_https && UrlComponents.nPort == INTERNET_INVALID_PORT_NUMBER) - UrlComponents.nPort = INTERNET_DEFAULT_HTTPS_PORT; - new_server = get_server(substr(UrlComponents.lpszHostName, UrlComponents.dwHostNameLength), - UrlComponents.nPort, is_https, TRUE); + UrlComponents.nPort, UrlComponents.nScheme == INTERNET_SCHEME_HTTPS, TRUE); } heap_free(proxy); if(!new_server) @@ -4171,10 +4166,7 @@ static DWORD HTTP_HandleRedirect(http_request_t *request, LPCWSTR lpszUrl) request->hdr.dwFlags &= ~INTERNET_FLAG_SECURE; } - if(urlComponents.nPort == INTERNET_INVALID_PORT_NUMBER) - urlComponents.nPort = INTERNET_DEFAULT_HTTP_PORT; - else if(urlComponents.nPort != INTERNET_DEFAULT_HTTP_PORT) - custom_port = TRUE; + custom_port = urlComponents.nPort != INTERNET_DEFAULT_HTTP_PORT; }else if(!strcmpiW(protocol, httpsW)) { if(!(request->hdr.dwFlags & INTERNET_FLAG_SECURE)) { TRACE("redirect from non-secure page to secure page\n"); @@ -4182,10 +4174,7 @@ static DWORD HTTP_HandleRedirect(http_request_t *request, LPCWSTR lpszUrl) request->hdr.dwFlags |= INTERNET_FLAG_SECURE; } - if(urlComponents.nPort == INTERNET_INVALID_PORT_NUMBER) - urlComponents.nPort = INTERNET_DEFAULT_HTTPS_PORT; - else if(urlComponents.nPort != INTERNET_DEFAULT_HTTPS_PORT) - custom_port = TRUE; + custom_port = urlComponents.nPort != INTERNET_DEFAULT_HTTPS_PORT; } heap_free(session->hostName); diff --git a/dlls/wininet/internet.c b/dlls/wininet/internet.c index 9751a49..bd380dc 100644 --- a/dlls/wininet/internet.c +++ b/dlls/wininet/internet.c @@ -3477,8 +3477,6 @@ static HINTERNET INTERNET_InternetOpenUrlW(appinfo_t *hIC, LPCWSTR lpszUrl, switch(urlComponents.nScheme) { case INTERNET_SCHEME_FTP: - if(urlComponents.nPort == 0) - urlComponents.nPort = INTERNET_DEFAULT_FTP_PORT; client = FTP_Connect(hIC, host, urlComponents.nPort, user, pass, dwFlags, dwContext, INET_OPENURL); if(client == NULL) @@ -3494,12 +3492,7 @@ static HINTERNET INTERNET_InternetOpenUrlW(appinfo_t *hIC, LPCWSTR lpszUrl, case INTERNET_SCHEME_HTTPS: { static const WCHAR szStars[] = { '*','/','*', 0 }; LPCWSTR accept[2] = { szStars, NULL }; - if(urlComponents.nPort == 0) { - if(urlComponents.nScheme == INTERNET_SCHEME_HTTP) - urlComponents.nPort = INTERNET_DEFAULT_HTTP_PORT; - else - urlComponents.nPort = INTERNET_DEFAULT_HTTPS_PORT; - } + if (urlComponents.nScheme == INTERNET_SCHEME_HTTPS) dwFlags |= INTERNET_FLAG_SECURE; /* FIXME: should use pointers, not handles, as handles are not thread-safe */

1 0

Jacek Caban : wininet: Don' t assume maximum URL length in HTTP_DealWithProxy.
by Alexandre Julliard 18 May '16

18 May '16

Module: wine Branch: master Commit: dfa014a5723c0241a95494892d70a4ebe9126207 URL: http://source.winehq.org/git/wine.git/?a=commit;h=dfa014a5723c0241a95494892… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue May 17 19:19:59 2016 +0200 wininet: Don't assume maximum URL length in HTTP_DealWithProxy. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wininet/http.c | 53 ++++++++++++++++++------------------ dlls/wininet/internet.c | 72 +++++++++++++++++-------------------------------- dlls/wininet/internet.h | 2 +- 3 files changed, 51 insertions(+), 76 deletions(-) diff --git a/dlls/wininet/http.c b/dlls/wininet/http.c index 0f155cf..96f481e 100644 --- a/dlls/wininet/http.c +++ b/dlls/wininet/http.c @@ -1788,40 +1788,39 @@ static BOOL HTTP_DealWithProxy(appinfo_t *hIC, http_session_t *session, http_req { static const WCHAR protoHttp[] = { 'h','t','t','p',0 }; static const WCHAR szHttp[] = { 'h','t','t','p',':','/','/',0 }; - static const WCHAR szFormat[] = { 'h','t','t','p',':','/','/','%','s',0 }; - WCHAR protoProxy[INTERNET_MAX_URL_LENGTH]; - DWORD protoProxyLen = INTERNET_MAX_URL_LENGTH; - WCHAR proxy[INTERNET_MAX_URL_LENGTH]; static WCHAR szNul[] = { 0 }; - URL_COMPONENTSW UrlComponents; - server_t *new_server; + URL_COMPONENTSW UrlComponents = { sizeof(UrlComponents) }; + server_t *new_server = NULL; + WCHAR *proxy; BOOL is_https; - memset( &UrlComponents, 0, sizeof UrlComponents ); - UrlComponents.dwStructSize = sizeof UrlComponents; - UrlComponents.dwHostNameLength = 1; - - if (!INTERNET_FindProxyForProtocol(hIC->proxy, protoHttp, protoProxy, &protoProxyLen)) - return FALSE; - if( CSTR_EQUAL != CompareStringW(LOCALE_SYSTEM_DEFAULT, NORM_IGNORECASE, - protoProxy,strlenW(szHttp),szHttp,strlenW(szHttp)) ) - sprintfW(proxy, szFormat, protoProxy); - else - strcpyW(proxy, protoProxy); - if( !InternetCrackUrlW(proxy, 0, 0, &UrlComponents) ) - return FALSE; - if( UrlComponents.dwHostNameLength == 0 ) + proxy = INTERNET_FindProxyForProtocol(hIC->proxy, protoHttp); + if(!proxy) return FALSE; + if(CSTR_EQUAL != CompareStringW(LOCALE_SYSTEM_DEFAULT, NORM_IGNORECASE, + proxy, strlenW(szHttp), szHttp, strlenW(szHttp))) { + WCHAR *proxy_url = heap_alloc(strlenW(proxy)*sizeof(WCHAR) + sizeof(szHttp)); + if(!proxy_url) + return FALSE; + strcpyW(proxy_url, szHttp); + strcatW(proxy_url, proxy); + heap_free(proxy); + proxy = proxy_url; + } - if( !request->path ) - request->path = szNul; + UrlComponents.dwHostNameLength = 1; + if(InternetCrackUrlW(proxy, 0, 0, &UrlComponents) && UrlComponents.dwHostNameLength) { + if( !request->path ) + request->path = szNul; - is_https = (UrlComponents.nScheme == INTERNET_SCHEME_HTTPS); - if (is_https && UrlComponents.nPort == INTERNET_INVALID_PORT_NUMBER) - UrlComponents.nPort = INTERNET_DEFAULT_HTTPS_PORT; + is_https = (UrlComponents.nScheme == INTERNET_SCHEME_HTTPS); + if (is_https && UrlComponents.nPort == INTERNET_INVALID_PORT_NUMBER) + UrlComponents.nPort = INTERNET_DEFAULT_HTTPS_PORT; - new_server = get_server(substr(UrlComponents.lpszHostName, UrlComponents.dwHostNameLength), - UrlComponents.nPort, is_https, TRUE); + new_server = get_server(substr(UrlComponents.lpszHostName, UrlComponents.dwHostNameLength), + UrlComponents.nPort, is_https, TRUE); + } + heap_free(proxy); if(!new_server) return FALSE; diff --git a/dlls/wininet/internet.c b/dlls/wininet/internet.c index fdad378..9751a49 100644 --- a/dlls/wininet/internet.c +++ b/dlls/wininet/internet.c @@ -390,15 +390,15 @@ static LONG INTERNET_SaveProxySettings( proxyinfo_t *lpwpi ) * *foundProxyLen is set to the required size in WCHARs, including the * NULL terminator, and the last error is set to ERROR_INSUFFICIENT_BUFFER. */ -BOOL INTERNET_FindProxyForProtocol(LPCWSTR szProxy, LPCWSTR proto, WCHAR *foundProxy, DWORD *foundProxyLen) +WCHAR *INTERNET_FindProxyForProtocol(LPCWSTR szProxy, LPCWSTR proto) { - LPCWSTR ptr; - BOOL ret = FALSE; + WCHAR *ret = NULL; + const WCHAR *ptr; TRACE("(%s, %s)\n", debugstr_w(szProxy), debugstr_w(proto)); /* First, look for the specified protocol (proto=scheme://host:port) */ - for (ptr = szProxy; !ret && ptr && *ptr; ) + for (ptr = szProxy; ptr && *ptr; ) { LPCWSTR end, equal; @@ -408,60 +408,36 @@ BOOL INTERNET_FindProxyForProtocol(LPCWSTR szProxy, LPCWSTR proto, WCHAR *foundP equal - ptr == strlenW(proto) && !strncmpiW(proto, ptr, strlenW(proto))) { - if (end - equal > *foundProxyLen) - { - WARN("buffer too short for %s\n", - debugstr_wn(equal + 1, end - equal - 1)); - *foundProxyLen = end - equal; - SetLastError(ERROR_INSUFFICIENT_BUFFER); - } - else - { - memcpy(foundProxy, equal + 1, (end - equal) * sizeof(WCHAR)); - foundProxy[end - equal] = 0; - ret = TRUE; - } + ret = heap_strndupW(equal + 1, end - equal - 1); + TRACE("found proxy for %s: %s\n", debugstr_w(proto), debugstr_w(ret)); + return ret; } if (*end == ' ') ptr = end + 1; else ptr = end; } - if (!ret) + + /* It wasn't found: look for no protocol */ + for (ptr = szProxy; ptr && *ptr; ) { - /* It wasn't found: look for no protocol */ - for (ptr = szProxy; !ret && ptr && *ptr; ) - { - LPCWSTR end; + LPCWSTR end; - if (!(end = strchrW(ptr, ' '))) - end = ptr + strlenW(ptr); - if (!strchrW(ptr, '=')) - { - if (end - ptr + 1 > *foundProxyLen) - { - WARN("buffer too short for %s\n", - debugstr_wn(ptr, end - ptr)); - *foundProxyLen = end - ptr + 1; - SetLastError(ERROR_INSUFFICIENT_BUFFER); - } - else - { - memcpy(foundProxy, ptr, (end - ptr) * sizeof(WCHAR)); - foundProxy[end - ptr] = 0; - ret = TRUE; - } - } - if (*end == ' ') - ptr = end + 1; - else - ptr = end; + if (!(end = strchrW(ptr, ' '))) + end = ptr + strlenW(ptr); + if (!strchrW(ptr, '=')) + { + ret = heap_strndupW(ptr, end - ptr); + TRACE("found proxy for %s: %s\n", debugstr_w(proto), debugstr_w(ret)); + return ret; } + if (*end == ' ') + ptr = end + 1; + else + ptr = end; } - if (ret) - TRACE("found proxy for %s: %s\n", debugstr_w(proto), - debugstr_w(foundProxy)); - return ret; + + return NULL; } /*********************************************************************** diff --git a/dlls/wininet/internet.h b/dlls/wininet/internet.h index 8cd1a8e..91c09b8 100644 --- a/dlls/wininet/internet.h +++ b/dlls/wininet/internet.h @@ -450,7 +450,7 @@ VOID SendAsyncCallback(object_header_t *hdr, DWORD_PTR dwContext, VOID INTERNET_SendCallback(object_header_t *hdr, DWORD_PTR dwContext, DWORD dwInternetStatus, LPVOID lpvStatusInfo, DWORD dwStatusInfoLength) DECLSPEC_HIDDEN; -BOOL INTERNET_FindProxyForProtocol(LPCWSTR szProxy, LPCWSTR proto, WCHAR *foundProxy, DWORD *foundProxyLen) DECLSPEC_HIDDEN; +WCHAR *INTERNET_FindProxyForProtocol(LPCWSTR szProxy, LPCWSTR proto) DECLSPEC_HIDDEN; DWORD create_netconn(BOOL,server_t*,DWORD,BOOL,DWORD,netconn_t**) DECLSPEC_HIDDEN; void free_netconn(netconn_t*) DECLSPEC_HIDDEN;

1 0

Jacek Caban : wininet: Pass host name as substring to get_server.
by Alexandre Julliard 18 May '16

18 May '16

Module: wine Branch: master Commit: 3413d01764b435a708d67b5a8a00d973f16e9720 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3413d01764b435a708d67b5a8… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue May 17 19:18:32 2016 +0200 wininet: Pass host name as substring to get_server. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wininet/http.c | 18 +++++++++--------- dlls/wininet/internet.c | 10 +--------- dlls/wininet/internet.h | 2 +- 3 files changed, 11 insertions(+), 19 deletions(-) diff --git a/dlls/wininet/http.c b/dlls/wininet/http.c index daaf3aa..0f155cf 100644 --- a/dlls/wininet/http.c +++ b/dlls/wininet/http.c @@ -265,7 +265,7 @@ static BOOL process_host_port(server_t *server) return TRUE; } -server_t *get_server(const WCHAR *name, INTERNET_PORT port, BOOL is_https, BOOL do_create) +server_t *get_server(substr_t name, INTERNET_PORT port, BOOL is_https, BOOL do_create) { server_t *iter, *server = NULL; @@ -275,7 +275,8 @@ server_t *get_server(const WCHAR *name, INTERNET_PORT port, BOOL is_https, BOOL EnterCriticalSection(&connection_pool_cs); LIST_FOR_EACH_ENTRY(iter, &connection_pool, server_t, entry) { - if(iter->port == port && !strcmpW(iter->name, name) && iter->is_https == is_https) { + if(iter->port == port && name.len == strlenW(iter->name) && !strncmpW(iter->name, name.str, name.len) + && iter->is_https == is_https) { server = iter; server_addref(server); break; @@ -289,7 +290,7 @@ server_t *get_server(const WCHAR *name, INTERNET_PORT port, BOOL is_https, BOOL server->port = port; server->is_https = is_https; list_init(&server->conn_pool); - server->name = heap_strdupW(name); + server->name = heap_strndupW(name.str, name.len); if(server->name && process_host_port(server)) { list_add_head(&connection_pool, &server->entry); }else { @@ -1788,7 +1789,6 @@ static BOOL HTTP_DealWithProxy(appinfo_t *hIC, http_session_t *session, http_req static const WCHAR protoHttp[] = { 'h','t','t','p',0 }; static const WCHAR szHttp[] = { 'h','t','t','p',':','/','/',0 }; static const WCHAR szFormat[] = { 'h','t','t','p',':','/','/','%','s',0 }; - WCHAR buf[INTERNET_MAX_HOST_NAME_LENGTH]; WCHAR protoProxy[INTERNET_MAX_URL_LENGTH]; DWORD protoProxyLen = INTERNET_MAX_URL_LENGTH; WCHAR proxy[INTERNET_MAX_URL_LENGTH]; @@ -1799,8 +1799,7 @@ static BOOL HTTP_DealWithProxy(appinfo_t *hIC, http_session_t *session, http_req memset( &UrlComponents, 0, sizeof UrlComponents ); UrlComponents.dwStructSize = sizeof UrlComponents; - UrlComponents.lpszHostName = buf; - UrlComponents.dwHostNameLength = INTERNET_MAX_HOST_NAME_LENGTH; + UrlComponents.dwHostNameLength = 1; if (!INTERNET_FindProxyForProtocol(hIC->proxy, protoHttp, protoProxy, &protoProxyLen)) return FALSE; @@ -1821,7 +1820,8 @@ static BOOL HTTP_DealWithProxy(appinfo_t *hIC, http_session_t *session, http_req if (is_https && UrlComponents.nPort == INTERNET_INVALID_PORT_NUMBER) UrlComponents.nPort = INTERNET_DEFAULT_HTTPS_PORT; - new_server = get_server(UrlComponents.lpszHostName, UrlComponents.nPort, is_https, TRUE); + new_server = get_server(substr(UrlComponents.lpszHostName, UrlComponents.dwHostNameLength), + UrlComponents.nPort, is_https, TRUE); if(!new_server) return FALSE; @@ -3391,7 +3391,7 @@ static DWORD HTTP_HttpOpenRequestW(http_session_t *session, request->session = session; list_add_head( &session->hdr.children, &request->hdr.entry ); - request->server = get_server(session->hostName, session->hostPort, (dwFlags & INTERNET_FLAG_SECURE) != 0, TRUE); + request->server = get_server(substrz(session->hostName), session->hostPort, (dwFlags & INTERNET_FLAG_SECURE) != 0, TRUE); if(!request->server) { WININET_Release(&request->hdr); return ERROR_OUTOFMEMORY; @@ -4204,7 +4204,7 @@ static DWORD HTTP_HandleRedirect(http_request_t *request, LPCWSTR lpszUrl) if(strcmpiW(request->server->name, hostName) || request->server->port != urlComponents.nPort) { server_t *new_server; - new_server = get_server(hostName, urlComponents.nPort, urlComponents.nScheme == INTERNET_SCHEME_HTTPS, TRUE); + new_server = get_server(substrz(hostName), urlComponents.nPort, urlComponents.nScheme == INTERNET_SCHEME_HTTPS, TRUE); server_release(request->server); request->server = new_server; } diff --git a/dlls/wininet/internet.c b/dlls/wininet/internet.c index 78bea7c..fdad378 100644 --- a/dlls/wininet/internet.c +++ b/dlls/wininet/internet.c @@ -4496,7 +4496,6 @@ BOOL WINAPI InternetGetSecurityInfoByURLW(LPCWSTR lpszURL, PCCERT_CHAIN_CONTEXT { URL_COMPONENTSW url = {sizeof(url)}; server_t *server; - WCHAR *hostname; BOOL res; TRACE("(%s %p %p)\n", debugstr_w(lpszURL), ppCertChain, pdwSecureFlags); @@ -4508,14 +4507,7 @@ BOOL WINAPI InternetGetSecurityInfoByURLW(LPCWSTR lpszURL, PCCERT_CHAIN_CONTEXT return FALSE; } - hostname = heap_strndupW(url.lpszHostName, url.dwHostNameLength); - if(!hostname) { - SetLastError(ERROR_OUTOFMEMORY); - return FALSE; - } - - server = get_server(hostname, url.nPort, TRUE, FALSE); - heap_free(hostname); + server = get_server(substr(url.lpszHostName, url.dwHostNameLength), url.nPort, TRUE, FALSE); if(!server) { SetLastError(ERROR_INTERNET_ITEM_NOT_FOUND); return FALSE; diff --git a/dlls/wininet/internet.h b/dlls/wininet/internet.h index 20ee999..8cd1a8e 100644 --- a/dlls/wininet/internet.h +++ b/dlls/wininet/internet.h @@ -467,7 +467,7 @@ DWORD NETCON_set_timeout(netconn_t *connection, BOOL send, DWORD value) DECLSPEC int sock_send(int fd, const void *msg, size_t len, int flags) DECLSPEC_HIDDEN; int sock_recv(int fd, void *msg, size_t len, int flags) DECLSPEC_HIDDEN; -server_t *get_server(const WCHAR*,INTERNET_PORT,BOOL,BOOL) DECLSPEC_HIDDEN; +server_t *get_server(substr_t,INTERNET_PORT,BOOL,BOOL) DECLSPEC_HIDDEN; DWORD create_req_file(const WCHAR*,req_file_t**) DECLSPEC_HIDDEN; void req_file_release(req_file_t*) DECLSPEC_HIDDEN;

1 0

Jacek Caban : wininet: Don' t assume maximal URL length in HTTP_ShouldBypassProxy.
by Alexandre Julliard 18 May '16

18 May '16

Module: wine Branch: master Commit: bd1f39ea276bf8d96e82bc59bc8872dd563361a9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=bd1f39ea276bf8d96e82bc59b… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue May 17 19:18:05 2016 +0200 wininet: Don't assume maximal URL length in HTTP_ShouldBypassProxy. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wininet/http.c | 106 ++++++++++++++++++++++------------------------------ 1 file changed, 45 insertions(+), 61 deletions(-) diff --git a/dlls/wininet/http.c b/dlls/wininet/http.c index 9ccb776..daaf3aa 100644 --- a/dlls/wininet/http.c +++ b/dlls/wininet/http.c @@ -1714,55 +1714,47 @@ static WCHAR *build_proxy_path_url(http_request_t *req) return url; } -static BOOL HTTP_DomainMatches(LPCWSTR server, LPCWSTR domain) +static BOOL HTTP_DomainMatches(LPCWSTR server, substr_t domain) { static const WCHAR localW[] = { '<','l','o','c','a','l','>',0 }; - BOOL ret = FALSE; + const WCHAR *dot, *ptr; + int len; - if (!strcmpiW( domain, localW ) && !strchrW( server, '.' )) - ret = TRUE; - else if (*domain == '*') - { - if (domain[1] == '.') - { - LPCWSTR dot; - - /* For a hostname to match a wildcard, the last domain must match - * the wildcard exactly. E.g. if the wildcard is *.a.b, and the - * hostname is www.foo.a.b, it matches, but a.b does not. - */ - dot = strchrW( server, '.' ); - if (dot) - { - int len = strlenW( dot + 1 ); + if(domain.len == sizeof(localW)/sizeof(WCHAR)-1 && !strncmpiW(domain.str, localW, domain.len) && !strchrW(server, '.' )) + return TRUE; - if (len > strlenW( domain + 2 )) - { - LPCWSTR ptr; - - /* The server's domain is longer than the wildcard, so it - * could be a subdomain. Compare the last portion of the - * server's domain. - */ - ptr = dot + len + 1 - strlenW( domain + 2 ); - if (!strcmpiW( ptr, domain + 2 )) - { - /* This is only a match if the preceding character is - * a '.', i.e. that it is a matching domain. E.g. - * if domain is '*.b.c' and server is 'www.ab.c' they - * do not match. - */ - ret = *(ptr - 1) == '.'; - } - } - else - ret = !strcmpiW( dot + 1, domain + 2 ); - } - } - } - else - ret = !strcmpiW( server, domain ); - return ret; + if(domain.len && *domain.str != '*') + return domain.len == strlenW(server) && !strncmpiW(server, domain.str, domain.len); + + if(domain.len < 2 || domain.str[1] != '.') + return FALSE; + + /* For a hostname to match a wildcard, the last domain must match + * the wildcard exactly. E.g. if the wildcard is *.a.b, and the + * hostname is www.foo.a.b, it matches, but a.b does not. + */ + dot = strchrW(server, '.'); + if(!dot) + return FALSE; + + len = strlenW(dot + 1); + if(len <= domain.len - 2) + return FALSE; + + /* The server's domain is longer than the wildcard, so it + * could be a subdomain. Compare the last portion of the + * server's domain. + */ + ptr = dot + 1 + len - domain.len + 2; + if(!strncmpiW(ptr, domain.str+2, domain.len-2)) + /* This is only a match if the preceding character is + * a '.', i.e. that it is a matching domain. E.g. + * if domain is '*.b.c' and server is 'www.ab.c' they + * do not match. + */ + return *(ptr - 1) == '.'; + + return len == domain.len-2 && !strncmpiW(dot + 1, domain.str + 2, len); } static BOOL HTTP_ShouldBypassProxy(appinfo_t *lpwai, LPCWSTR server) @@ -1772,27 +1764,19 @@ static BOOL HTTP_ShouldBypassProxy(appinfo_t *lpwai, LPCWSTR server) if (!lpwai->proxyBypass) return FALSE; ptr = lpwai->proxyBypass; - do { + while(1) { LPCWSTR tmp = ptr; ptr = strchrW( ptr, ';' ); if (!ptr) ptr = strchrW( tmp, ' ' ); - if (ptr) - { - if (ptr - tmp < INTERNET_MAX_HOST_NAME_LENGTH) - { - WCHAR domain[INTERNET_MAX_HOST_NAME_LENGTH]; - - memcpy( domain, tmp, (ptr - tmp) * sizeof(WCHAR) ); - domain[ptr - tmp] = 0; - ret = HTTP_DomainMatches( server, domain ); - } - ptr += 1; - } - else if (*tmp) - ret = HTTP_DomainMatches( server, tmp ); - } while (ptr && !ret); + if (!ptr) + ptr = tmp + strlenW(ptr); + ret = HTTP_DomainMatches( server, substr(tmp, ptr-tmp) ); + if (ret || !*ptr) + break; + ptr++; + } return ret; }

1 0

Jacek Caban : wininet: Removed maximum URL length assumption from get_proxy_autoconfig_url.
by Alexandre Julliard 18 May '16

18 May '16

Module: wine Branch: master Commit: 1d5670c7c45af92565e19973bbd9e2d7f16acbae URL: http://source.winehq.org/git/wine.git/?a=commit;h=1d5670c7c45af92565e19973b… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue May 17 19:16:21 2016 +0200 wininet: Removed maximum URL length assumption from get_proxy_autoconfig_url. Also use correct encoding. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wininet/internet.c | 33 +++++++++++++++++++-------------- 1 file changed, 19 insertions(+), 14 deletions(-) diff --git a/dlls/wininet/internet.c b/dlls/wininet/internet.c index 371c3d0..78bea7c 100644 --- a/dlls/wininet/internet.c +++ b/dlls/wininet/internet.c @@ -2307,13 +2307,14 @@ BOOL WINAPI InternetReadFileExW(HINTERNET hFile, LPINTERNET_BUFFERSW lpBuffer, return res == ERROR_SUCCESS; } -static BOOL get_proxy_autoconfig_url( char *buf, DWORD buflen ) +static WCHAR *get_proxy_autoconfig_url(void) { #if defined(MAC_OS_X_VERSION_10_6) && MAC_OS_X_VERSION_MAX_ALLOWED >= MAC_OS_X_VERSION_10_6 CFDictionaryRef settings = CFNetworkCopySystemProxySettings(); + WCHAR *ret = NULL; + SIZE_T len; const void *ref; - BOOL ret = FALSE; if (!settings) return FALSE; @@ -2322,16 +2323,20 @@ static BOOL get_proxy_autoconfig_url( char *buf, DWORD buflen ) CFRelease( settings ); return FALSE; } - if (CFStringGetCString( ref, buf, buflen, kCFStringEncodingASCII )) + len = CFStringGetLength( ref ); + if (len) + ret = heap_alloc( (len+1) * sizeof(WCHAR) ); + if (ret) { - TRACE( "returning %s\n", debugstr_a(buf) ); - ret = TRUE; + CFStringGetCharacters( ref, CFRangeMake(0, len), ret ); + ret[len] = 0; } + TRACE( "returning %s\n", debugstr_w(ret) ); CFRelease( settings ); return ret; #else FIXME( "no support on this platform\n" ); - return FALSE; + return NULL; #endif } @@ -2418,20 +2423,17 @@ static DWORD query_global_option(DWORD option, void *buffer, DWORD *size, BOOL u } case INTERNET_OPTION_PER_CONNECTION_OPTION: { - char url[INTERNET_MAX_URL_LENGTH + 1]; + WCHAR *url; INTERNET_PER_CONN_OPTION_LISTW *con = buffer; INTERNET_PER_CONN_OPTION_LISTA *conA = buffer; DWORD res = ERROR_SUCCESS, i; proxyinfo_t pi; - BOOL have_url; LONG ret; TRACE("Getting global proxy info\n"); if((ret = INTERNET_LoadProxySettings(&pi))) return ret; - have_url = get_proxy_autoconfig_url(url, sizeof(url)); - FIXME("INTERNET_OPTION_PER_CONNECTION_OPTION stub\n"); if (*size < sizeof(INTERNET_PER_CONN_OPTION_LISTW)) { @@ -2439,6 +2441,8 @@ static DWORD query_global_option(DWORD option, void *buffer, DWORD *size, BOOL u return ERROR_INSUFFICIENT_BUFFER; } + url = get_proxy_autoconfig_url(); + for (i = 0; i < con->dwOptionCount; i++) { INTERNET_PER_CONN_OPTIONW *optionW = con->pOptions + i; INTERNET_PER_CONN_OPTIONA *optionA = conA->pOptions + i; @@ -2449,7 +2453,7 @@ static DWORD query_global_option(DWORD option, void *buffer, DWORD *size, BOOL u optionW->Value.dwValue = PROXY_TYPE_PROXY; else optionW->Value.dwValue = PROXY_TYPE_DIRECT; - if (have_url) + if (url) /* native includes PROXY_TYPE_DIRECT even if PROXY_TYPE_PROXY is set */ optionW->Value.dwValue |= PROXY_TYPE_DIRECT|PROXY_TYPE_AUTO_PROXY_URL; break; @@ -2469,12 +2473,12 @@ static DWORD query_global_option(DWORD option, void *buffer, DWORD *size, BOOL u break; case INTERNET_PER_CONN_AUTOCONFIG_URL: - if (!have_url) + if (!url) optionW->Value.pszValue = NULL; else if (unicode) - optionW->Value.pszValue = heap_strdupAtoW(url); + optionW->Value.pszValue = heap_strdupW(url); else - optionA->Value.pszValue = heap_strdupA(url); + optionA->Value.pszValue = heap_strdupWtoA(url); break; case INTERNET_PER_CONN_AUTODISCOVERY_FLAGS: @@ -2495,6 +2499,7 @@ static DWORD query_global_option(DWORD option, void *buffer, DWORD *size, BOOL u break; } } + heap_free(url); FreeProxyInfo(&pi); return res;

1 0

Jacek Caban : wininet: Include config.h in internet.c for Mac configure macros.
by Alexandre Julliard 18 May '16

18 May '16

Module: wine Branch: master Commit: 7d541b3fd3531e9edb334780f9ff3093fed6fd1b URL: http://source.winehq.org/git/wine.git/?a=commit;h=7d541b3fd3531e9edb334780f… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue May 17 19:15:49 2016 +0200 wininet: Include config.h in internet.c for Mac configure macros. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wininet/internet.c | 22 ++++++++++++---------- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git a/dlls/wininet/internet.c b/dlls/wininet/internet.c index f7e974a..371c3d0 100644 --- a/dlls/wininet/internet.c +++ b/dlls/wininet/internet.c @@ -26,15 +26,7 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ - -#include "ws2tcpip.h" - -#include <string.h> -#include <stdarg.h> -#include <stdio.h> -#include <stdlib.h> -#include <ctype.h> -#include <assert.h> +#include "config.h" #ifdef HAVE_CORESERVICES_CORESERVICES_H #define GetCurrentThread MacGetCurrentThread @@ -45,6 +37,16 @@ #undef DPRINTF #endif +#include "winsock2.h" +#include "ws2ipdef.h" + +#include <string.h> +#include <stdarg.h> +#include <stdio.h> +#include <stdlib.h> +#include <ctype.h> +#include <assert.h> + #include "windef.h" #include "winbase.h" #include "winreg.h" @@ -3370,7 +3372,7 @@ BOOL WINAPI InternetCheckConnectionW( LPCWSTR lpszUrl, DWORD dwFlags, DWORD dwRe if (dwFlags & FLAG_ICC_FORCE_CONNECTION) { struct sockaddr_storage saddr; - socklen_t sa_len = sizeof(saddr); + int sa_len = sizeof(saddr); WCHAR *host_z; int fd; BOOL b;

1 0

Jacek Caban : wininet: Initialize url_w in InternetCrackUrlA.
by Alexandre Julliard 18 May '16

18 May '16

Module: wine Branch: master Commit: e0e0b6b7c6c74a58f41cde96972c045b2e34e167 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e0e0b6b7c6c74a58f41cde969… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue May 17 19:15:34 2016 +0200 wininet: Initialize url_w in InternetCrackUrlA. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wininet/internet.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/wininet/internet.c b/dlls/wininet/internet.c index f012d96..f7e974a 100644 --- a/dlls/wininet/internet.c +++ b/dlls/wininet/internet.c @@ -1551,7 +1551,7 @@ BOOL WINAPI InternetCrackUrlA(const char *url, DWORD url_length, DWORD flags, UR { WCHAR *host = NULL, *user = NULL, *pass = NULL, *path = NULL, *scheme = NULL, *extra = NULL; URL_COMPONENTSW comp; - WCHAR *url_w; + WCHAR *url_w = NULL; BOOL ret; TRACE("(%s %u %x %p)\n", url_length ? debugstr_an(url, url_length) : debugstr_a(url), url_length, flags, ret_comp);

1 0

Jacek Caban : wininet: Don' t assume maximal URL length in INTERNET_InternetOpenUrlW.
by Alexandre Julliard 18 May '16

18 May '16

Module: wine Branch: master Commit: 41e45509b6799fa5e9f8f8f202d016aa5e72294d URL: http://source.winehq.org/git/wine.git/?a=commit;h=41e45509b6799fa5e9f8f8f20… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue May 17 19:15:24 2016 +0200 wininet: Don't assume maximal URL length in INTERNET_InternetOpenUrlW. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wininet/internet.c | 72 +++++++++++++++++++++---------------------------- 1 file changed, 30 insertions(+), 42 deletions(-) diff --git a/dlls/wininet/internet.c b/dlls/wininet/internet.c index 75ae2d6..f012d96 100644 --- a/dlls/wininet/internet.c +++ b/dlls/wininet/internet.c @@ -3464,40 +3464,40 @@ BOOL WINAPI InternetCheckConnectionA(LPCSTR lpszUrl, DWORD dwFlags, DWORD dwRese static HINTERNET INTERNET_InternetOpenUrlW(appinfo_t *hIC, LPCWSTR lpszUrl, LPCWSTR lpszHeaders, DWORD dwHeadersLength, DWORD dwFlags, DWORD_PTR dwContext) { - URL_COMPONENTSW urlComponents; - WCHAR protocol[INTERNET_MAX_SCHEME_LENGTH]; - WCHAR hostName[INTERNET_MAX_HOST_NAME_LENGTH]; - WCHAR userName[INTERNET_MAX_USER_NAME_LENGTH]; - WCHAR password[INTERNET_MAX_PASSWORD_LENGTH]; - WCHAR path[INTERNET_MAX_PATH_LENGTH]; - WCHAR extra[1024]; + URL_COMPONENTSW urlComponents = { sizeof(urlComponents) }; + WCHAR *host, *user = NULL, *pass = NULL, *path; HINTERNET client = NULL, client1 = NULL; DWORD res; TRACE("(%p, %s, %s, %08x, %08x, %08lx)\n", hIC, debugstr_w(lpszUrl), debugstr_w(lpszHeaders), dwHeadersLength, dwFlags, dwContext); - urlComponents.dwStructSize = sizeof(URL_COMPONENTSW); - urlComponents.lpszScheme = protocol; - urlComponents.dwSchemeLength = INTERNET_MAX_SCHEME_LENGTH; - urlComponents.lpszHostName = hostName; - urlComponents.dwHostNameLength = INTERNET_MAX_HOST_NAME_LENGTH; - urlComponents.lpszUserName = userName; - urlComponents.dwUserNameLength = INTERNET_MAX_USER_NAME_LENGTH; - urlComponents.lpszPassword = password; - urlComponents.dwPasswordLength = INTERNET_MAX_PASSWORD_LENGTH; - urlComponents.lpszUrlPath = path; - urlComponents.dwUrlPathLength = INTERNET_MAX_PATH_LENGTH; - urlComponents.lpszExtraInfo = extra; - urlComponents.dwExtraInfoLength = 1024; + urlComponents.dwHostNameLength = 1; + urlComponents.dwUserNameLength = 1; + urlComponents.dwPasswordLength = 1; + urlComponents.dwUrlPathLength = 1; + urlComponents.dwExtraInfoLength = 1; if(!InternetCrackUrlW(lpszUrl, strlenW(lpszUrl), 0, &urlComponents)) return NULL; + + if(urlComponents.nScheme == INTERNET_SCHEME_HTTP && urlComponents.dwExtraInfoLength) { + assert(urlComponents.lpszUrlPath + urlComponents.dwUrlPathLength == urlComponents.lpszExtraInfo); + urlComponents.dwUrlPathLength += urlComponents.dwExtraInfoLength; + } + + host = heap_strndupW(urlComponents.lpszHostName, urlComponents.dwHostNameLength); + path = heap_strndupW(urlComponents.lpszUrlPath, urlComponents.dwUrlPathLength); + if(urlComponents.dwUserNameLength) + user = heap_strndupW(urlComponents.lpszUserName, urlComponents.dwUserNameLength); + if(urlComponents.dwPasswordLength) + pass = heap_strndupW(urlComponents.lpszPassword, urlComponents.dwPasswordLength); + switch(urlComponents.nScheme) { case INTERNET_SCHEME_FTP: if(urlComponents.nPort == 0) urlComponents.nPort = INTERNET_DEFAULT_FTP_PORT; - client = FTP_Connect(hIC, hostName, urlComponents.nPort, - userName, password, dwFlags, dwContext, INET_OPENURL); + client = FTP_Connect(hIC, host, urlComponents.nPort, + user, pass, dwFlags, dwContext, INET_OPENURL); if(client == NULL) break; client1 = FtpOpenFileW(client, path, GENERIC_READ, dwFlags, dwContext); @@ -3520,30 +3520,14 @@ static HINTERNET INTERNET_InternetOpenUrlW(appinfo_t *hIC, LPCWSTR lpszUrl, if (urlComponents.nScheme == INTERNET_SCHEME_HTTPS) dwFlags |= INTERNET_FLAG_SECURE; /* FIXME: should use pointers, not handles, as handles are not thread-safe */ - res = HTTP_Connect(hIC, hostName, urlComponents.nPort, - userName, password, dwFlags, dwContext, INET_OPENURL, &client); + res = HTTP_Connect(hIC, host, urlComponents.nPort, + user, pass, dwFlags, dwContext, INET_OPENURL, &client); if(res != ERROR_SUCCESS) { INTERNET_SetLastError(res); break; } - if (urlComponents.dwExtraInfoLength) { - WCHAR *path_extra; - DWORD len = urlComponents.dwUrlPathLength + urlComponents.dwExtraInfoLength + 1; - - if (!(path_extra = heap_alloc(len * sizeof(WCHAR)))) - { - InternetCloseHandle(client); - break; - } - strcpyW(path_extra, urlComponents.lpszUrlPath); - strcatW(path_extra, urlComponents.lpszExtraInfo); - client1 = HttpOpenRequestW(client, NULL, path_extra, NULL, NULL, accept, dwFlags, dwContext); - heap_free(path_extra); - } - else - client1 = HttpOpenRequestW(client, NULL, path, NULL, NULL, accept, dwFlags, dwContext); - + client1 = HttpOpenRequestW(client, NULL, path, NULL, NULL, accept, dwFlags, dwContext); if(client1 == NULL) { InternetCloseHandle(client); break; @@ -3565,7 +3549,11 @@ static HINTERNET INTERNET_InternetOpenUrlW(appinfo_t *hIC, LPCWSTR lpszUrl, } TRACE(" %p <--\n", client1); - + + heap_free(host); + heap_free(path); + heap_free(user); + heap_free(pass); return client1; }

1 0

Jacek Caban : wininet: Added test of request with a long URL.
by Alexandre Julliard 18 May '16

18 May '16

Module: wine Branch: master Commit: 9e497aeca2658c33625618fd63731a8150721dac URL: http://source.winehq.org/git/wine.git/?a=commit;h=9e497aeca2658c33625618fd6… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue May 17 19:14:47 2016 +0200 wininet: Added test of request with a long URL. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wininet/tests/http.c | 22 ++++++++++++++++++++++ 1 file changed, 22 insertions(+) diff --git a/dlls/wininet/tests/http.c b/dlls/wininet/tests/http.c index d42c3e7..29dd965 100644 --- a/dlls/wininet/tests/http.c +++ b/dlls/wininet/tests/http.c @@ -4717,6 +4717,27 @@ static void test_http_read(int port) CloseHandle(server_req_rec_event); } +static void test_long_url(int port) +{ + char long_path[INTERNET_MAX_PATH_LENGTH*2] = "/echo_request?"; + char buf[sizeof(long_path)*2]; + test_request_t req; + BOOL ret; + + memset(long_path+strlen(long_path), 'x', sizeof(long_path)-strlen(long_path)); + long_path[sizeof(long_path)-1] = 0; + open_simple_request(&req, "localhost", port, NULL, long_path); + + ret = HttpSendRequestA(req.request, NULL, 0, NULL, 0); + ok(ret, "HttpSendRequest failed: %u\n", GetLastError()); + test_status_code(req.request, 200); + + receive_simple_request(req.request, buf, sizeof(buf)); + ok(strstr(buf, long_path) != NULL, "long pathnot found in %s\n", buf); + + close_request(&req); +} + static void test_http_connection(void) { struct server_info si; @@ -4767,6 +4788,7 @@ static void test_http_connection(void) test_basic_auth_credentials_reuse(si.port); test_async_read(si.port); test_http_read(si.port); + test_long_url(si.port); /* send the basic request again to shutdown the server thread */ test_basic_request(si.port, "GET", "/quit");

1 0

← Newer
1
...
25
26
27
28
29
30
31
...
55
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits May 2016