wine-commits May 2010

wine-commits@winehq.org

3 participants
955 discussions

Jacek Caban : mshtml: Test return value of OnStateChange.
by Alexandre Julliard 08 May '10

08 May '10

Module: wine Branch: master Commit: f279b0cd4d215d8e5565c20556864500f097f338 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f279b0cd4d215d8e5565c2055… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Sat May 8 18:34:03 2010 +0200 mshtml: Test return value of OnStateChange. --- dlls/mshtml/tests/script.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/dlls/mshtml/tests/script.c b/dlls/mshtml/tests/script.c index bda1e49..23f1ab8 100644 --- a/dlls/mshtml/tests/script.c +++ b/dlls/mshtml/tests/script.c @@ -2073,7 +2073,9 @@ static HRESULT WINAPI ActiveScript_SetScriptState(IActiveScript *iface, SCRIPTST } hres = IActiveScriptSite_OnStateChange(site, (state = ss)); - return hres; + ok(hres == S_OK, "OnStateChange failed: %08x\n", hres); + + return S_OK; } static HRESULT WINAPI ActiveScript_GetScriptState(IActiveScript *iface, SCRIPTSTATE *pssState)

1 0

Jacek Caban : mshtml: Check return value of SetText.
by Alexandre Julliard 08 May '10

08 May '10

Module: wine Branch: master Commit: 11484a0f68a1052decae76630d883265a520f2a4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=11484a0f68a1052decae76630… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Sat May 8 18:33:52 2010 +0200 mshtml: Check return value of SetText. Spotted by Gerald Pfeifer. --- dlls/mshtml/htmlbody.c | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/dlls/mshtml/htmlbody.c b/dlls/mshtml/htmlbody.c index e4bb1d0..18a6041 100644 --- a/dlls/mshtml/htmlbody.c +++ b/dlls/mshtml/htmlbody.c @@ -434,6 +434,10 @@ static HRESULT WINAPI HTMLBodyElement_put_text(IHTMLBodyElement *iface, VARIANT nsres = nsIDOMHTMLBodyElement_SetText(This->nsbody, &text); nsAString_Finish(&text); + if(NS_FAILED(nsres)) { + ERR("SetText failed: %08x\n", nsres); + return E_FAIL; + } return S_OK; }

1 0

Piotr Caban : wininet: Fix trace messages.
by Alexandre Julliard 08 May '10

08 May '10

