December 2010 - Wine-Commits

Rico Schüller : wined3d: Fix memset size in IWineD3DVolumeImpl_Unmap.
by Alexandre Julliard Dec. 20, 2010

Dec. 20, 2010

Module: wine Branch: master Commit: 5b985bcb03c153c1304f89d2f901e702cc2d3e6f URL: http://source.winehq.org/git/wine.git/?a=commit;h=5b985bcb03c153c1304f89d2f… Author: Rico Schüller <kgbricola(a)web.de> Date: Sun Dec 19 11:49:45 2010 +0100 wined3d: Fix memset size in IWineD3DVolumeImpl_Unmap. --- dlls/wined3d/volume.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/wined3d/volume.c b/dlls/wined3d/volume.c index 66bf86a..719fddc 100644 --- a/dlls/wined3d/volume.c +++ b/dlls/wined3d/volume.c @@ -268,7 +268,7 @@ static HRESULT WINAPI IWineD3DVolumeImpl_Unmap(IWineD3DVolume *iface) } TRACE("(%p) : unlocking volume\n", This); This->locked = FALSE; - memset(&This->lockedBox, 0, sizeof(RECT)); + memset(&This->lockedBox, 0, sizeof(This->lockedBox)); return WINED3D_OK; }

1 0

Nikolay Sivov : oleaut32/varformat: Fix day string representation format output.
by Alexandre Julliard Dec. 20, 2010

Dec. 20, 2010

Module: wine Branch: master Commit: 495792e228e3f96cf54ac956307db70d03c9523c URL: http://source.winehq.org/git/wine.git/?a=commit;h=495792e228e3f96cf54ac9563… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Sun Dec 19 19:25:50 2010 +0300 oleaut32/varformat: Fix day string representation format output. --- dlls/oleaut32/tests/varformat.c | 4 +++- dlls/oleaut32/varformat.c | 4 ++-- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/dlls/oleaut32/tests/varformat.c b/dlls/oleaut32/tests/varformat.c index 2d4ef62..d83f218 100644 --- a/dlls/oleaut32/tests/varformat.c +++ b/dlls/oleaut32/tests/varformat.c @@ -212,7 +212,9 @@ static const FMTDATERES VarFormat_date_results[] = { 2.525, "hh :mm:mm", "12 :36:01" }, { 2.525, "dd :mm:mm", "01 :01:01" }, { 2.525, "dd :mm:nn", "01 :01:36" }, - { 2.725, "hh:nn:ss A/P", "05:24:00 P" } + { 2.725, "hh:nn:ss A/P", "05:24:00 P" }, + { 40531.0, "dddd", "Sunday" }, + { 40531.0, "ddd", "Sun" } }; #define VNUMFMT(vt,v) \ diff --git a/dlls/oleaut32/varformat.c b/dlls/oleaut32/varformat.c index f297641..529528b 100644 --- a/dlls/oleaut32/varformat.c +++ b/dlls/oleaut32/varformat.c @@ -1725,14 +1725,14 @@ static HRESULT VARIANT_FormatDate(LPVARIANT pVarIn, LPOLESTR lpszFormat, case FMT_DATE_DAY_SHORT: /* FIXME: VARIANT_CALENDAR HIJRI should cause Hijri output */ TRACE("short day\n"); - localeValue = LOCALE_SABBREVDAYNAME1 + udate.st.wMonth - 1; + localeValue = LOCALE_SABBREVDAYNAME1 + (udate.st.wDayOfWeek + 6)%7; defaultChar = '?'; break; case FMT_DATE_DAY_LONG: /* FIXME: VARIANT_CALENDAR HIJRI should cause Hijri output */ TRACE("long day\n"); - localeValue = LOCALE_SDAYNAME1 + udate.st.wMonth - 1; + localeValue = LOCALE_SDAYNAME1 + (udate.st.wDayOfWeek + 6)%7; defaultChar = '?'; break;

1 0

