wine-commits September 2012

wine-commits@winehq.org

2 participants
828 discussions

Piotr Caban : wininet: Unlock urlcache entries that were locked for over a day.
by Alexandre Julliard 18 Sep '12

18 Sep '12

Module: wine Branch: master Commit: 6935b7c73b0368f4c2e9252019ecd7a9f6223c04 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6935b7c73b0368f4c2e925201… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Tue Sep 18 14:43:44 2012 +0200 wininet: Unlock urlcache entries that were locked for over a day. --- dlls/wininet/urlcache.c | 43 +++++++++++++++++++++++++++++++++++++------ 1 files changed, 37 insertions(+), 6 deletions(-) diff --git a/dlls/wininet/urlcache.c b/dlls/wininet/urlcache.c index 1b62fda..e1d189e 100644 --- a/dlls/wininet/urlcache.c +++ b/dlls/wininet/urlcache.c @@ -1553,6 +1553,38 @@ static BOOL URLCache_DeleteCacheDirectory(LPCWSTR lpszPath) } /*********************************************************************** + * URLCache_IsLocked (Internal) + * + * Checks if entry is locked. Unlocks it if possible. + */ +static BOOL URLCache_IsLocked(struct _HASH_ENTRY *hash_entry, URL_CACHEFILE_ENTRY *url_entry) +{ + FILETIME cur_time; + ULARGE_INTEGER acc_time, time; + + if ((hash_entry->dwHashKey & ((1<<HASHTABLE_FLAG_BITS)-1)) != HASHTABLE_LOCK) + return FALSE; + + GetSystemTimeAsFileTime(&cur_time); + time.u.LowPart = cur_time.dwLowDateTime; + time.u.HighPart = cur_time.dwHighDateTime; + + acc_time.u.LowPart = url_entry->LastAccessTime.dwLowDateTime; + acc_time.u.HighPart = url_entry->LastAccessTime.dwHighDateTime; + + time.QuadPart -= acc_time.QuadPart; + + /* check if entry was locked for at least a day */ + if(time.QuadPart > (ULONGLONG)24*60*60*10000000) { + URLCache_HashEntrySetFlags(hash_entry, HASHTABLE_URL); + url_entry->dwUseCount = 0; + return FALSE; + } + + return TRUE; +} + +/*********************************************************************** * FreeUrlCacheSpaceW (WININET.@) * * Frees up some cache. @@ -2193,9 +2225,8 @@ static BOOL DeleteUrlCacheEntryInternal(const URLCACHECONTAINER * pContainer, } pUrlEntry = (URL_CACHEFILE_ENTRY *)pEntry; - if ((pHashEntry->dwHashKey & ((1<<HASHTABLE_FLAG_BITS)-1)) == HASHTABLE_LOCK) + if(URLCache_IsLocked(pHashEntry, pUrlEntry)) { - /* FIXME: implement timeout object unlocking */ TRACE("Trying to delete locked entry\n"); pUrlEntry->CacheEntryType |= PENDING_DELETE_CACHE_ENTRY; SetLastError(ERROR_SHARING_VIOLATION); @@ -2736,11 +2767,11 @@ static BOOL CommitUrlCacheEntryInternal( if (URLCache_FindHash(pHeader, lpszUrlNameA, &pHashEntry)) { - if ((pHashEntry->dwHashKey & ((1<<HASHTABLE_FLAG_BITS)-1)) == HASHTABLE_LOCK) + URL_CACHEFILE_ENTRY *pUrlEntry = (URL_CACHEFILE_ENTRY*)((LPBYTE)pHeader + pHashEntry->dwOffsetEntry); + if (URLCache_IsLocked(pHashEntry, pUrlEntry)) { - /* FIXME: implement timeout object unlocking */ - FIXME("Trying to overwrite locked entry\n"); - SetLastError(ERROR_SHARING_VIOLATION); + TRACE("Trying to overwrite locked entry\n"); + error = ERROR_SHARING_VIOLATION; goto cleanup; }

1 0

Piotr Caban : wininet: Delete file when cache entry is deleted.
by Alexandre Julliard 18 Sep '12

18 Sep '12

Module: wine Branch: master Commit: 359ed3382442db00c2bf55dc958629d812e3e6be URL: http://source.winehq.org/git/wine.git/?a=commit;h=359ed3382442db00c2bf55dc9… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Tue Sep 18 14:43:23 2012 +0200 wininet: Delete file when cache entry is deleted. --- dlls/wininet/tests/urlcache.c | 5 ----- dlls/wininet/urlcache.c | 19 +++++++++++++------ 2 files changed, 13 insertions(+), 11 deletions(-) diff --git a/dlls/wininet/tests/urlcache.c b/dlls/wininet/tests/urlcache.c index 12827e9..4238c60 100644 --- a/dlls/wininet/tests/urlcache.c +++ b/dlls/wininet/tests/urlcache.c @@ -481,7 +481,6 @@ static void test_urlcacheA(void) SetLastError(0xdeadbeef); ret = DeleteFile(filenameA); - todo_wine ok(!ret && GetLastError() == ERROR_FILE_NOT_FOUND, "local file should no longer exist\n"); /* Creating two entries with the same URL */ @@ -537,7 +536,6 @@ static void test_urlcacheA(void) { ret = pDeleteUrlCacheEntryA(TEST_URL); ok(ret, "DeleteUrlCacheEntryA failed with error %d\n", GetLastError()); - todo_wine check_file_not_exists(filenameA); todo_wine check_file_not_exists(filenameA1); @@ -595,7 +593,6 @@ static void test_urlcacheA(void) /* By unlocking the already-deleted cache entry, the file associated * with it is deleted.. */ - todo_wine check_file_not_exists(filenameA); /* (just in case, delete file) */ DeleteFileA(filenameA); @@ -692,7 +689,6 @@ static void test_urlcacheA(void) ret = pDeleteUrlCacheEntryA(TEST_URL); ok(ret, "DeleteUrlCacheEntryA failed with error %d\n", GetLastError()); /* When explicitly deleting the cache entry, the file is also deleted */ - todo_wine check_file_not_exists(filenameA); } /* Test once again, setting the exempt delta via SetUrlCacheEntryInfo */ @@ -760,7 +756,6 @@ static void test_urlcacheA(void) { ret = pDeleteUrlCacheEntryA(TEST_URL); ok(ret, "DeleteUrlCacheEntryA failed with error %d\n", GetLastError()); - todo_wine check_file_not_exists(filenameA); } } diff --git a/dlls/wininet/urlcache.c b/dlls/wininet/urlcache.c index 0536093..1b62fda 100644 --- a/dlls/wininet/urlcache.c +++ b/dlls/wininet/urlcache.c @@ -2175,11 +2175,13 @@ BOOL WINAPI RetrieveUrlCacheEntryFileW( return TRUE; } -static BOOL DeleteUrlCacheEntryInternal(LPURLCACHE_HEADER pHeader, - struct _HASH_ENTRY *pHashEntry) +static BOOL DeleteUrlCacheEntryInternal(const URLCACHECONTAINER * pContainer, + LPURLCACHE_HEADER pHeader, struct _HASH_ENTRY *pHashEntry) { CACHEFILE_ENTRY * pEntry; URL_CACHEFILE_ENTRY * pUrlEntry; + WCHAR path[MAX_PATH]; + LONG path_size = sizeof(path); pEntry = (CACHEFILE_ENTRY *)((LPBYTE)pHeader + pHashEntry->dwOffsetEntry); if (pEntry->dwSignature != URL_SIGNATURE) @@ -2220,8 +2222,13 @@ static BOOL DeleteUrlCacheEntryInternal(LPURLCACHE_HEADER pHeader, pHeader->CacheUsage.QuadPart = 0; } - URLCache_DeleteEntry(pHeader, pEntry); + if (pUrlEntry->dwOffsetLocalName && URLCache_LocalFileNameToPathW(pContainer, pHeader, + (LPCSTR)pUrlEntry+pUrlEntry->dwOffsetLocalName, pUrlEntry->CacheDir, path, &path_size)) + { + DeleteFileW(path); + } + URLCache_DeleteEntry(pHeader, pEntry); URLCache_DeleteEntryFromHash(pHashEntry); return TRUE; } @@ -2297,7 +2304,7 @@ BOOL WINAPI UnlockUrlCacheEntryFileA( { URLCache_HashEntrySetFlags(pHashEntry, HASHTABLE_URL); if (pUrlEntry->CacheEntryType & PENDING_DELETE_CACHE_ENTRY) - DeleteUrlCacheEntryInternal(pHeader, pHashEntry); + DeleteUrlCacheEntryInternal(pContainer, pHeader, pHashEntry); } URLCacheContainer_UnlockIndex(pContainer, pHeader); @@ -3241,7 +3248,7 @@ BOOL WINAPI DeleteUrlCacheEntryA(LPCSTR lpszUrlName) return FALSE; } - ret = DeleteUrlCacheEntryInternal(pHeader, pHashEntry); + ret = DeleteUrlCacheEntryInternal(pContainer, pHeader, pHashEntry); URLCacheContainer_UnlockIndex(pContainer, pHeader); @@ -3301,7 +3308,7 @@ BOOL WINAPI DeleteUrlCacheEntryW(LPCWSTR lpszUrlName) return FALSE; } - ret = DeleteUrlCacheEntryInternal(pHeader, pHashEntry); + ret = DeleteUrlCacheEntryInternal(pContainer, pHeader, pHashEntry); URLCacheContainer_UnlockIndex(pContainer, pHeader); heap_free(urlA);

1 0

Piotr Caban : wininet: Don't define DELETED_CACHE_ENTRY in public headers.
by Alexandre Julliard 18 Sep '12

18 Sep '12

Module: wine Branch: master Commit: 7967f8beecf6a44d768365aa7921ba9ea4ae1a14 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7967f8beecf6a44d768365aa7… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Tue Sep 18 14:43:12 2012 +0200 wininet: Don't define DELETED_CACHE_ENTRY in public headers. --- dlls/wininet/tests/urlcache.c | 4 ++-- dlls/wininet/urlcache.c | 6 ++++-- include/wininet.h | 1 - 3 files changed, 6 insertions(+), 5 deletions(-) diff --git a/dlls/wininet/tests/urlcache.c b/dlls/wininet/tests/urlcache.c index 02d8d28..12827e9 100644 --- a/dlls/wininet/tests/urlcache.c +++ b/dlls/wininet/tests/urlcache.c @@ -582,8 +582,8 @@ static void test_urlcacheA(void) memset(lpCacheEntryInfo, 0, cbCacheEntryInfo); ret = GetUrlCacheEntryInfo(TEST_URL, lpCacheEntryInfo, &cbCacheEntryInfo); ok(ret, "GetUrlCacheEntryInfo failed with error %d\n", GetLastError()); - ok(lpCacheEntryInfo->CacheEntryType & DELETED_CACHE_ENTRY, - "CacheEntryType hasn't DELETED_CACHE_ENTRY set, (flags %08x)\n", + ok(lpCacheEntryInfo->CacheEntryType & 0x400000, + "CacheEntryType hasn't PENDING_DELETE_CACHE_ENTRY set, (flags %08x)\n", lpCacheEntryInfo->CacheEntryType); HeapFree(GetProcessHeap(), 0, lpCacheEntryInfo); diff --git a/dlls/wininet/urlcache.c b/dlls/wininet/urlcache.c index e958d00..0536093 100644 --- a/dlls/wininet/urlcache.c +++ b/dlls/wininet/urlcache.c @@ -79,6 +79,8 @@ WINE_DEFAULT_DEBUG_CHANNEL(wininet); #define HASHTABLE_REDR 5 #define HASHTABLE_FLAG_BITS 5 +#define PENDING_DELETE_CACHE_ENTRY 0x00400000 + #define DWORD_SIG(a,b,c,d) (a | (b << 8) | (c << 16) | (d << 24)) #define URL_SIGNATURE DWORD_SIG('U','R','L',' ') #define REDR_SIGNATURE DWORD_SIG('R','E','D','R') @@ -2193,7 +2195,7 @@ static BOOL DeleteUrlCacheEntryInternal(LPURLCACHE_HEADER pHeader, { /* FIXME: implement timeout object unlocking */ TRACE("Trying to delete locked entry\n"); - pUrlEntry->CacheEntryType |= DELETED_CACHE_ENTRY; + pUrlEntry->CacheEntryType |= PENDING_DELETE_CACHE_ENTRY; SetLastError(ERROR_SHARING_VIOLATION); return FALSE; } @@ -2294,7 +2296,7 @@ BOOL WINAPI UnlockUrlCacheEntryFileA( if (!pUrlEntry->dwUseCount) { URLCache_HashEntrySetFlags(pHashEntry, HASHTABLE_URL); - if (pUrlEntry->CacheEntryType & DELETED_CACHE_ENTRY) + if (pUrlEntry->CacheEntryType & PENDING_DELETE_CACHE_ENTRY) DeleteUrlCacheEntryInternal(pHeader, pHashEntry); } diff --git a/include/wininet.h b/include/wininet.h index c7a428a..51d9864 100644 --- a/include/wininet.h +++ b/include/wininet.h @@ -1497,7 +1497,6 @@ INTERNETAPI DWORD WINAPI PrivacyGetZonePreferenceW(DWORD,DWORD,LPDWORD,LPWSTR,LP #define EDITED_CACHE_ENTRY 0x00000008 #define COOKIE_CACHE_ENTRY 0x00100000 #define URLHISTORY_CACHE_ENTRY 0x00200000 -#define DELETED_CACHE_ENTRY 0x00400000 #define TRACK_OFFLINE_CACHE_ENTRY 0x00000010 #define TRACK_ONLINE_CACHE_ENTRY 0x00000020 #define SPARSE_CACHE_ENTRY 0x00010000

1 0

Hans Leidekker : wbemprox: Implement Win32_OperatingSystem.LastBootUptime.
by Alexandre Julliard 18 Sep '12

18 Sep '12

Module: wine Branch: master Commit: 22454d6341b1937a09dacb4ae8e6ded84f5cc9dc URL: http://source.winehq.org/git/wine.git/?a=commit;h=22454d6341b1937a09dacb4ae… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Tue Sep 18 13:12:20 2012 +0200 wbemprox: Implement Win32_OperatingSystem.LastBootUptime. --- dlls/wbemprox/builtin.c | 27 +++++++++++++++++++++++++++ 1 files changed, 27 insertions(+), 0 deletions(-) diff --git a/dlls/wbemprox/builtin.c b/dlls/wbemprox/builtin.c index 4b6ff38..54de2bb 100644 --- a/dlls/wbemprox/builtin.c +++ b/dlls/wbemprox/builtin.c @@ -111,6 +111,8 @@ static const WCHAR prop_handleW[] = {'H','a','n','d','l','e',0}; static const WCHAR prop_interfaceindexW[] = {'I','n','t','e','r','f','a','c','e','I','n','d','e','x',0}; +static const WCHAR prop_lastbootuptimeW[] = + {'L','a','s','t','B','o','o','t','U','p','T','i','m','e',0}; static const WCHAR prop_macaddressW[] = {'M','A','C','A','d','d','r','e','s','s',0}; static const WCHAR prop_manufacturerW[] = @@ -231,6 +233,7 @@ static const struct column col_os[] = { { prop_captionW, CIM_STRING }, { prop_csdversionW, CIM_STRING }, + { prop_lastbootuptimeW, CIM_DATETIME|COL_FLAG_DYNAMIC }, { prop_osarchitectureW, CIM_STRING }, { prop_oslanguageW, CIM_UINT32, VT_I4 }, { prop_systemdirectoryW, CIM_STRING|COL_FLAG_DYNAMIC } @@ -373,6 +376,7 @@ struct record_operatingsystem { const WCHAR *caption; const WCHAR *csdversion; + const WCHAR *lastbootuptime; const WCHAR *osarchitecture; UINT32 oslanguage; const WCHAR *systemdirectory; @@ -831,6 +835,28 @@ static void fill_processor( struct table *table ) table->num_rows = count; } +static WCHAR *get_lastbootuptime(void) +{ + static const WCHAR fmtW[] = + {'%','0','4','u','%','0','2','u','%','0','2','u','%','0','2','u','%','0','2','u','%','0','2','u', + '.','%','0','6','u','+','0','0','0',0}; + SYSTEMTIME st; + FILETIME ft; + ULARGE_INTEGER ticks; + WCHAR *ret; + + if (!(ret = heap_alloc( 26 * sizeof(WCHAR) ))) return NULL; + GetSystemTime( &st ); + SystemTimeToFileTime( &st, &ft ); + ticks.u.LowPart = ft.dwLowDateTime; + ticks.u.HighPart = ft.dwHighDateTime; + ticks.QuadPart -= GetTickCount64() * 10000; + ft.dwLowDateTime = ticks.u.LowPart; + ft.dwHighDateTime = ticks.u.HighPart; + FileTimeToSystemTime( &ft, &st ); + sprintfW( ret, fmtW, st.wYear, st.wMonth, st.wDay, st.wHour, st.wMinute, st.wSecond, st.wMilliseconds * 1000 ); + return ret; +} static const WCHAR *get_osarchitecture(void) { SYSTEM_INFO info; @@ -859,6 +885,7 @@ static void fill_os( struct table *table ) rec = (struct record_operatingsystem *)table->data; rec->caption = os_captionW; rec->csdversion = os_csdversionW; + rec->lastbootuptime = get_lastbootuptime(); rec->osarchitecture = get_osarchitecture(); rec->oslanguage = MAKELANGID( LANG_ENGLISH, SUBLANG_ENGLISH_US ); rec->systemdirectory = get_systemdirectory();

1 0

Hans Leidekker : wbemprox: Add helpers to retrieve the system directory and OS architecture.
by Alexandre Julliard 18 Sep '12

18 Sep '12

Module: wine Branch: master Commit: 381d30ac17a8cf68099e8fa3e2d0bb7669327d64 URL: http://source.winehq.org/git/wine.git/?a=commit;h=381d30ac17a8cf68099e8fa3e… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Tue Sep 18 13:11:59 2012 +0200 wbemprox: Add helpers to retrieve the system directory and OS architecture. --- dlls/wbemprox/builtin.c | 42 ++++++++++++++++++++++++------------------ 1 files changed, 24 insertions(+), 18 deletions(-) diff --git a/dlls/wbemprox/builtin.c b/dlls/wbemprox/builtin.c index 54992b3..4b6ff38 100644 --- a/dlls/wbemprox/builtin.c +++ b/dlls/wbemprox/builtin.c @@ -831,31 +831,37 @@ static void fill_processor( struct table *table ) table->num_rows = count; } -static void fill_os( struct table *table ) +static const WCHAR *get_osarchitecture(void) { - struct record_operatingsystem *rec; - WCHAR path[MAX_PATH]; SYSTEM_INFO info; + GetNativeSystemInfo( &info ); + if (info.u.s.wProcessorArchitecture == PROCESSOR_ARCHITECTURE_AMD64) return os_64bitW; + return os_32bitW; +} +static WCHAR *get_systemdirectory(void) +{ void *redir; + WCHAR *ret; - if (!(table->data = heap_alloc( sizeof(*rec) ))) return; - - rec = (struct record_operatingsystem *)table->data; - rec->caption = os_captionW; - rec->csdversion = os_csdversionW; + if (!(ret = heap_alloc( MAX_PATH * sizeof(WCHAR) ))) return NULL; + Wow64DisableWow64FsRedirection( &redir ); + GetSystemDirectoryW( ret, MAX_PATH ); + Wow64RevertWow64FsRedirection( redir ); + return ret; +} - GetNativeSystemInfo( &info ); - if (info.u.s.wProcessorArchitecture == PROCESSOR_ARCHITECTURE_AMD64) - rec->osarchitecture = os_64bitW; - else - rec->osarchitecture = os_32bitW; +static void fill_os( struct table *table ) +{ + struct record_operatingsystem *rec; - rec->oslanguage = MAKELANGID( LANG_ENGLISH, SUBLANG_ENGLISH_US ); + if (!(table->data = heap_alloc( sizeof(*rec) ))) return; - Wow64DisableWow64FsRedirection( &redir ); - GetSystemDirectoryW( path, MAX_PATH ); - Wow64RevertWow64FsRedirection( redir ); - rec->systemdirectory = heap_strdupW( path ); + rec = (struct record_operatingsystem *)table->data; + rec->caption = os_captionW; + rec->csdversion = os_csdversionW; + rec->osarchitecture = get_osarchitecture(); + rec->oslanguage = MAKELANGID( LANG_ENGLISH, SUBLANG_ENGLISH_US ); + rec->systemdirectory = get_systemdirectory(); TRACE("created 1 row\n"); table->num_rows = 1;

1 0

Hans Leidekker : wbemprox: Fix a memory leak.
by Alexandre Julliard 18 Sep '12

18 Sep '12

Module: wine Branch: master Commit: d8d6821cd49b29736d7a2886df585ee939ccfb7c URL: http://source.winehq.org/git/wine.git/?a=commit;h=d8d6821cd49b29736d7a2886d… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Tue Sep 18 13:11:42 2012 +0200 wbemprox: Fix a memory leak. --- dlls/wbemprox/builtin.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/wbemprox/builtin.c b/dlls/wbemprox/builtin.c index 5434fc5..54992b3 100644 --- a/dlls/wbemprox/builtin.c +++ b/dlls/wbemprox/builtin.c @@ -233,7 +233,7 @@ static const struct column col_os[] = { prop_csdversionW, CIM_STRING }, { prop_osarchitectureW, CIM_STRING }, { prop_oslanguageW, CIM_UINT32, VT_I4 }, - { prop_systemdirectoryW, CIM_STRING } + { prop_systemdirectoryW, CIM_STRING|COL_FLAG_DYNAMIC } }; static const struct column col_params[] = {

1 0

Charles Davis : oleaut32/tests: Don't use a C99 keyword as a method name.
by Alexandre Julliard 18 Sep '12

18 Sep '12

Module: wine Branch: master Commit: 8e14c9413b33dc47673d48d7ffcf7e7ae5c51dbf URL: http://source.winehq.org/git/wine.git/?a=commit;h=8e14c9413b33dc47673d48d7f… Author: Charles Davis <cdavis5x(a)gmail.com> Date: Tue Sep 18 05:02:04 2012 -0600 oleaut32/tests: Don't use a C99 keyword as a method name. --- dlls/oleaut32/tests/tmarshal.c | 4 ++-- dlls/oleaut32/tests/tmarshal.idl | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/oleaut32/tests/tmarshal.c b/dlls/oleaut32/tests/tmarshal.c index a4a2b6f..2c1b8e8 100644 --- a/dlls/oleaut32/tests/tmarshal.c +++ b/dlls/oleaut32/tests/tmarshal.c @@ -706,7 +706,7 @@ static HRESULT WINAPI Widget_put_prop_req_arg( return S_OK; } -static HRESULT WINAPI Widget_restrict(IWidget* iface, INT *i) +static HRESULT WINAPI Widget__restrict(IWidget* iface, INT *i) { trace("restrict\n"); *i = DISPID_TM_RESTRICTED; @@ -756,7 +756,7 @@ static const struct IWidgetVtbl Widget_VTable = Widget_ByRefUInt, Widget_put_prop_opt_arg, Widget_put_prop_req_arg, - Widget_restrict, + Widget__restrict, Widget_neg_restrict }; diff --git a/dlls/oleaut32/tests/tmarshal.idl b/dlls/oleaut32/tests/tmarshal.idl index c589574..8dca84c 100644 --- a/dlls/oleaut32/tests/tmarshal.idl +++ b/dlls/oleaut32/tests/tmarshal.idl @@ -161,7 +161,7 @@ library TestTypelib HRESULT prop_req_arg([in] INT req, [in] INT i); [id(DISPID_TM_RESTRICTED), restricted] - HRESULT restrict([out, retval] INT *i); + HRESULT _restrict([out, retval] INT *i); [id(DISPID_TM_NEG_RESTRICTED), restricted] HRESULT neg_restrict([out, retval] INT *i);

1 0

Charles Davis : oleaut32/tests: Don't take the size of a pointer (Clang).
by Alexandre Julliard 18 Sep '12

18 Sep '12

Module: wine Branch: master Commit: 8cba35d5d2bd2d027ef1dd935538e4b5031caac7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8cba35d5d2bd2d027ef1dd935… Author: Charles Davis <cdavis5x(a)gmail.com> Date: Tue Sep 18 05:02:03 2012 -0600 oleaut32/tests: Don't take the size of a pointer (Clang). --- dlls/oleaut32/tests/tmarshal.c | 12 ++++++------ 1 files changed, 6 insertions(+), 6 deletions(-) diff --git a/dlls/oleaut32/tests/tmarshal.c b/dlls/oleaut32/tests/tmarshal.c index be8ef94..a4a2b6f 100644 --- a/dlls/oleaut32/tests/tmarshal.c +++ b/dlls/oleaut32/tests/tmarshal.c @@ -596,11 +596,11 @@ static HRESULT WINAPI Widget_VarArg( } -static BOOL mystruct_uint_ordered(UINT uarr[8]) +static BOOL mystruct_uint_ordered(MYSTRUCT *mystruct) { int i; - for (i = 0; i < sizeof(uarr) / sizeof(uarr[0]); i++) - if (uarr[i] != i) + for (i = 0; i < sizeof(mystruct->uarr)/sizeof(mystruct->uarr[0]); i++) + if (mystruct->uarr[i] != i) return 0; return 1; @@ -615,16 +615,16 @@ static HRESULT WINAPI Widget_StructArgs( int i, diff = 0; ok(byval.field1 == MYSTRUCT_BYVAL.field1 && byval.field2 == MYSTRUCT_BYVAL.field2 && - mystruct_uint_ordered(byval.uarr), + mystruct_uint_ordered(&byval), "Struct parameter passed by value corrupted\n"); ok(byptr->field1 == MYSTRUCT_BYPTR.field1 && byptr->field2 == MYSTRUCT_BYPTR.field2 && - mystruct_uint_ordered(byptr->uarr), + mystruct_uint_ordered(byptr), "Struct parameter passed by pointer corrupted\n"); for (i = 0; i < 5; i++) if (arr[i].field1 != MYSTRUCT_ARRAY[i].field1 || arr[i].field2 != MYSTRUCT_ARRAY[i].field2 || - ! mystruct_uint_ordered(arr[i].uarr)) + ! mystruct_uint_ordered(&arr[i])) diff++; ok(diff == 0, "Array of structs corrupted\n"); return S_OK;

1 0

Jacek Caban : vbscript: Return error for unsupported flags in ScriptDisp:: InvokeEx.
by Alexandre Julliard 18 Sep '12

18 Sep '12

Module: wine Branch: master Commit: b6515030c88ca67c4b73ad0d94a79fe9eb0637eb URL: http://source.winehq.org/git/wine.git/?a=commit;h=b6515030c88ca67c4b73ad0d9… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Sep 18 12:26:30 2012 +0200 vbscript: Return error for unsupported flags in ScriptDisp::InvokeEx. --- dlls/vbscript/vbdisp.c | 15 +++++++++++---- 1 files changed, 11 insertions(+), 4 deletions(-) diff --git a/dlls/vbscript/vbdisp.c b/dlls/vbscript/vbdisp.c index a003bf1..4721753 100644 --- a/dlls/vbscript/vbdisp.c +++ b/dlls/vbscript/vbdisp.c @@ -783,10 +783,17 @@ static HRESULT WINAPI ScriptDisp_InvokeEx(IDispatchEx *iface, DISPID id, LCID lc return invoke_variant_prop(&ident->u.var->v, wFlags, pdp, pvarRes); } - - IActiveScriptSite_OnEnterScript(This->ctx->site); - hres = exec_script(This->ctx, ident->u.func, NULL, pdp, pvarRes); - IActiveScriptSite_OnLeaveScript(This->ctx->site); + switch(wFlags) { + case DISPATCH_METHOD: + case DISPATCH_METHOD|DISPATCH_PROPERTYGET: + IActiveScriptSite_OnEnterScript(This->ctx->site); + hres = exec_script(This->ctx, ident->u.func, NULL, pdp, pvarRes); + IActiveScriptSite_OnLeaveScript(This->ctx->site); + break; + default: + FIXME("Unsupported flags %x\n", wFlags); + hres = E_NOTIMPL; + } return hres; }

1 0

Jacek Caban : vbscript: Added ScriptDisp_Invoke implementation.
by Alexandre Julliard 18 Sep '12

18 Sep '12

Module: wine Branch: master Commit: 91b41eb4942758de835571aa307b79a1cf488444 URL: http://source.winehq.org/git/wine.git/?a=commit;h=91b41eb4942758de835571aa3… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Sep 18 12:26:16 2012 +0200 vbscript: Added ScriptDisp_Invoke implementation. --- dlls/vbscript/tests/vbscript.c | 14 ++++++++++++++ dlls/vbscript/vbdisp.c | 12 +++++++----- 2 files changed, 21 insertions(+), 5 deletions(-) diff --git a/dlls/vbscript/tests/vbscript.c b/dlls/vbscript/tests/vbscript.c index 3dd64dd..7cd6e2d 100644 --- a/dlls/vbscript/tests/vbscript.c +++ b/dlls/vbscript/tests/vbscript.c @@ -468,6 +468,20 @@ static void test_scriptdisp(void) CHECK_CALLED(OnEnterScript); CHECK_CALLED(OnLeaveScript); + SET_EXPECT(OnEnterScript); + SET_EXPECT(OnLeaveScript); + + memset(&dp, 0, sizeof(dp)); + memset(&ei, 0, sizeof(ei)); + V_VT(&v) = VT_EMPTY; + hres = IDispatchEx_Invoke(script_disp, id, &IID_NULL, 0, DISPATCH_PROPERTYGET|DISPATCH_METHOD, &dp, &v, &ei, NULL); + ok(hres == S_OK, "InvokeEx failed: %08x\n", hres); + ok(V_VT(&v) == VT_I2, "V_VT(v) = %d\n", V_VT(&v)); + ok(V_I2(&v) == 5, "V_I2(v) = %d\n", V_I2(&v)); + + CHECK_CALLED(OnEnterScript); + CHECK_CALLED(OnLeaveScript); + get_disp_id(script_disp, "globalSub", DISP_E_UNKNOWNNAME, &id); parse_script(parser, "sub globalSub()\nend sub"); get_disp_id(script_disp, "globalSub", S_OK, &id); diff --git a/dlls/vbscript/vbdisp.c b/dlls/vbscript/vbdisp.c index 8b0e4f2..a003bf1 100644 --- a/dlls/vbscript/vbdisp.c +++ b/dlls/vbscript/vbdisp.c @@ -700,14 +700,16 @@ static HRESULT WINAPI ScriptDisp_GetIDsOfNames(IDispatchEx *iface, REFIID riid, return S_OK; } -static HRESULT WINAPI ScriptDisp_Invoke(IDispatchEx *iface, DISPID dispIdMember, - REFIID riid, LCID lcid, WORD wFlags, DISPPARAMS *pDispParams, - VARIANT *pVarResult, EXCEPINFO *pExcepInfo, UINT *puArgErr) +static HRESULT WINAPI ScriptDisp_Invoke(IDispatchEx *iface, DISPID dispIdMember, REFIID riid, LCID lcid, + WORD wFlags, DISPPARAMS *pDispParams, VARIANT *pVarResult, EXCEPINFO *pExcepInfo, UINT *puArgErr) { ScriptDisp *This = ScriptDisp_from_IDispatchEx(iface); - FIXME("(%p)->(%d %s %d %d %p %p %p %p)\n", This, dispIdMember, debugstr_guid(riid), + + TRACE("(%p)->(%d %s %d %d %p %p %p %p)\n", This, dispIdMember, debugstr_guid(riid), lcid, wFlags, pDispParams, pVarResult, pExcepInfo, puArgErr); - return E_NOTIMPL; + + return IDispatchEx_InvokeEx(&This->IDispatchEx_iface, dispIdMember, lcid, wFlags, + pDispParams, pVarResult, pExcepInfo, NULL); } static HRESULT WINAPI ScriptDisp_GetDispID(IDispatchEx *iface, BSTR bstrName, DWORD grfdex, DISPID *pid)

1 0

← Newer
1
...
35
36
37
38
39
40
41
...
83
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 September 2012