Module: wine Branch: master Commit: c0890590ab56f017720b36d569dbc641980f70e5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c0890590ab56f017720b36d56… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Sat May 8 18:01:15 2010 +0200 wininet: Fix trace messages. --- dlls/wininet/internet.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/wininet/internet.c b/dlls/wininet/internet.c index c82ee29..eb3c8e2 100644 --- a/dlls/wininet/internet.c +++ b/dlls/wininet/internet.c @@ -1447,7 +1447,7 @@ BOOL WINAPI InternetCrackUrlA(LPCSTR lpszUrl, DWORD dwUrlLength, DWORD dwFlags, lpUrlComponents->nScheme = UCW.nScheme; lpUrlComponents->nPort = UCW.nPort; - TRACE("%s: scheme(%s) host(%s) path(%s) extra(%s)\n", lpszUrl, + TRACE("%s: scheme(%s) host(%s) path(%s) extra(%s)\n", debugstr_a(lpszUrl), debugstr_an(lpUrlComponents->lpszScheme, lpUrlComponents->dwSchemeLength), debugstr_an(lpUrlComponents->lpszHostName, lpUrlComponents->dwHostNameLength), debugstr_an(lpUrlComponents->lpszUrlPath, lpUrlComponents->dwUrlPathLength), @@ -1860,7 +1860,7 @@ BOOL WINAPI InternetCanonicalizeUrlA(LPCSTR lpszUrl, LPSTR lpszBuffer, DWORD dwURLFlags = URL_WININET_COMPATIBILITY | URL_ESCAPE_UNSAFE; TRACE("(%s, %p, %p, 0x%08x) bufferlength: %d\n", debugstr_a(lpszUrl), lpszBuffer, - lpdwBufferLength, lpdwBufferLength ? *lpdwBufferLength : -1, dwFlags); + lpdwBufferLength, dwFlags, lpdwBufferLength ? *lpdwBufferLength : -1); if(dwFlags & ICU_DECODE) {

1 0

Piotr Caban : shlwapi: Added more UrlCanonicalizeA tests.
by Alexandre Julliard 08 May '10

08 May '10

Module: wine Branch: master Commit: ee2abe7f82ec2c04b7710376b8cf59e7cbf23640 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ee2abe7f82ec2c04b7710376b… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Sat May 8 18:01:02 2010 +0200 shlwapi: Added more UrlCanonicalizeA tests. --- dlls/shlwapi/tests/url.c | 21 ++++++++++++++++++++- 1 files changed, 20 insertions(+), 1 deletions(-) diff --git a/dlls/shlwapi/tests/url.c b/dlls/shlwapi/tests/url.c index 0dd9643..c828ea9 100644 --- a/dlls/shlwapi/tests/url.c +++ b/dlls/shlwapi/tests/url.c @@ -793,7 +793,8 @@ static void test_UrlEscape(void) static void test_UrlCanonicalizeA(void) { unsigned int i; - CHAR szReturnUrl[INTERNET_MAX_URL_LENGTH]; + CHAR szReturnUrl[4*INTERNET_MAX_URL_LENGTH]; + CHAR longurl[4*INTERNET_MAX_URL_LENGTH]; DWORD dwSize; DWORD urllen; HRESULT hr; @@ -845,6 +846,24 @@ static void test_UrlCanonicalizeA(void) "got 0x%x with %u and size %u for '%s' and %u (expected 'S_OK' and size %u)\n", hr, GetLastError(), dwSize, szReturnUrl, lstrlenA(szReturnUrl), urllen); + /* length is set to 0 */ + dwSize=0; + memset(szReturnUrl, '#', urllen+4); + szReturnUrl[urllen+4] = '\0'; + SetLastError(0xdeadbeef); + hr = pUrlCanonicalizeA(winehqA, szReturnUrl, &dwSize, URL_WININET_COMPATIBILITY | URL_ESCAPE_UNSAFE); + ok( (hr == E_INVALIDARG) && (dwSize == 0), + "got 0x%x with %u and size %u for '%s' and %u (expected 'E_INVALIDARG' and size %u)\n", + hr, GetLastError(), dwSize, szReturnUrl, lstrlenA(szReturnUrl), 0); + + /* url length > INTERNET_MAX_URL_SIZE */ + dwSize=sizeof(szReturnUrl); + memset(longurl, 'a', sizeof(longurl)); + memcpy(longurl, winehqA, sizeof(winehqA)-1); + longurl[sizeof(longurl)-1] = '\0'; + hr = pUrlCanonicalizeA(longurl, szReturnUrl, &dwSize, URL_WININET_COMPATIBILITY | URL_ESCAPE_UNSAFE); + ok(hr == S_OK, "hr = %x\n", hr); + test_url_canonicalize(-1, "", 0, S_OK, S_FALSE /* Vista/win2k8 */, "", FALSE); /* test url-modification */

1 0

Piotr Caban : shlwapi: Make UrlCanonicalizeA working with long urls (> INTERNET_MAX_URL_LENGTH).
by Alexandre Julliard 08 May '10

08 May '10

Module: wine Branch: master Commit: 40119690b8c86d227a3b9bffeceb79d328ea4424 URL: http://source.winehq.org/git/wine.git/?a=commit;h=40119690b8c86d227a3b9bffe… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Sat May 8 18:00:49 2010 +0200 shlwapi: Make UrlCanonicalizeA working with long urls (>INTERNET_MAX_URL_LENGTH). --- dlls/shlwapi/url.c | 42 ++++++++++++++++++------------------------ 1 files changed, 18 insertions(+), 24 deletions(-) diff --git a/dlls/shlwapi/url.c b/dlls/shlwapi/url.c index 28ea818..ad7d989 100644 --- a/dlls/shlwapi/url.c +++ b/dlls/shlwapi/url.c @@ -232,40 +232,34 @@ HRESULT WINAPI ParseURLW(LPCWSTR x, PARSEDURLW *y) HRESULT WINAPI UrlCanonicalizeA(LPCSTR pszUrl, LPSTR pszCanonicalized, LPDWORD pcchCanonicalized, DWORD dwFlags) { - LPWSTR base, canonical; + LPWSTR url, canonical; HRESULT ret; - DWORD len, len2; + DWORD len; TRACE("(%s, %p, %p, 0x%08x) *pcchCanonicalized: %d\n", debugstr_a(pszUrl), pszCanonicalized, pcchCanonicalized, dwFlags, pcchCanonicalized ? *pcchCanonicalized : -1); - if(!pszUrl || !pszCanonicalized || !pcchCanonicalized) + if(!pszUrl || !pszCanonicalized || !pcchCanonicalized || !*pcchCanonicalized) return E_INVALIDARG; - base = HeapAlloc(GetProcessHeap(), 0, - (2*INTERNET_MAX_URL_LENGTH) * sizeof(WCHAR)); - canonical = base + INTERNET_MAX_URL_LENGTH; + len = strlen(pszUrl)+1; + url = HeapAlloc(GetProcessHeap(), 0, len*sizeof(WCHAR)); + canonical = HeapAlloc(GetProcessHeap(), 0, *pcchCanonicalized*sizeof(WCHAR)); + if(!url || !canonical) { + HeapFree(GetProcessHeap(), 0, url); + HeapFree(GetProcessHeap(), 0, canonical); + return E_OUTOFMEMORY; + } - MultiByteToWideChar(0, 0, pszUrl, -1, base, INTERNET_MAX_URL_LENGTH); - len = INTERNET_MAX_URL_LENGTH; + MultiByteToWideChar(0, 0, pszUrl, -1, url, len); - ret = UrlCanonicalizeW(base, canonical, &len, dwFlags); - if (ret != S_OK) { - *pcchCanonicalized = len * 2; - HeapFree(GetProcessHeap(), 0, base); - return ret; - } + ret = UrlCanonicalizeW(url, canonical, pcchCanonicalized, dwFlags); + if(ret == S_OK) + WideCharToMultiByte(0, 0, canonical, -1, pszCanonicalized, + *pcchCanonicalized+1, 0, 0); - len2 = WideCharToMultiByte(0, 0, canonical, -1, 0, 0, 0, 0); - if (len2 > *pcchCanonicalized) { - *pcchCanonicalized = len2; - HeapFree(GetProcessHeap(), 0, base); - return E_POINTER; - } - WideCharToMultiByte(0, 0, canonical, -1, pszCanonicalized, *pcchCanonicalized, 0, 0); - *pcchCanonicalized = len; - HeapFree(GetProcessHeap(), 0, base); - return S_OK; + HeapFree(GetProcessHeap(), 0, canonical); + return ret; } /*************************************************************************

1 0

Piotr Caban : shlwapi: Make UrlCanonicalizeW working with long urls (> INTERNET_MAX_URL_LENGTH).
by Alexandre Julliard 08 May '10

08 May '10

Module: wine Branch: master Commit: fc543890d103d87f1a343c924278f89d953071b9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=fc543890d103d87f1a343c924… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Sat May 8 18:00:36 2010 +0200 shlwapi: Make UrlCanonicalizeW working with long urls (>INTERNET_MAX_URL_LENGTH). --- dlls/shlwapi/url.c | 10 ++++++---- 1 files changed, 6 insertions(+), 4 deletions(-) diff --git a/dlls/shlwapi/url.c b/dlls/shlwapi/url.c index cd83b45..28ea818 100644 --- a/dlls/shlwapi/url.c +++ b/dlls/shlwapi/url.c @@ -287,11 +287,12 @@ HRESULT WINAPI UrlCanonicalizeW(LPCWSTR pszUrl, LPWSTR pszCanonicalized, static const WCHAR wszFile[] = {'f','i','l','e',':'}; static const WCHAR wszRes[] = {'r','e','s',':'}; static const WCHAR wszLocalhost[] = {'l','o','c','a','l','h','o','s','t'}; + static const WCHAR wszFilePrefix[] = {'f','i','l','e',':','/','/','/'}; TRACE("(%s, %p, %p, 0x%08x) *pcchCanonicalized: %d\n", debugstr_w(pszUrl), pszCanonicalized, pcchCanonicalized, dwFlags, pcchCanonicalized ? *pcchCanonicalized : -1); - if(!pszUrl || !pszCanonicalized || !pcchCanonicalized) + if(!pszUrl || !pszCanonicalized || !pcchCanonicalized || !*pcchCanonicalized) return E_INVALIDARG; if(!*pszUrl) { @@ -300,8 +301,9 @@ HRESULT WINAPI UrlCanonicalizeW(LPCWSTR pszUrl, LPWSTR pszCanonicalized, } nByteLen = (strlenW(pszUrl) + 1) * sizeof(WCHAR); /* length in bytes */ + /* Allocate memory for simplified URL (before escaping) */ lpszUrlCpy = HeapAlloc(GetProcessHeap(), 0, - INTERNET_MAX_URL_LENGTH * sizeof(WCHAR)); + nByteLen+sizeof(wszFilePrefix)+sizeof(WCHAR)); if((dwFlags & URL_FILE_USE_PATHURL) && nByteLen >= sizeof(wszFile) && !memcmp(wszFile, pszUrl, sizeof(wszFile))) @@ -328,8 +330,6 @@ HRESULT WINAPI UrlCanonicalizeW(LPCWSTR pszUrl, LPWSTR pszCanonicalized, state = 0; if(pszUrl[1] == ':') { /* Assume path */ - static const WCHAR wszFilePrefix[] = {'f','i','l','e',':','/','/','/'}; - memcpy(wk2, wszFilePrefix, sizeof(wszFilePrefix)); wk2 += sizeof(wszFilePrefix)/sizeof(WCHAR); if (dwFlags & URL_FILE_USE_PATHURL) @@ -833,6 +833,8 @@ HRESULT WINAPI UrlCombineW(LPCWSTR pszBase, LPCWSTR pszRelative, if (ret == S_OK) { /* Reuse mrelative as temp storage as its already allocated and not needed anymore */ + if(*pcchCombined == 0) + *pcchCombined = 1; ret = UrlCanonicalizeW(preliminary, mrelative, pcchCombined, (dwFlags & ~URL_FILE_USE_PATHURL)); if(SUCCEEDED(ret) && pszCombined) { lstrcpyW(pszCombined, mrelative);

1 0

Piotr Caban : include: Fix typo in shlwapi.h.
by Alexandre Julliard 08 May '10

08 May '10

Module: wine Branch: master Commit: f4f9b1acdf5019d5fd7e15fae7019784f093472d URL: http://source.winehq.org/git/wine.git/?a=commit;h=f4f9b1acdf5019d5fd7e15fae… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Sat May 8 18:00:17 2010 +0200 include: Fix typo in shlwapi.h. --- include/shlwapi.h | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/include/shlwapi.h b/include/shlwapi.h index d4362ac..e17789b 100644 --- a/include/shlwapi.h +++ b/include/shlwapi.h @@ -643,7 +643,7 @@ HRESULT WINAPI UrlApplySchemeW(LPCWSTR,LPWSTR,LPDWORD,DWORD); HRESULT WINAPI UrlCanonicalizeA(LPCSTR,LPSTR,LPDWORD,DWORD); HRESULT WINAPI UrlCanonicalizeW(LPCWSTR,LPWSTR,LPDWORD,DWORD); -#define UrlCanonicalize WINELIB_NAME_AW(UrlCanoncalize) +#define UrlCanonicalize WINELIB_NAME_AW(UrlCanonicalize) HRESULT WINAPI UrlCombineA(LPCSTR,LPCSTR,LPSTR,LPDWORD,DWORD); HRESULT WINAPI UrlCombineW(LPCWSTR,LPCWSTR,LPWSTR,LPDWORD,DWORD);

1 0

Paul Vriens : winhttp/tests: Show the correct error.
by Alexandre Julliard 08 May '10

08 May '10

Module: wine Branch: master Commit: f8470fe408f70bbe3ef67911d6fb0a4adcac23cd URL: http://source.winehq.org/git/wine.git/?a=commit;h=f8470fe408f70bbe3ef67911d… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Sat May 8 11:39:51 2010 +0200 winhttp/tests: Show the correct error. --- dlls/winhttp/tests/winhttp.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/dlls/winhttp/tests/winhttp.c b/dlls/winhttp/tests/winhttp.c index bf2db07..26477ee 100644 --- a/dlls/winhttp/tests/winhttp.c +++ b/dlls/winhttp/tests/winhttp.c @@ -1654,7 +1654,8 @@ static void test_resolve_timeout(void) SetLastError(0xdeadbeef); ret = WinHttpSendRequest(req, NULL, 0, NULL, 0, 0, 0); ok(!ret, "sent request\n"); - ok(GetLastError() == ERROR_WINHTTP_NAME_NOT_RESOLVED, "expected ERROR_WINHTTP_NAME_NOT_RESOLVED got %u\n", ret); + ok(GetLastError() == ERROR_WINHTTP_NAME_NOT_RESOLVED, + "expected ERROR_WINHTTP_NAME_NOT_RESOLVED got %u\n", GetLastError()); WinHttpCloseHandle(req); WinHttpCloseHandle(con);

1 0

Aric Stewart : usp10: Add ETO_IGNORELANGUAGE when calling ExtTextOut so we do not re-reverse already formated RTL strings inside of GDI .
by Alexandre Julliard 08 May '10

08 May '10

Module: wine Branch: master Commit: 421a997085844bac9e40ca79992abde170d5f39d URL: http://source.winehq.org/git/wine.git/?a=commit;h=421a997085844bac9e40ca799… Author: Aric Stewart <aric(a)codeweavers.com> Date: Fri May 7 22:10:26 2010 -0500 usp10: Add ETO_IGNORELANGUAGE when calling ExtTextOut so we do not re-reverse already formated RTL strings inside of GDI. --- dlls/usp10/usp10.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/usp10/usp10.c b/dlls/usp10/usp10.c index 43a3b5e..b9294e2 100644 --- a/dlls/usp10/usp10.c +++ b/dlls/usp10/usp10.c @@ -1600,6 +1600,7 @@ HRESULT WINAPI ScriptTextOut(const HDC hdc, SCRIPT_CACHE *psc, int x, int y, UIN if (!piAdvance || !psa || !pwGlyphs) return E_INVALIDARG; fuOptions &= ETO_CLIPPED + ETO_OPAQUE; + fuOptions |= ETO_IGNORELANGUAGE; if (!psa->fNoGlyphIndex) /* Have Glyphs? */ fuOptions |= ETO_GLYPH_INDEX; /* Say don't do translation to glyph */

1 0

Andrew Eikum : user32: Don't use DLGTEMPLATE' s ExStyle when creating dialogs.
by Alexandre Julliard 08 May '10

08 May '10

Module: wine Branch: master Commit: 897f11d33178f1c5b591ba1a516e79ec8834055e URL: http://source.winehq.org/git/wine.git/?a=commit;h=897f11d33178f1c5b591ba1a5… Author: Andrew Eikum <aeikum(a)codeweavers.com> Date: Fri May 7 17:16:41 2010 -0500 user32: Don't use DLGTEMPLATE's ExStyle when creating dialogs. --- dlls/user32/dialog.c | 1 + dlls/user32/tests/dialog.c | 2 +- 2 files changed, 2 insertions(+), 1 deletions(-) diff --git a/dlls/user32/dialog.c b/dlls/user32/dialog.c index 79f5182..af241a2 100644 --- a/dlls/user32/dialog.c +++ b/dlls/user32/dialog.c @@ -498,6 +498,7 @@ static HWND DIALOG_CreateIndirect( HINSTANCE hInst, LPCVOID dlgTemplate, if (!dlgTemplate) return 0; dlgTemplate = DIALOG_ParseTemplate32( dlgTemplate, &template ); + template.exStyle = 0; /* exstyle is ignored */ /* Load menu */ diff --git a/dlls/user32/tests/dialog.c b/dlls/user32/tests/dialog.c index d18ba0e..b5d6271 100644 --- a/dlls/user32/tests/dialog.c +++ b/dlls/user32/tests/dialog.c @@ -1061,7 +1061,7 @@ static void test_MessageBoxFontTest(void) 0x01,0x00, /* Version */ 0xff,0xff, /* Extended template marker */ 0x00,0x00,0x00,0x00, /* Context Help ID */ - 0x00,0x00,0x00,0x00, /* Extended style */ + 0x40,0x00,0x00,0x00, /* Extended style (WM_EX_MDICHILD) */ 0xc0,0x00,0xc8,0x80, /* Style (WS_SYSMENU|WS_CAPTION|WS_POPUP|DS_SETFONT|DS_MODALFRAME) */ 0x01,0x00, /* Control count */ 0x00,0x00, /* X */

1 0

← Newer
1
...
67
68
69
70
71
72
73
...
96
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 2010