David Hedberg : wininet: Respect SECURITY_FLAG_IGNORE_CERT_DATE_INVALID.
by Alexandre Julliard Dec. 20, 2010

Dec. 20, 2010

Module: wine Branch: master Commit: 3ce9304592c16d5bdc4053460b9d266f15cc551f URL: http://source.winehq.org/git/wine.git/?a=commit;h=3ce9304592c16d5bdc4053460… Author: David Hedberg <dhedberg(a)codeweavers.com> Date: Sun Dec 19 21:27:23 2010 +0100 wininet: Respect SECURITY_FLAG_IGNORE_CERT_DATE_INVALID. --- dlls/wininet/netconnection.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/dlls/wininet/netconnection.c b/dlls/wininet/netconnection.c index 6a06b9a..958abfb 100644 --- a/dlls/wininet/netconnection.c +++ b/dlls/wininet/netconnection.c @@ -248,7 +248,8 @@ static DWORD netconn_verify_cert(PCCERT_CONTEXT cert, HCERTSTORE store, CERT_TRUST_IS_REVOKED | CERT_TRUST_IS_NOT_VALID_FOR_USAGE; - if (chain->TrustStatus.dwErrorStatus & CERT_TRUST_IS_NOT_TIME_VALID) + if (chain->TrustStatus.dwErrorStatus & CERT_TRUST_IS_NOT_TIME_VALID && + !(security_flags & SECURITY_FLAG_IGNORE_CERT_DATE_INVALID)) err = ERROR_INTERNET_SEC_CERT_DATE_INVALID; else if (chain->TrustStatus.dwErrorStatus & CERT_TRUST_IS_UNTRUSTED_ROOT &&

1 0

David Hedberg : wininet: Make module handle scope global.
by Alexandre Julliard Dec. 20, 2010

Dec. 20, 2010

Module: wine Branch: master Commit: 46be41271cd6f4d6264e751ab72c9f5400c90e5e URL: http://source.winehq.org/git/wine.git/?a=commit;h=46be41271cd6f4d6264e751ab… Author: David Hedberg <dhedberg(a)codeweavers.com> Date: Sun Dec 19 21:34:27 2010 +0100 wininet: Make module handle scope global. --- dlls/wininet/dialogs.c | 5 ++--- dlls/wininet/internet.c | 2 +- dlls/wininet/internet.h | 2 ++ 3 files changed, 5 insertions(+), 4 deletions(-) diff --git a/dlls/wininet/dialogs.c b/dlls/wininet/dialogs.c index 2c4a527..b3713a5 100644 --- a/dlls/wininet/dialogs.c +++ b/dlls/wininet/dialogs.c @@ -490,7 +490,6 @@ DWORD WINAPI InternetErrorDlg(HWND hWnd, HINTERNET hRequest, DWORD dwError, DWORD dwFlags, LPVOID* lppvData) { struct WININET_ErrorDlgParams params; - HMODULE hwininet = GetModuleHandleA( "wininet.dll" ); INT dwStatus; TRACE("%p %p %d %08x %p\n", hWnd, hRequest, dwError, dwFlags, lppvData); @@ -512,10 +511,10 @@ DWORD WINAPI InternetErrorDlg(HWND hWnd, HINTERNET hRequest, switch (dwStatus) { case HTTP_STATUS_PROXY_AUTH_REQ: - return DialogBoxParamW( hwininet, MAKEINTRESOURCEW( IDD_PROXYDLG ), + return DialogBoxParamW( WININET_hModule, MAKEINTRESOURCEW( IDD_PROXYDLG ), hWnd, WININET_ProxyPasswordDialog, (LPARAM) &params ); case HTTP_STATUS_DENIED: - return DialogBoxParamW( hwininet, MAKEINTRESOURCEW( IDD_AUTHDLG ), + return DialogBoxParamW( WININET_hModule, MAKEINTRESOURCEW( IDD_AUTHDLG ), hWnd, WININET_PasswordDialog, (LPARAM) &params ); default: WARN("unhandled status %u\n", dwStatus); diff --git a/dlls/wininet/internet.c b/dlls/wininet/internet.c index 95776df..002f4e4 100644 --- a/dlls/wininet/internet.c +++ b/dlls/wininet/internet.c @@ -88,7 +88,7 @@ typedef struct } WITHREADERROR, *LPWITHREADERROR; static DWORD g_dwTlsErrIndex = TLS_OUT_OF_INDEXES; -static HMODULE WININET_hModule; +HMODULE WININET_hModule; #define HANDLE_CHUNK_SIZE 0x10 diff --git a/dlls/wininet/internet.h b/dlls/wininet/internet.h index 86b16bd..3963578 100644 --- a/dlls/wininet/internet.h +++ b/dlls/wininet/internet.h @@ -47,6 +47,8 @@ #define ioctlsocket ioctl #endif /* __MINGW32__ */ +extern HMODULE WININET_hModule; + /* used for netconnection.c stuff */ typedef struct {

