wine-commits February 2011

wine-commits@winehq.org

2 participants
674 discussions

Joris Huizer : crypt32: Remove helper variable i (Clang).
by Alexandre Julliard 08 Feb '11

08 Feb '11

Module: wine Branch: master Commit: 4d41bdd1074dc77029336d7170aa9f17500f5a12 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4d41bdd1074dc77029336d717… Author: Joris Huizer <joris_huizer(a)yahoo.com> Date: Sat Feb 5 15:04:39 2011 +0100 crypt32: Remove helper variable i (Clang). --- dlls/crypt32/base64.c | 22 ++++++++++------------ 1 files changed, 10 insertions(+), 12 deletions(-) diff --git a/dlls/crypt32/base64.c b/dlls/crypt32/base64.c index 7a23dd9..5d0d604 100644 --- a/dlls/crypt32/base64.c +++ b/dlls/crypt32/base64.c @@ -498,14 +498,13 @@ static inline BYTE decodeBase64Byte(int c) static LONG decodeBase64Block(const char *in_buf, int in_len, const char **nextBlock, PBYTE out_buf, DWORD *out_len) { - int len = in_len, i; + int len = in_len; const char *d = in_buf; int ip0, ip1, ip2, ip3; if (len < 4) return ERROR_INVALID_DATA; - i = 0; if (d[2] == '=') { if ((ip0 = decodeBase64Byte(d[0])) > 63) @@ -514,8 +513,8 @@ static LONG decodeBase64Block(const char *in_buf, int in_len, return ERROR_INVALID_DATA; if (out_buf) - out_buf[i] = (ip0 << 2) | (ip1 >> 4); - i++; + out_buf[0] = (ip0 << 2) | (ip1 >> 4); + *out_len = 1; } else if (d[3] == '=') { @@ -528,10 +527,10 @@ static LONG decodeBase64Block(const char *in_buf, int in_len, if (out_buf) { - out_buf[i + 0] = (ip0 << 2) | (ip1 >> 4); - out_buf[i + 1] = (ip1 << 4) | (ip2 >> 2); + out_buf[0] = (ip0 << 2) | (ip1 >> 4); + out_buf[1] = (ip1 << 4) | (ip2 >> 2); } - i += 2; + *out_len = 2; } else { @@ -546,11 +545,11 @@ static LONG decodeBase64Block(const char *in_buf, int in_len, if (out_buf) { - out_buf[i + 0] = (ip0 << 2) | (ip1 >> 4); - out_buf[i + 1] = (ip1 << 4) | (ip2 >> 2); - out_buf[i + 2] = (ip2 << 6) | ip3; + out_buf[0] = (ip0 << 2) | (ip1 >> 4); + out_buf[1] = (ip1 << 4) | (ip2 >> 2); + out_buf[2] = (ip2 << 6) | ip3; } - i += 3; + *out_len = 3; } if (len >= 6 && d[4] == '\r' && d[5] == '\n') *nextBlock = d + 6; @@ -560,7 +559,6 @@ static LONG decodeBase64Block(const char *in_buf, int in_len, *nextBlock = d + 4; else *nextBlock = NULL; - *out_len = i; return ERROR_SUCCESS; }

1 0

Austin English : msi/tests: Make sure to use return values (LLVM/Clang).
by Alexandre Julliard 08 Feb '11

08 Feb '11

Module: wine Branch: master Commit: 6816aa295e9a3cb2643dfc65b81d40a930e53203 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6816aa295e9a3cb2643dfc65b… Author: Austin English <austinenglish(a)gmail.com> Date: Fri Feb 4 18:29:14 2011 -0800 msi/tests: Make sure to use return values (LLVM/Clang). --- dlls/msi/tests/format.c | 408 +++++++++++++++++++++++++++++++++++++++++------ 1 files changed, 359 insertions(+), 49 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=6816aa295e9a3cb2643df…

1 0

Austin English : dplayx/tests: Make sure to use return values (LLVM/Clang).
by Alexandre Julliard 08 Feb '11

08 Feb '11

Module: wine Branch: master Commit: 4036cc06b059a963a363c39feedced9ce13a397f URL: http://source.winehq.org/git/wine.git/?a=commit;h=4036cc06b059a963a363c39fe… Author: Austin English <austinenglish(a)gmail.com> Date: Fri Feb 4 17:21:24 2011 -0800 dplayx/tests: Make sure to use return values (LLVM/Clang). --- dlls/dplayx/tests/dplayx.c | 27 +++++++++++++++++++++++++-- 1 files changed, 25 insertions(+), 2 deletions(-) diff --git a/dlls/dplayx/tests/dplayx.c b/dlls/dplayx/tests/dplayx.c index 6636999..b9d641b 100644 --- a/dlls/dplayx/tests/dplayx.c +++ b/dlls/dplayx/tests/dplayx.c @@ -1359,6 +1359,7 @@ static void test_EnumSessions(void) callbackData.dwCounter1 = -1; hr = IDirectPlayX_EnumSessions( pDP, &dpsd, 0, EnumSessions_cb, &callbackData, callbackData.dwFlags ); + checkHR( DP_OK, hr ); check( N_SESSIONS-2, callbackData.dwCounter1 ); /* Doesn't list private */ @@ -1367,6 +1368,7 @@ static void test_EnumSessions(void) callbackData.dwCounter1 = -1; hr = IDirectPlayX_EnumSessions( pDP, &dpsd, 0, EnumSessions_cb, &callbackData, callbackData.dwFlags ); + checkHR( DP_OK, hr ); check( N_SESSIONS-1, callbackData.dwCounter1 ); /* Doesn't list full, no new, no join, private, protected */ @@ -1374,6 +1376,7 @@ static void test_EnumSessions(void) callbackData.dwCounter1 = -1; hr = IDirectPlayX_EnumSessions( pDP, &dpsd, 0, EnumSessions_cb, &callbackData, callbackData.dwFlags ); + checkHR( DP_OK, hr ); check( N_SESSIONS-5, callbackData.dwCounter1 ); /* Like with DPENUMSESSIONS_AVAILABLE */ @@ -1381,6 +1384,7 @@ static void test_EnumSessions(void) callbackData.dwCounter1 = -1; hr = IDirectPlayX_EnumSessions( pDP, &dpsd, 0, EnumSessions_cb, &callbackData, callbackData.dwFlags ); + checkHR( DP_OK, hr ); check( N_SESSIONS-5, callbackData.dwCounter1 ); /* Doesn't list full, no new, no join, private */ @@ -1388,6 +1392,7 @@ static void test_EnumSessions(void) callbackData.dwCounter1 = -1; hr = IDirectPlayX_EnumSessions( pDP, &dpsd, 0, EnumSessions_cb, &callbackData, callbackData.dwFlags ); + checkHR( DP_OK, hr ); check( N_SESSIONS-4, callbackData.dwCounter1 ); @@ -1396,6 +1401,7 @@ static void test_EnumSessions(void) callbackData.dwCounter1 = -1; hr = IDirectPlayX_EnumSessions( pDP, &dpsd, 0, EnumSessions_cb, &callbackData, callbackData.dwFlags ); + checkHR( DP_OK, hr ); check( N_SESSIONS-4, callbackData.dwCounter1 ); /* Read cache of last sync enumeration */ @@ -1403,12 +1409,14 @@ static void test_EnumSessions(void) callbackData.dwCounter1 = -1; hr = IDirectPlayX_EnumSessions( pDP, &dpsd, 0, EnumSessions_cb, &callbackData, callbackData.dwFlags ); + checkHR( DP_OK, hr ); check( 0, callbackData.dwCounter1 ); /* Stop enumeration */ callbackData.dwFlags = DPENUMSESSIONS_ASYNC; callbackData.dwCounter1 = -1; hr = IDirectPlayX_EnumSessions( pDP, &dpsd, 0, EnumSessions_cb, &callbackData, callbackData.dwFlags ); + checkHR( DP_OK, hr ); check( 0, callbackData.dwCounter1 ); /* Start enumeration */ Sleep(500); /* Give time to fill the cache */ @@ -1417,12 +1425,14 @@ static void test_EnumSessions(void) callbackData.dwCounter1 = -1; hr = IDirectPlayX_EnumSessions( pDP, &dpsd, 0, EnumSessions_cb, &callbackData, callbackData.dwFlags ); + checkHR( DP_OK, hr ); check( N_SESSIONS-5, callbackData.dwCounter1 ); /* Retrieve results */ callbackData.dwFlags = DPENUMSESSIONS_STOPASYNC; callbackData.dwCounter1 = -1; hr = IDirectPlayX_EnumSessions( pDP, &dpsd, 0, EnumSessions_cb, &callbackData, callbackData.dwFlags ); + checkHR( DP_OK, hr ); check( 0, callbackData.dwCounter1 ); /* Stop enumeration */ @@ -1445,12 +1455,14 @@ static void test_EnumSessions(void) callbackData.dwCounter1 = -1; hr = IDirectPlayX_EnumSessions( pDP, &dpsd, 0, EnumSessions_cb, &callbackData, callbackData.dwFlags ); + checkHR( DP_OK, hr ); check( 0, callbackData.dwCounter1 ); callbackData.dwFlags = DPENUMSESSIONS_PASSWORDREQUIRED; callbackData.dwCounter1 = -1; hr = IDirectPlayX_EnumSessions( pDP, &dpsd, 0, EnumSessions_cb, &callbackData, callbackData.dwFlags ); + checkHR( DP_OK, hr ); check( 2, callbackData.dwCounter1 ); /* Both sessions automatically set DPSESSION_PASSWORDREQUIRED */ @@ -1471,6 +1483,7 @@ static void test_EnumSessions(void) callbackData.dwCounter1 = -1; hr = IDirectPlayX_EnumSessions( pDP, &dpsd, 0, EnumSessions_cb, &callbackData, callbackData.dwFlags ); + checkHR( DP_OK, hr ); check( 2, callbackData.dwCounter1 ); /* Without password, the flag is ignored */ @@ -1491,12 +1504,14 @@ static void test_EnumSessions(void) callbackData.dwCounter1 = -1; hr = IDirectPlayX_EnumSessions( pDP, &dpsd, 0, EnumSessions_cb, &callbackData, callbackData.dwFlags ); + checkHR( DP_OK, hr ); check( 0, callbackData.dwCounter1 ); callbackData.dwFlags = DPENUMSESSIONS_PASSWORDREQUIRED; callbackData.dwCounter1 = -1; hr = IDirectPlayX_EnumSessions( pDP, &dpsd, 0, EnumSessions_cb, &callbackData, callbackData.dwFlags ); + checkHR( DP_OK, hr ); check( 1, callbackData.dwCounter1 ); /* - Listing with incorrect password */ @@ -1505,12 +1520,14 @@ static void test_EnumSessions(void) callbackData.dwCounter1 = -1; hr = IDirectPlayX_EnumSessions( pDP, &dpsd, 0, EnumSessions_cb, &callbackData, callbackData.dwFlags ); + checkHR( DP_OK, hr ); check( 0, callbackData.dwCounter1 ); callbackData.dwFlags = DPENUMSESSIONS_PASSWORDREQUIRED; callbackData.dwCounter1 = -1; hr = IDirectPlayX_EnumSessions( pDP, &dpsd, 0, EnumSessions_cb, &callbackData, callbackData.dwFlags ); + checkHR( DP_OK, hr ); check( 1, callbackData.dwCounter1 ); /* - Listing with correct password */ @@ -1518,6 +1535,7 @@ static void test_EnumSessions(void) callbackData.dwCounter1 = -1; hr = IDirectPlayX_EnumSessions( pDP, &dpsd, 0, EnumSessions_cb, &callbackData, callbackData.dwFlags ); + checkHR( DP_OK, hr ); check( 2, callbackData.dwCounter1 ); @@ -1526,6 +1544,7 @@ static void test_EnumSessions(void) callbackData.dwCounter1 = -1; hr = IDirectPlayX_EnumSessions( pDP, &dpsd, 0, EnumSessions_cb, &callbackData, callbackData.dwFlags ); + checkHR( DP_OK, hr ); check( 2, callbackData.dwCounter1 ); /* Read cache of last sync enumeration, even private sessions */ @@ -1556,12 +1575,14 @@ static void test_EnumSessions(void) callbackData.dwCounter1 = -1; hr = IDirectPlayX_EnumSessions( pDP, &dpsd, 0, EnumSessions_cb, &callbackData, callbackData.dwFlags ); + checkHR( DP_OK, hr ); check( 1, callbackData.dwCounter1 ); /* Only one of the sessions */ dpsd.guidApplication = appGuid; callbackData.dwCounter1 = -1; hr = IDirectPlayX_EnumSessions( pDP, &dpsd, 0, EnumSessions_cb, &callbackData, callbackData.dwFlags ); + checkHR( DP_OK, hr ); check( 1, callbackData.dwCounter1 ); /* The other session */ /* FIXME: For some reason, if we enum 1st with appGuid and 2nd with appGuid2, @@ -1571,6 +1592,7 @@ static void test_EnumSessions(void) callbackData.dwCounter1 = -1; hr = IDirectPlayX_EnumSessions( pDP, &dpsd, 0, EnumSessions_cb, &callbackData, callbackData.dwFlags ); + checkHR( DP_OK, hr ); check( 2, callbackData.dwCounter1 ); /* Both sessions */ for (i=4; i<=5; i++) @@ -2460,7 +2482,7 @@ static void test_PlayerName(void) /* Regular operation */ hr = IDirectPlayX_SetPlayerName( pDP[0], dpid[0], &playerName, 0 ); - + checkHR( DP_OK, hr ); dwDataSize = 1024; hr = IDirectPlayX_GetPlayerName( pDP[0], dpid[0], lpData, &dwDataSize ); checkHR( DP_OK, hr ); @@ -2470,7 +2492,7 @@ static void test_PlayerName(void) check( 0, ((LPDPNAME)lpData)->dwFlags ); hr = IDirectPlayX_SetPlayerName( pDP[0], dpid[0], NULL, 0 ); - + checkHR( DP_OK, hr ); dwDataSize = 1024; hr = IDirectPlayX_GetPlayerName( pDP[0], dpid[0], lpData, &dwDataSize ); checkHR( DP_OK, hr ); @@ -5359,6 +5381,7 @@ static void test_Receive(void) { hr = IDirectPlayX_Receive( pDP, &idFrom, &idTo, DPRECEIVE_PEEK, lpData, &dwDataSize ); + checkHR( DP_OK, hr ); checkStr( message, (LPSTR) lpData ); }

1 0

André Hentschel : msvcrt: Fixed a typo.
by Alexandre Julliard 08 Feb '11

08 Feb '11

Module: wine Branch: master Commit: 68b0a13572131acd05c7f454cf40d5b7d0081368 URL: http://source.winehq.org/git/wine.git/?a=commit;h=68b0a13572131acd05c7f454c… Author: André Hentschel <nerv(a)dawncrow.de> Date: Fri Feb 4 17:01:48 2011 +0100 msvcrt: Fixed a typo. --- dlls/msvcr90/msvcr90.c | 2 +- dlls/msvcrt/cpp.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/msvcr90/msvcr90.c b/dlls/msvcr90/msvcr90.c index 2c811cb..32a874b 100644 --- a/dlls/msvcr90/msvcr90.c +++ b/dlls/msvcr90/msvcr90.c @@ -315,7 +315,7 @@ const char * __thiscall MSVCRT_type_info_name_internal_method(type_info * _this, if (!_this->name) { /* Create and set the demangled name */ - /* Nota: mangled name in type_info struct always start with a '.', while + /* Note: mangled name in type_info struct always starts with a '.', while * it isn't valid for mangled name. * Is this '.' really part of the mangled name, or has it some other meaning ? */ diff --git a/dlls/msvcrt/cpp.c b/dlls/msvcrt/cpp.c index 1e7a20a..6f6b5c6 100644 --- a/dlls/msvcrt/cpp.c +++ b/dlls/msvcrt/cpp.c @@ -605,7 +605,7 @@ const char * __thiscall MSVCRT_type_info_name(type_info * _this) if (!_this->name) { /* Create and set the demangled name */ - /* Nota: mangled name in type_info struct always start with a '.', while + /* Note: mangled name in type_info struct always starts with a '.', while * it isn't valid for mangled name. * Is this '.' really part of the mangled name, or has it some other meaning ? */

1 0

Frédéric Delanoy : po: Update French translation.
by Alexandre Julliard 08 Feb '11

08 Feb '11

Module: wine Branch: master Commit: 2b0e162ecca47549fa2b67d00eec46fdf5c6eba2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2b0e162ecca47549fa2b67d00… Author: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Date: Tue Feb 8 17:28:12 2011 +0100 po: Update French translation. --- po/fr.po | 1029 +++++++++++++++++++++++++++----------------------------------- 1 files changed, 440 insertions(+), 589 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=2b0e162ecca47549fa2b6…

1 0

Alexander Lakhin : po: Update Russian translation.
by Alexandre Julliard 08 Feb '11

08 Feb '11

Module: wine Branch: master Commit: a33f19bf0da65529babc463cc851b5d6d619d0b4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a33f19bf0da65529babc463cc… Author: Alexander Lakhin <exclusion(a)gmail.com> Date: Mon Feb 7 23:44:07 2011 +0300 po: Update Russian translation. --- po/ru.po | 2652 +++++++++++++++++++++++++++----------------------------------- 1 files changed, 1170 insertions(+), 1482 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=a33f19bf0da65529babc4…

1 0

Hwang YunSong : po: Update Korean resource.
by Alexandre Julliard 08 Feb '11

08 Feb '11

Module: wine Branch: master Commit: e7a41a1ef352ad302ba27c2425d655effbd0edc2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e7a41a1ef352ad302ba27c242… Author: Hwang YunSong <hys545(a)dreamwiz.com> Date: Tue Feb 8 11:18:12 2011 +0900 po: Update Korean resource. --- po/ko.po | 286 +++++++++++++++++++++++--------------------------------------- 1 files changed, 104 insertions(+), 182 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=e7a41a1ef352ad302ba27…

1 0

Jacek Caban : urlmon: Added more CoInternetGetSecurityUrl[Ex] tests.
by Alexandre Julliard 08 Feb '11

08 Feb '11

Module: wine Branch: master Commit: 380e300855d0adf1bcae0ac81339a48c15621272 URL: http://source.winehq.org/git/wine.git/?a=commit;h=380e300855d0adf1bcae0ac81… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Feb 8 15:17:37 2011 +0100 urlmon: Added more CoInternetGetSecurityUrl[Ex] tests. --- dlls/urlmon/tests/sec_mgr.c | 109 ++++++++++++++++++++++++++++++++++++------- 1 files changed, 92 insertions(+), 17 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=380e300855d0adf1bcae0…

1 0

Jacek Caban : urlmon: CoInternetGetSecurityUrlEx rewrite.
by Alexandre Julliard 08 Feb '11

08 Feb '11

Module: wine Branch: master Commit: 139382a3f8b0cffede0c64c85c8fdbb4c1337601 URL: http://source.winehq.org/git/wine.git/?a=commit;h=139382a3f8b0cffede0c64c85… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Feb 8 15:17:15 2011 +0100 urlmon: CoInternetGetSecurityUrlEx rewrite. --- dlls/urlmon/sec_mgr.c | 89 ++++++++++-------------------------------------- 1 files changed, 19 insertions(+), 70 deletions(-) diff --git a/dlls/urlmon/sec_mgr.c b/dlls/urlmon/sec_mgr.c index 4206d5d..562680f 100644 --- a/dlls/urlmon/sec_mgr.c +++ b/dlls/urlmon/sec_mgr.c @@ -293,59 +293,6 @@ static HRESULT get_action_policy(DWORD zone, DWORD action, BYTE *policy, DWORD s return hres; } -static HRESULT parse_security_uri(IUri *uri, PSUACTION action, IUri **result) { - WCHAR buf1[INTERNET_MAX_URL_LENGTH], buf2[INTERNET_MAX_URL_LENGTH]; - LPWSTR url, tmp; - HRESULT hres; - DWORD len = 0; - BOOL use_url = FALSE; - - url = buf1; - tmp = buf2; - *result = NULL; - - hres = IUri_GetPropertyLength(uri, Uri_PROPERTY_ABSOLUTE_URI, &len, 0); - if(FAILED(hres)) - return hres; - - hres = CoInternetParseIUri(uri, PARSE_SECURITY_URL, 0, url, len+1, &len, 0); - if(hres == S_OK) { - use_url = TRUE; - while(TRUE) { - hres = CoInternetParseUrl(url, PARSE_SECURITY_URL, 0, tmp, len+1, &len, 0); - if(hres != S_OK || !strcmpW(url, tmp)) - break; - - if(url == buf1) { - url = buf2; - tmp = buf1; - } else { - url = buf1; - tmp = buf2; - } - } - } - - if(action == PSU_DEFAULT) { - if(use_url) { - hres = CoInternetParseUrl(url, PARSE_SECURITY_DOMAIN, 0, tmp, len+1, &len, 0); - url = tmp; - } else { - hres = CoInternetParseIUri(uri, PARSE_SECURITY_DOMAIN, 0, url, len+1, &len, 0); - if(hres == S_OK) - use_url = TRUE; - } - } - - if(use_url) { - hres = CreateUri(url, 0, 0, result); - if(FAILED(hres)) - return hres; - } - - return S_OK; -} - /*********************************************************************** * InternetSecurityManager implementation * @@ -1464,42 +1411,44 @@ HRESULT WINAPI CoInternetGetSecurityUrl(LPCWSTR pwzUrl, LPWSTR *ppwzSecUrl, PSUA */ HRESULT WINAPI CoInternetGetSecurityUrlEx(IUri *pUri, IUri **ppSecUri, PSUACTION psuAction, DWORD_PTR dwReserved) { - HRESULT hres; - BSTR secure_uri; URL_SCHEME scheme_type; + BSTR secure_uri; + WCHAR *ret_url; + HRESULT hres; TRACE("(%p,%p,%u,%u)\n", pUri, ppSecUri, psuAction, (DWORD)dwReserved); if(!pUri || !ppSecUri) return E_INVALIDARG; - /* Try to find the Security url using pluggable protocols first. */ - hres = parse_security_uri(pUri, psuAction, ppSecUri); - if(FAILED(hres) || *ppSecUri) + hres = IUri_GetDisplayUri(pUri, &secure_uri); + if(FAILED(hres)) return hres; - hres = IUri_GetScheme(pUri, (DWORD*)&scheme_type); + hres = parse_security_url(secure_uri, psuAction, &ret_url); + SysFreeString(secure_uri); if(FAILED(hres)) return hres; - hres = IUri_GetDisplayUri(pUri, &secure_uri); - if(FAILED(hres)) + hres = CreateUri(ret_url, Uri_CREATE_ALLOW_IMPLICIT_WILDCARD_SCHEME, 0, ppSecUri); + if(FAILED(hres)) { + CoTaskMemFree(ret_url); return hres; + } /* File URIs have to hierarchical. */ - if(scheme_type == URL_SCHEME_FILE) { - const WCHAR *tmp = secure_uri; + hres = IUri_GetScheme(pUri, (DWORD*)&scheme_type); + if(SUCCEEDED(hres) && scheme_type == URL_SCHEME_FILE) { + const WCHAR *tmp = ret_url; /* Check and see if a "//" is after the scheme name. */ tmp += sizeof(fileW)/sizeof(WCHAR); - if(*tmp != '/' || *(tmp+1) != '/') { - SysFreeString(secure_uri); - return E_INVALIDARG; - } + if(*tmp != '/' || *(tmp+1) != '/') + hres = E_INVALIDARG; } - hres = CreateUri(secure_uri, Uri_CREATE_ALLOW_IMPLICIT_WILDCARD_SCHEME, 0, ppSecUri); - SysFreeString(secure_uri); - + if(SUCCEEDED(hres)) + hres = CreateUri(ret_url, Uri_CREATE_ALLOW_IMPLICIT_WILDCARD_SCHEME, 0, ppSecUri); + CoTaskMemFree(ret_url); return hres; }

1 0

Jacek Caban : urlmon: CoInternetGetSecurityUrl rewrite.
by Alexandre Julliard 08 Feb '11

08 Feb '11

Module: wine Branch: master Commit: 69ee496fbd5d88201c7b29a652121f01eb810023 URL: http://source.winehq.org/git/wine.git/?a=commit;h=69ee496fbd5d88201c7b29a65… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Feb 8 15:16:38 2011 +0100 urlmon: CoInternetGetSecurityUrl rewrite. This includes proper URL buffers, error handling and wrong pluggable protocol calls fixes. --- dlls/urlmon/sec_mgr.c | 202 +++++++++++++++++++++++++++++++++--------------- 1 files changed, 139 insertions(+), 63 deletions(-) diff --git a/dlls/urlmon/sec_mgr.c b/dlls/urlmon/sec_mgr.c index 2aca1a9..4206d5d 100644 --- a/dlls/urlmon/sec_mgr.c +++ b/dlls/urlmon/sec_mgr.c @@ -1299,87 +1299,163 @@ HRESULT WINAPI CoInternetCreateZoneManager(IServiceProvider* pSP, IInternetZoneM return ZoneMgrImpl_Construct(NULL, (void**)ppZM); } -/******************************************************************** - * CoInternetGetSecurityUrl (URLMON.@) - */ -HRESULT WINAPI CoInternetGetSecurityUrl(LPCWSTR pwzUrl, LPWSTR *ppwzSecUrl, PSUACTION psuAction, DWORD dwReserved) -{ - WCHAR buf1[INTERNET_MAX_URL_LENGTH], buf2[INTERNET_MAX_URL_LENGTH]; - LPWSTR url, domain; - DWORD len; - HRESULT hres; +static HRESULT parse_security_url(const WCHAR *url, PSUACTION action, WCHAR **result) { + IInternetProtocolInfo *protocol_info; + WCHAR *tmp, *new_url = NULL, *alloc_url = NULL; + DWORD size, new_size; + HRESULT hres = S_OK, parse_hres; - TRACE("(%p,%p,%u,%u)\n", pwzUrl, ppwzSecUrl, psuAction, dwReserved); + while(1) { + TRACE("parsing %s\n", debugstr_w(url)); - url = buf1; - domain = buf2; - strcpyW(url, pwzUrl); + protocol_info = get_protocol_info(url); + if(!protocol_info) + break; - while(1) { - hres = CoInternetParseUrl(url, PARSE_SECURITY_URL, 0, domain, INTERNET_MAX_URL_LENGTH, &len, 0); - if(hres!=S_OK || !strcmpW(url, domain)) + size = strlenW(url)+1; + new_url = CoTaskMemAlloc(size*sizeof(WCHAR)); + if(!new_url) { + hres = E_OUTOFMEMORY; break; + } - if(url == buf1) { - url = buf2; - domain = buf1; - } else { - url = buf1; - domain = buf2; + new_size = 0; + parse_hres = IInternetProtocolInfo_ParseUrl(protocol_info, url, PARSE_SECURITY_URL, 0, new_url, size, &new_size, 0); + if(parse_hres == S_FALSE) { + if(!new_size) { + hres = E_UNEXPECTED; + break; + } + + tmp = CoTaskMemRealloc(new_url, new_size*sizeof(WCHAR)); + if(!tmp) { + hres = E_OUTOFMEMORY; + break; + } + new_url = tmp; + parse_hres = IInternetProtocolInfo_ParseUrl(protocol_info, url, PARSE_SECURITY_URL, 0, new_url, + new_size, &new_size, 0); + if(parse_hres == S_FALSE) { + hres = E_FAIL; + break; + } } - } - if(psuAction==PSU_SECURITY_URL_ONLY) { - len = lstrlenW(url)+1; - *ppwzSecUrl = CoTaskMemAlloc(len*sizeof(WCHAR)); - if(!*ppwzSecUrl) - return E_OUTOFMEMORY; + if(parse_hres != S_OK || !strcmpW(url, new_url)) + break; - memcpy(*ppwzSecUrl, url, len*sizeof(WCHAR)); - return S_OK; + CoTaskMemFree(alloc_url); + url = alloc_url = new_url; + new_url = NULL; } - hres = CoInternetParseUrl(url, PARSE_SECURITY_DOMAIN, 0, domain, - INTERNET_MAX_URL_LENGTH, &len, 0); - if(SUCCEEDED(hres)) { - len++; - *ppwzSecUrl = CoTaskMemAlloc(len*sizeof(WCHAR)); - if(!*ppwzSecUrl) - return E_OUTOFMEMORY; + CoTaskMemFree(new_url); - memcpy(*ppwzSecUrl, domain, len*sizeof(WCHAR)); - return S_OK; + if(hres != S_OK) { + WARN("failed: %08x\n", hres); + CoTaskMemFree(alloc_url); + return hres; } - hres = CoInternetParseUrl(url, PARSE_SCHEMA, 0, domain, - INTERNET_MAX_URL_LENGTH, &len, 0); - if(hres == S_OK){ - const WCHAR fileW[] = {'f','i','l','e',0}; - if(!strcmpW(domain, fileW)){ - hres = CoInternetParseUrl(url, PARSE_ROOTDOCUMENT, 0, domain, INTERNET_MAX_URL_LENGTH, &len, 0); - }else{ - domain[len] = ':'; - hres = CoInternetParseUrl(url, PARSE_DOMAIN, 0, domain+len+1, - INTERNET_MAX_URL_LENGTH-len-1, &len, 0); - if(hres == S_OK) { - len = lstrlenW(domain)+1; - *ppwzSecUrl = CoTaskMemAlloc(len*sizeof(WCHAR)); - if(!*ppwzSecUrl) - return E_OUTOFMEMORY; - - memcpy(*ppwzSecUrl, domain, len*sizeof(WCHAR)); - return S_OK; + if(action == PSU_DEFAULT && (protocol_info = get_protocol_info(url))) { + size = strlenW(url)+1; + new_url = CoTaskMemAlloc(size * sizeof(WCHAR)); + if(new_url) { + new_size = 0; + parse_hres = IInternetProtocolInfo_ParseUrl(protocol_info, url, PARSE_SECURITY_DOMAIN, 0, + new_url, size, &new_size, 0); + if(parse_hres == S_FALSE) { + if(new_size) { + tmp = CoTaskMemRealloc(new_url, new_size*sizeof(WCHAR)); + if(tmp) { + new_url = tmp; + parse_hres = IInternetProtocolInfo_ParseUrl(protocol_info, url, PARSE_SECURITY_DOMAIN, 0, new_url, + new_size, &new_size, 0); + if(parse_hres == S_FALSE) + hres = E_FAIL; + }else { + hres = E_OUTOFMEMORY; + } + }else { + hres = E_UNEXPECTED; + } + } + + if(hres == S_OK && parse_hres == S_OK) { + CoTaskMemFree(alloc_url); + url = alloc_url = new_url; + new_url = NULL; } + + CoTaskMemFree(new_url); + }else { + hres = E_OUTOFMEMORY; } - }else + IInternetProtocolInfo_Release(protocol_info); + } + + if(FAILED(hres)) { + WARN("failed %08x\n", hres); + CoTaskMemFree(alloc_url); return hres; + } - len = lstrlenW(url)+1; - *ppwzSecUrl = CoTaskMemAlloc(len*sizeof(WCHAR)); - if(!*ppwzSecUrl) - return E_OUTOFMEMORY; + if(!alloc_url) { + size = strlenW(url)+1; + alloc_url = CoTaskMemAlloc(size * sizeof(WCHAR)); + if(!alloc_url) + return E_OUTOFMEMORY; + memcpy(alloc_url, url, size * sizeof(WCHAR)); + } + + *result = alloc_url; + return S_OK; +} + +/******************************************************************** + * CoInternetGetSecurityUrl (URLMON.@) + */ +HRESULT WINAPI CoInternetGetSecurityUrl(LPCWSTR pwzUrl, LPWSTR *ppwzSecUrl, PSUACTION psuAction, DWORD dwReserved) +{ + WCHAR *secure_url; + HRESULT hres; + + TRACE("(%p,%p,%u,%u)\n", pwzUrl, ppwzSecUrl, psuAction, dwReserved); + + hres = parse_security_url(pwzUrl, psuAction, &secure_url); + if(FAILED(hres)) + return hres; + + if(psuAction != PSU_SECURITY_URL_ONLY) { + PARSEDURLW parsed_url = { sizeof(parsed_url) }; + DWORD size; + + /* FIXME: Use helpers from uri.c */ + if(SUCCEEDED(ParseURLW(secure_url, &parsed_url))) { + WCHAR *new_url; + + switch(parsed_url.nScheme) { + case URL_SCHEME_FTP: + case URL_SCHEME_HTTP: + case URL_SCHEME_HTTPS: + size = strlenW(secure_url)+1; + new_url = CoTaskMemAlloc(size * sizeof(WCHAR)); + if(new_url) + hres = UrlGetPartW(secure_url, new_url, &size, URL_PART_HOSTNAME, URL_PARTFLAG_KEEPSCHEME); + else + hres = E_OUTOFMEMORY; + CoTaskMemFree(secure_url); + if(hres != S_OK) { + WARN("UrlGetPart failed: %08x\n", hres); + CoTaskMemFree(new_url); + return FAILED(hres) ? hres : E_FAIL; + } + secure_url = new_url; + } + } + } - memcpy(*ppwzSecUrl, url, len*sizeof(WCHAR)); + *ppwzSecUrl = secure_url; return S_OK; }

1 0

← Newer
1
...
46
47
48
49
50
51
52
...
68
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 February 2011