1 0

David Hedberg : shell32: Return S_OK in ::SetPropertyBag.
by Alexandre Julliard Dec. 20, 2010

Dec. 20, 2010

Module: wine Branch: master Commit: dfc2f20b892c565356529c199d78c18dcf944974 URL: http://source.winehq.org/git/wine.git/?a=commit;h=dfc2f20b892c565356529c199… Author: David Hedberg <david.hedberg(a)gmail.com> Date: Sun Dec 19 22:11:23 2010 +0100 shell32: Return S_OK in ::SetPropertyBag. --- dlls/shell32/ebrowser.c | 10 +++++++++- dlls/shell32/tests/ebrowser.c | 7 +++++++ 2 files changed, 16 insertions(+), 1 deletions(-) diff --git a/dlls/shell32/ebrowser.c b/dlls/shell32/ebrowser.c index e6d1165..f781082 100644 --- a/dlls/shell32/ebrowser.c +++ b/dlls/shell32/ebrowser.c @@ -931,7 +931,15 @@ static HRESULT WINAPI IExplorerBrowser_fnSetPropertyBag(IExplorerBrowser *iface, ExplorerBrowserImpl *This = (ExplorerBrowserImpl*)iface; FIXME("stub, %p (%s)\n", This, debugstr_w(pszPropertyBag)); - return E_NOTIMPL; + if(!pszPropertyBag) + return E_INVALIDARG; + + /* FIXME: This method is currently useless as we don't save any + * settings anywhere, but at least one application breaks if we + * return E_NOTIMPL. + */ + + return S_OK; } static HRESULT WINAPI IExplorerBrowser_fnSetEmptyText(IExplorerBrowser *iface, diff --git a/dlls/shell32/tests/ebrowser.c b/dlls/shell32/tests/ebrowser.c index 99368d0..15c2cb4 100644 --- a/dlls/shell32/tests/ebrowser.c +++ b/dlls/shell32/tests/ebrowser.c @@ -1037,6 +1037,7 @@ static void test_basics(void) HDWP hdwp; RECT rc; HRESULT hr; + static const WCHAR winetest[] = {'W','i','n','e','T','e','s','t',0}; ebrowser_instantiate(&peb); ebrowser_initialize(peb); @@ -1145,6 +1146,12 @@ static void test_basics(void) hr = IExplorerBrowser_SetFolderSettings(peb, &fs); todo_wine ok(hr == E_INVALIDARG, "got (0x%08x)\n", hr); + /* SetPropertyBag */ + hr = IExplorerBrowser_SetPropertyBag(peb, NULL); + ok(hr == E_INVALIDARG, "Got 0x%08x\n", hr); + hr = IExplorerBrowser_SetPropertyBag(peb, winetest); + ok(hr == S_OK, "Got 0x%08x\n", hr); + /* TODO: Test after browsing somewhere. */ IExplorerBrowser_Destroy(peb);

1 0

David Hedberg : shell32: IShellItem:: GetAttributes should also work on the desktop shellfolder.
by Alexandre Julliard Dec. 20, 2010

Dec. 20, 2010

Module: wine Branch: master Commit: 0d87e1f9fdfb96648ce03709e9c6bca285ec5c73 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0d87e1f9fdfb96648ce03709e… Author: David Hedberg <david.hedberg(a)gmail.com> Date: Sun Dec 19 23:15:12 2010 +0100 shell32: IShellItem::GetAttributes should also work on the desktop shellfolder. --- dlls/shell32/shellitem.c | 5 +++- dlls/shell32/tests/shlfolder.c | 44 ++++++++++++++++++++++++++++++++++++++++ 2 files changed, 48 insertions(+), 1 deletions(-) diff --git a/dlls/shell32/shellitem.c b/dlls/shell32/shellitem.c index d694503..fab7541 100644 --- a/dlls/shell32/shellitem.c +++ b/dlls/shell32/shellitem.c @@ -259,7 +259,10 @@ static HRESULT WINAPI ShellItem_GetAttributes(IShellItem *iface, SFGAOF sfgaoMas TRACE("(%p,%x,%p)\n", iface, sfgaoMask, psfgaoAttribs); - ret = ShellItem_get_parent_shellfolder(This, &parent_folder); + if (_ILIsDesktop(This->pidl)) + ret = SHGetDesktopFolder(&parent_folder); + else + ret = ShellItem_get_parent_shellfolder(This, &parent_folder); if (SUCCEEDED(ret)) { child_pidl = ILFindLastID(This->pidl); diff --git a/dlls/shell32/tests/shlfolder.c b/dlls/shell32/tests/shlfolder.c index c72eed2..0bf2ec7 100644 --- a/dlls/shell32/tests/shlfolder.c +++ b/dlls/shell32/tests/shlfolder.c @@ -3670,6 +3670,49 @@ static void test_ShellItemBindToHandler(void) pILFree(pidl_desktop); } +void test_ShellItemGetAttributes(void) +{ + IShellItem *psi; + LPITEMIDLIST pidl_desktop; + SFGAOF sfgao; + HRESULT hr; + + if(!pSHCreateShellItem) + { + skip("SHCreateShellItem missing.\n"); + return; + } + + hr = pSHGetSpecialFolderLocation(NULL, CSIDL_DESKTOP, &pidl_desktop); + ok(hr == S_OK, "Got 0x%08x\n", hr); + if(SUCCEEDED(hr)) + { + hr = pSHCreateShellItem(NULL, NULL, pidl_desktop, &psi); + ok(hr == S_OK, "Got 0x%08x\n", hr); + pILFree(pidl_desktop); + } + if(FAILED(hr)) + { + skip("Skipping tests."); + return; + } + + if(0) + { + /* Crashes on native (Win 7) */ + hr = IShellItem_GetAttributes(psi, 0, NULL); + ok(hr == S_OK, "Got 0x%08x\n", hr); + } + + /* Test GetAttributes on the desktop folder. */ + sfgao = 0xdeadbeef; + hr = IShellItem_GetAttributes(psi, SFGAO_FOLDER, &sfgao); + ok(hr == S_OK || broken(hr == E_FAIL) /* <Vista */, "Got 0x%08x\n", hr); + ok(sfgao == SFGAO_FOLDER || broken(sfgao == 0) /* <Vista */, "Got 0x%08x\n", sfgao); + + IShellItem_Release(psi); +} + static void test_SHParseDisplayName(void) { LPITEMIDLIST pidl1, pidl2; @@ -4432,6 +4475,7 @@ START_TEST(shlfolder) test_ShellItemCompare(); test_SHChangeNotify(); test_ShellItemBindToHandler(); + test_ShellItemGetAttributes(); OleUninitialize(); }

1 0

David Hedberg : include: Add IShellItem2 interface declaration.
by Alexandre Julliard Dec. 20, 2010

Dec. 20, 2010

Module: wine Branch: master Commit: 7b67d4f5c66b5a435dd996bc67adf46c6bceffbb URL: http://source.winehq.org/git/wine.git/?a=commit;h=7b67d4f5c66b5a435dd996bc6… Author: David Hedberg <david.hedberg(a)gmail.com> Date: Sun Dec 19 23:15:11 2010 +0100 include: Add IShellItem2 interface declaration. --- include/shobjidl.idl | 69 ++++++++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 69 insertions(+), 0 deletions(-) diff --git a/include/shobjidl.idl b/include/shobjidl.idl index fece0dd..176b63f 100644 --- a/include/shobjidl.idl +++ b/include/shobjidl.idl @@ -389,6 +389,75 @@ interface IShellItem : IUnknown [out] int *piOrder); } +/***************************************************************************** + * IShellItem2 interface + */ +[ + object, + uuid(7E9FB0D3-919F-4307-AB2E-9B1860310C93), + pointer_default(unique) +] +interface IShellItem2 : IShellItem +{ + HRESULT GetPropertyStore( + [in] GETPROPERTYSTOREFLAGS flags, + [in] REFIID riid, + [out, iid_is(riid)] void **ppv); + + HRESULT GetPropertyStoreWithCreateObject( + [in] GETPROPERTYSTOREFLAGS flags, + [in] IUnknown *punkCreateObject, + [in] REFIID riid, + [out, iid_is(riid)] void **ppv); + + HRESULT GetPropertyStoreForKeys( + [in, size_is(cKeys)] const PROPERTYKEY *rgKeys, + [in] UINT cKeys, + [in] GETPROPERTYSTOREFLAGS flags, + [in] REFIID riid, + [out, iid_is(riid)] void **ppv); + + HRESULT GetPropertyDescriptionList( + [in] REFPROPERTYKEY keyType, + [in] REFIID riid, + [out, iid_is(riid)] void **ppv); + + HRESULT Update( + [in, unique] IBindCtx *pbc); + + HRESULT GetProperty( + [in] REFPROPERTYKEY key, + [out] PROPVARIANT *ppropvar); + + HRESULT GetCLSID( + [in] REFPROPERTYKEY key, + [out] CLSID *pclsid); + + HRESULT GetFileTime ( + [in] REFPROPERTYKEY key, + [out] FILETIME *pft); + + HRESULT GetInt32( + [in] REFPROPERTYKEY key, + [out] int *pi); + + HRESULT GetString( + [in] REFPROPERTYKEY key, + [out, string] LPWSTR *ppsz); + + HRESULT GetUInt32( + [in] REFPROPERTYKEY key, + [out] ULONG *pui); + + HRESULT GetUInt64( + [in] REFPROPERTYKEY key, + [out] ULONGLONG *pull); + + HRESULT GetBool( + [in] REFPROPERTYKEY key, + [out] BOOL *pf); +} + typedef enum tagNWMF { NWMF_UNLOADING = 0x0001, NWMF_USERINITED = 0x0002,

1 0

David Hedberg : wininet: Add a simple certificate dialog to InternetErrorDlg.
by Alexandre Julliard Dec. 20, 2010

Dec. 20, 2010

Module: wine Branch: master Commit: 08fe24de06ea46c1e9c03595a162a8b42b143c37 URL: http://source.winehq.org/git/wine.git/?a=commit;h=08fe24de06ea46c1e9c03595a… Author: David Hedberg <dhedberg(a)codeweavers.com> Date: Sun Dec 19 21:34:28 2010 +0100 wininet: Add a simple certificate dialog to InternetErrorDlg. --- dlls/wininet/dialogs.c | 125 ++++++++++++++++++++++++++++++++- dlls/wininet/resource.h | 6 ++ dlls/wininet/tests/Makefile.in | 2 +- dlls/wininet/tests/internet.c | 148 ++++++++++++++++++++++++++++++++++++++++ dlls/wininet/wininet_En.rc | 16 ++++ dlls/wininet/wininet_Sv.rc | 16 ++++ 6 files changed, 308 insertions(+), 5 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=08fe24de06ea46c1e9c03…

1 0

David Hedberg : urlmon: Set BINDF_NEEDFILE if BINDF_PULLDATA is not specified.
by Alexandre Julliard Dec. 20, 2010

Dec. 20, 2010

Module: wine Branch: master Commit: 7af43bb1b234c318b4016f0bfb7296d3007c20b9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7af43bb1b234c318b4016f0bf… Author: David Hedberg <dhedberg(a)codeweavers.com> Date: Sun Dec 19 21:26:06 2010 +0100 urlmon: Set BINDF_NEEDFILE if BINDF_PULLDATA is not specified. --- dlls/urlmon/binding.c | 2 +- dlls/urlmon/tests/url.c | 25 +++++++++++++++++++++++-- 2 files changed, 24 insertions(+), 3 deletions(-) diff --git a/dlls/urlmon/binding.c b/dlls/urlmon/binding.c index 699bf98..beedddd 100644 --- a/dlls/urlmon/binding.c +++ b/dlls/urlmon/binding.c @@ -1496,7 +1496,7 @@ static HRESULT Binding_Create(IMoniker *mon, Binding *binding_ctx, IUri *uri, IB if(to_obj) ret->bindinfo.dwOptions |= 0x100000; - if(!(ret->bindf & BINDF_ASYNCHRONOUS)) { + if(!(ret->bindf & BINDF_ASYNCHRONOUS) || !(ret->bindf & BINDF_PULLDATA)) { ret->bindf |= BINDF_NEEDFILE; ret->use_cache_file = TRUE; }else if(!is_urlmon_protocol(uri)) { diff --git a/dlls/urlmon/tests/url.c b/dlls/urlmon/tests/url.c index 818fff9..c635da3 100644 --- a/dlls/urlmon/tests/url.c +++ b/dlls/urlmon/tests/url.c @@ -192,6 +192,7 @@ static HRESULT binding_hres; static BOOL have_IHttpNegotiate2, use_bscex, is_async_prot; static BOOL test_redirect, use_cache_file, callback_read, no_callback, test_abort; static WCHAR cache_file_name[MAX_PATH]; +static BOOL only_check_prot_args = FALSE; static LPCWSTR urls[] = { WINE_ABOUT_URL, @@ -566,7 +567,8 @@ static HRESULT WINAPI Protocol_Start(IInternetProtocol *iface, LPCWSTR szUrl, ok(bind_info & BINDF_FROMURLMON, "BINDF_FROMURLMON is not set\n"); - if(filedwl_api || !is_urlmon_protocol(test_protocol) || !(bindf&BINDF_ASYNCSTORAGE) || tymed != TYMED_ISTREAM) + if(filedwl_api || !is_urlmon_protocol(test_protocol) || tymed != TYMED_ISTREAM || + !(bindf&BINDF_ASYNCSTORAGE) || !(bindf&BINDF_PULLDATA)) ok(bind_info & BINDF_NEEDFILE, "BINDF_NEEDFILE is not set\n"); else ok(!(bind_info & BINDF_NEEDFILE), "BINDF_NEEDFILE is set\n"); @@ -593,6 +595,9 @@ static HRESULT WINAPI Protocol_Start(IInternetProtocol *iface, LPCWSTR szUrl, ok(!bindinfo.pUnk, "bindinfo.pUnk = %p\n", bindinfo.pUnk); ok(!bindinfo.dwReserved, "bindinfo.dwReserved = %d\n", bindinfo.dwReserved); + if(only_check_prot_args) + return E_FAIL; + switch(test_protocol) { case MK_TEST: hres = IInternetProtocolSink_ReportProgress(pOIProtSink, @@ -1543,6 +1548,11 @@ static HRESULT WINAPI statusclb_OnStopBinding(IBindStatusCallbackEx *iface, HRES ok(GetCurrentThreadId() == thread_id, "wrong thread %d\n", GetCurrentThreadId()); + if(only_check_prot_args) { + todo_wine ok(hresult == S_OK, "Got %08x\n", hresult); + return S_OK; + } + /* ignore DNS failure */ if (hresult == HRESULT_FROM_WIN32(ERROR_INTERNET_NAME_NOT_RESOLVED)) return S_OK; @@ -2510,6 +2520,8 @@ static void test_BindToStorage(int protocol, DWORD flags, DWORD t) if(tymed == TYMED_FILE && (test_protocol == ABOUT_TEST || test_protocol == ITS_TEST)) binding_hres = INET_E_DATA_NOT_AVAILABLE; + if(only_check_prot_args) + SET_EXPECT(OnStopBinding); if(!no_callback) { SET_EXPECT(QueryInterface_IBindStatusCallbackEx); SET_EXPECT(GetBindInfo); @@ -2568,7 +2580,10 @@ static void test_BindToStorage(int protocol, DWORD flags, DWORD t) return; } - if(no_callback) { + if(only_check_prot_args) { + ok(hres == E_FAIL, "Got %08x\n", hres); + CHECK_CALLED(OnStopBinding); + }else if(no_callback) { if(emulate_protocol) WaitForSingleObject(complete_event2, INFINITE); ok(hres == S_OK, "IMoniker_BindToStorage failed: %08x\n", hres); @@ -3346,6 +3361,12 @@ START_TEST(url) trace("test failures...\n"); test_BindToStorage_fail(); + + bindf = BINDF_ASYNCHRONOUS | BINDF_ASYNCSTORAGE; + only_check_prot_args = TRUE; /* Fail after checking arguments to Protocol_Start */ + + trace("check emulated http protocol arguments...\n"); + test_BindToStorage(HTTP_TEST, BINDTEST_EMULATE, TYMED_ISTREAM); } DeleteFileA(wszIndexHtmlA);

1 0

Alexandre Julliard : loader: Define asm functions instead of inlines for x86-64 syscalls to avoid trouble with register constraints .
by Alexandre Julliard Dec. 20, 2010

Dec. 20, 2010

Module: wine Branch: master Commit: 61d2d80795d336cf344a1218642b93009396e984 URL: http://source.winehq.org/git/wine.git/?a=commit;h=61d2d80795d336cf344a12186… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Sat Dec 18 12:33:29 2010 +0100 loader: Define asm functions instead of inlines for x86-64 syscalls to avoid trouble with register constraints. --- loader/preloader.c | 125 +++++++++++++++++++--------------------------------- 1 files changed, 45 insertions(+), 80 deletions(-) diff --git a/loader/preloader.c b/loader/preloader.c index 995cf59..4344a16 100644 --- a/loader/preloader.c +++ b/loader/preloader.c @@ -371,93 +371,58 @@ __ASM_GLOBAL_FUNC(_start, "xorq %r11,%r11\n\t" "ret") -#define SYSCALL_RET(ret) (((ret) < 0 && (ret) > -4096) ? -1 : (ret)) - -static inline __attribute__((noreturn)) void wld_exit( int code ) -{ - for (;;) /* avoid warning */ - __asm__ __volatile__( "syscall" : : "a" (SYS_exit), "D" (code) ); -} - -static inline int wld_open( const char *name, int flags ) -{ - int ret; - __asm__ __volatile__( "syscall" : "=a" (ret) : "0" (SYS_open), "D" (name), "S" (flags) ); - return SYSCALL_RET(ret); -} - -static inline int wld_close( int fd ) -{ - int ret; - __asm__ __volatile__( "syscall" : "=a" (ret) : "0" (SYS_close), "D" (fd) ); - return SYSCALL_RET(ret); -} - -static inline ssize_t wld_read( int fd, void *buffer, size_t len ) -{ - int ret; - __asm__ __volatile__( "syscall" - : "=a" (ret) - : "0" (SYS_read), "D" (fd), "S" (buffer), "d" (len) - : "memory" ); - return SYSCALL_RET(ret); -} - -static inline ssize_t wld_write( int fd, const void *buffer, size_t len ) -{ - int ret; - __asm__ __volatile__( "syscall" : "=a" (ret) : "0" (SYS_write), "D" (fd), "S" (buffer), "d" (len) ); - return SYSCALL_RET(ret); -} - -static inline int wld_mprotect( const void *addr, size_t len, int prot ) -{ - int ret; - __asm__ __volatile__( "syscall" : "=a" (ret) : "0" (SYS_mprotect), "D" (addr), "S" (len), "d" (prot) ); - return SYSCALL_RET(ret); -} +#define SYSCALL_FUNC( name, nr ) \ + __ASM_GLOBAL_FUNC( name, \ + "movq $" #nr ",%rax\n\t" \ + "movq %rcx,%r10\n\t" \ + "syscall\n\t" \ + "leaq 4096(%rax),%rcx\n\t" \ + "movq $-1,%rdx\n\t" \ + "cmp $4096,%rcx\n\t" \ + "cmovb %rdx,%rax\n\t" \ + "ret" ) + +#define SYSCALL_NOERR( name, nr ) \ + __ASM_GLOBAL_FUNC( name, \ + "movq $" #nr ",%rax\n\t" \ + "syscall\n\t" \ + "ret" ) + +void wld_exit( int code ) __attribute__((noreturn)); +SYSCALL_NOERR( wld_exit, 60 /* SYS_exit */ ); + +ssize_t wld_read( int fd, void *buffer, size_t len ); +SYSCALL_FUNC( wld_read, 0 /* SYS_read */ ); + +ssize_t wld_write( int fd, const void *buffer, size_t len ); +SYSCALL_FUNC( wld_write, 1 /* SYS_write */ ); + +int wld_open( const char *name, int flags ); +SYSCALL_FUNC( wld_open, 2 /* SYS_open */ ); + +int wld_close( int fd ); +SYSCALL_FUNC( wld_close, 3 /* SYS_close */ ); void *wld_mmap( void *start, size_t len, int prot, int flags, int fd, off_t offset ); -__ASM_GLOBAL_FUNC( wld_mmap, - "movq %rcx,%r10\n\t" - "movq $9,%rax\n\t" /* SYS_mmap */ - "syscall\n\t" - "ret" ); +SYSCALL_FUNC( wld_mmap, 9 /* SYS_mmap */ ); -static inline uid_t wld_getuid(void) -{ - uid_t ret; - __asm__( "syscall" : "=a" (ret) : "0" (SYS_getuid) ); - return ret; -} +int wld_mprotect( const void *addr, size_t len, int prot ); +SYSCALL_FUNC( wld_mprotect, 10 /* SYS_mprotect */ ); -static inline uid_t wld_geteuid(void) -{ - uid_t ret; - __asm__( "syscall" : "=a" (ret) : "0" (SYS_geteuid) ); - return ret; -} +int wld_prctl( int code, int arg ); +SYSCALL_FUNC( wld_prctl, 157 /* SYS_prctl */ ); -static inline gid_t wld_getgid(void) -{ - gid_t ret; - __asm__( "syscall" : "=a" (ret) : "0" (SYS_getgid) ); - return ret; -} +uid_t wld_getuid(void); +SYSCALL_NOERR( wld_getuid, 102 /* SYS_getuid */ ); -static inline gid_t wld_getegid(void) -{ - gid_t ret; - __asm__( "syscall" : "=a" (ret) : "0" (SYS_getegid) ); - return ret; -} +gid_t wld_getgid(void); +SYSCALL_NOERR( wld_getgid, 104 /* SYS_getgid */ ); -static inline int wld_prctl( int code, int arg ) -{ - int ret; - __asm__ __volatile__( "syscall" : "=a" (ret) : "D" (SYS_prctl), "S" (code), "d" (arg) ); - return SYSCALL_RET(ret); -} +uid_t wld_geteuid(void); +SYSCALL_NOERR( wld_geteuid, 107 /* SYS_geteuid */ ); + +gid_t wld_getegid(void); +SYSCALL_NOERR( wld_getegid, 108 /* SYS_getegid */ ); #else #error preloader not implemented for this CPU

1 0