wine-commits July 2016

wine-commits@winehq.org

2 participants
680 discussions

Michael Stefaniuc : msctf: Avoid a cast from IEnumTfLanguageProfiles to the COM object.
by Alexandre Julliard 20 Jul '16

20 Jul '16

Module: wine Branch: master Commit: a944b4eed97d7f69d664aa1f960c5c620908a311 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a944b4eed97d7f69d664aa1f9… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Tue Jul 19 09:47:24 2016 +0200 msctf: Avoid a cast from IEnumTfLanguageProfiles to the COM object. Signed-off-by: Michael Stefaniuc <mstefani(a)redhat.de> Signed-off-by: Aric Stewart <aric(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msctf/inputprocessor.c | 20 +++++++++++++------- 1 file changed, 13 insertions(+), 7 deletions(-) diff --git a/dlls/msctf/inputprocessor.c b/dlls/msctf/inputprocessor.c index fca78e4..fdeba38 100644 --- a/dlls/msctf/inputprocessor.c +++ b/dlls/msctf/inputprocessor.c @@ -93,7 +93,7 @@ typedef struct { } EnumTfInputProcessorProfiles; static HRESULT ProfilesEnumGuid_Constructor(IEnumGUID **ppOut); -static HRESULT EnumTfLanguageProfiles_Constructor(LANGID langid, IEnumTfLanguageProfiles **ppOut); +static HRESULT EnumTfLanguageProfiles_Constructor(LANGID langid, EnumTfLanguageProfiles **out); static inline EnumTfInputProcessorProfiles *impl_from_IEnumTfInputProcessorProfiles(IEnumTfInputProcessorProfiles *iface) { @@ -616,12 +616,17 @@ static HRESULT WINAPI InputProcessorProfiles_EnumLanguageProfiles( IEnumTfLanguageProfiles **ppEnum) { InputProcessorProfiles *This = impl_from_ITfInputProcessorProfiles(iface); + EnumTfLanguageProfiles *profenum; + HRESULT hr; + TRACE("(%p) %x %p\n",This,langid,ppEnum); if (!ppEnum) return E_INVALIDARG; + hr = EnumTfLanguageProfiles_Constructor(langid, &profenum); + *ppEnum = &profenum->IEnumTfLanguageProfiles_iface; - return EnumTfLanguageProfiles_Constructor(langid, ppEnum); + return hr; } static HRESULT WINAPI InputProcessorProfiles_EnableLanguageProfile( @@ -1292,16 +1297,16 @@ static HRESULT WINAPI EnumTfLanguageProfiles_Clone( IEnumTfLanguageProfiles *ifa IEnumTfLanguageProfiles **ppenum) { EnumTfLanguageProfiles *This = impl_from_IEnumTfLanguageProfiles(iface); + EnumTfLanguageProfiles *new_This; HRESULT res; TRACE("(%p)\n",This); if (ppenum == NULL) return E_POINTER; - res = EnumTfLanguageProfiles_Constructor(This->langid, ppenum); + res = EnumTfLanguageProfiles_Constructor(This->langid, &new_This); if (SUCCEEDED(res)) { - EnumTfLanguageProfiles *new_This = (EnumTfLanguageProfiles *)*ppenum; new_This->tip_index = This->tip_index; lstrcpynW(new_This->szwCurrentClsid,This->szwCurrentClsid,39); @@ -1314,6 +1319,7 @@ static HRESULT WINAPI EnumTfLanguageProfiles_Clone( IEnumTfLanguageProfiles *ifa res = RegOpenKeyExW(new_This->tipkey, fullkey, 0, KEY_READ | KEY_WRITE, &This->langkey); new_This->lang_index = This->lang_index; } + *ppenum = &new_This->IEnumTfLanguageProfiles_iface; } return res; } @@ -1329,7 +1335,7 @@ static const IEnumTfLanguageProfilesVtbl EnumTfLanguageProfilesVtbl = EnumTfLanguageProfiles_Skip }; -static HRESULT EnumTfLanguageProfiles_Constructor(LANGID langid, IEnumTfLanguageProfiles **ppOut) +static HRESULT EnumTfLanguageProfiles_Constructor(LANGID langid, EnumTfLanguageProfiles **out) { HRESULT hr; EnumTfLanguageProfiles *This; @@ -1356,7 +1362,7 @@ static HRESULT EnumTfLanguageProfiles_Constructor(LANGID langid, IEnumTfLanguage return E_FAIL; } - *ppOut = &This->IEnumTfLanguageProfiles_iface; - TRACE("returning %p\n", *ppOut); + *out = This; + TRACE("returning %p\n", *out); return S_OK; }

1 0

Michael Stefaniuc : msctf: Don' t crash on NULL input in EnumLanguageProfiles().
by Alexandre Julliard 20 Jul '16

20 Jul '16

Module: wine Branch: master Commit: fd0101ab61e61c67bc91d35639bad125265da615 URL: http://source.winehq.org/git/wine.git/?a=commit;h=fd0101ab61e61c67bc91d3563… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Tue Jul 19 09:46:01 2016 +0200 msctf: Don't crash on NULL input in EnumLanguageProfiles(). Signed-off-by: Michael Stefaniuc <mstefani(a)redhat.de> Signed-off-by: Aric Stewart <aric(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msctf/inputprocessor.c | 4 ++++ dlls/msctf/tests/inputprocessor.c | 5 +++++ 2 files changed, 9 insertions(+) diff --git a/dlls/msctf/inputprocessor.c b/dlls/msctf/inputprocessor.c index 0e8ad22..fca78e4 100644 --- a/dlls/msctf/inputprocessor.c +++ b/dlls/msctf/inputprocessor.c @@ -617,6 +617,10 @@ static HRESULT WINAPI InputProcessorProfiles_EnumLanguageProfiles( { InputProcessorProfiles *This = impl_from_ITfInputProcessorProfiles(iface); TRACE("(%p) %x %p\n",This,langid,ppEnum); + + if (!ppEnum) + return E_INVALIDARG; + return EnumTfLanguageProfiles_Constructor(langid, ppEnum); } diff --git a/dlls/msctf/tests/inputprocessor.c b/dlls/msctf/tests/inputprocessor.c index 831e37a..2096bff 100644 --- a/dlls/msctf/tests/inputprocessor.c +++ b/dlls/msctf/tests/inputprocessor.c @@ -973,6 +973,11 @@ static void test_EnumLanguageProfiles(void) { BOOL found = FALSE; IEnumTfLanguageProfiles *ppEnum; + HRESULT hr; + + hr = ITfInputProcessorProfiles_EnumLanguageProfiles(g_ipp, gLangid, NULL); + ok(hr == E_INVALIDARG, "EnumLanguageProfiles failed: %x\n", hr); + if (SUCCEEDED(ITfInputProcessorProfiles_EnumLanguageProfiles(g_ipp,gLangid,&ppEnum))) { TF_LANGUAGEPROFILE profile;

1 0

Alex Henrie : shell32: Set output interfaces to NULL in FolderItems functions.
by Alexandre Julliard 19 Jul '16

19 Jul '16

Module: wine Branch: master Commit: a71128513767d44a8f42e10895578b650061bdce URL: http://source.winehq.org/git/wine.git/?a=commit;h=a71128513767d44a8f42e1089… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Wed Jul 6 23:35:45 2016 -0600 shell32: Set output interfaces to NULL in FolderItems functions. Signed-off-by: Alex Henrie <alexhenrie24(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/shell32/shelldispatch.c | 9 ++++++++- dlls/shell32/tests/shelldispatch.c | 3 --- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/dlls/shell32/shelldispatch.c b/dlls/shell32/shelldispatch.c index 0c1662d..ac79302 100644 --- a/dlls/shell32/shelldispatch.c +++ b/dlls/shell32/shelldispatch.c @@ -1063,12 +1063,16 @@ static HRESULT WINAPI FolderItemsImpl_get_Application(FolderItems3 *iface, IDisp if (!ppid) return E_INVALIDARG; + *ppid = NULL; return E_NOTIMPL; } static HRESULT WINAPI FolderItemsImpl_get_Parent(FolderItems3 *iface, IDispatch **ppid) { - FIXME("(%p,%p)\n", iface, ppid); + TRACE("(%p,%p)\n", iface, ppid); + + if (ppid) + *ppid = NULL; return E_NOTIMPL; } @@ -1077,6 +1081,7 @@ static HRESULT WINAPI FolderItemsImpl_Item(FolderItems3 *iface, VARIANT index, F { FIXME("(%p,%s,%p)\n", iface, debugstr_variant(&index), ppid); + *ppid = NULL; return E_NOTIMPL; } @@ -1087,6 +1092,7 @@ static HRESULT WINAPI FolderItemsImpl__NewEnum(FolderItems3 *iface, IUnknown **p if (!ppunk) return E_INVALIDARG; + *ppunk = NULL; return E_NOTIMPL; } @@ -1111,6 +1117,7 @@ static HRESULT WINAPI FolderItemsImpl_get_Verbs(FolderItems3 *iface, FolderItemV if (!ppfic) return E_INVALIDARG; + *ppfic = NULL; return E_NOTIMPL; } diff --git a/dlls/shell32/tests/shelldispatch.c b/dlls/shell32/tests/shelldispatch.c index c199549..1f50277 100644 --- a/dlls/shell32/tests/shelldispatch.c +++ b/dlls/shell32/tests/shelldispatch.c @@ -370,7 +370,6 @@ todo_wine r = FolderItems_Item(items, var, &item); todo_wine ok(r == S_FALSE, "expected S_FALSE, got %08x\n", r); -todo_wine ok(!item, "item is not null\n"); if (0) /* crashes on xp */ @@ -395,7 +394,6 @@ todo_wine disp = (IDispatch*)0xdeadbeef; r = FolderItems_get_Parent(items, &disp); ok(r == E_NOTIMPL, "expected E_NOTIMPL, got %08x\n", r); -todo_wine ok(!disp, "disp is not null\n"); if (0) /* crashes on xp */ @@ -426,7 +424,6 @@ todo_wine r = FolderItems3_get_Verbs(items3, &verbs); todo_wine ok(r == S_FALSE, "expected S_FALSE, got %08x\n", r); -todo_wine ok(!verbs, "verbs is not null\n"); }

1 0

Alex Henrie : shell32: Return E_INVALIDARG from FolderItems functions on null output.
by Alexandre Julliard 19 Jul '16

19 Jul '16

Module: wine Branch: master Commit: 970de7df997e29e9aff855ff10772724f17b93b9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=970de7df997e29e9aff855ff1… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Wed Jul 6 23:35:44 2016 -0600 shell32: Return E_INVALIDARG from FolderItems functions on null output. Signed-off-by: Alex Henrie <alexhenrie24(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/shell32/shelldispatch.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/dlls/shell32/shelldispatch.c b/dlls/shell32/shelldispatch.c index 93c851f..0c1662d 100644 --- a/dlls/shell32/shelldispatch.c +++ b/dlls/shell32/shelldispatch.c @@ -1060,6 +1060,9 @@ static HRESULT WINAPI FolderItemsImpl_get_Application(FolderItems3 *iface, IDisp { FIXME("(%p,%p)\n", iface, ppid); + if (!ppid) + return E_INVALIDARG; + return E_NOTIMPL; } @@ -1081,6 +1084,9 @@ static HRESULT WINAPI FolderItemsImpl__NewEnum(FolderItems3 *iface, IUnknown **p { FIXME("(%p,%p)\n", iface, ppunk); + if (!ppunk) + return E_INVALIDARG; + return E_NOTIMPL; } @@ -1102,6 +1108,9 @@ static HRESULT WINAPI FolderItemsImpl_get_Verbs(FolderItems3 *iface, FolderItemV { FIXME("(%p,%p)\n", iface, ppfic); + if (!ppfic) + return E_INVALIDARG; + return E_NOTIMPL; }

1 0

Alex Henrie : shell32/tests: Add tests for FolderItems.
by Alexandre Julliard 19 Jul '16

19 Jul '16

Module: wine Branch: master Commit: ddf76f6a6691eeb6f9b05af457534d4f2abf2b02 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ddf76f6a6691eeb6f9b05af45… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Wed Jul 6 23:35:43 2016 -0600 shell32/tests: Add tests for FolderItems. Signed-off-by: Alex Henrie <alexhenrie24(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/shell32/tests/shelldispatch.c | 134 ++++++++++++++++++++++++++++++++++++- 1 file changed, 133 insertions(+), 1 deletion(-) diff --git a/dlls/shell32/tests/shelldispatch.c b/dlls/shell32/tests/shelldispatch.c index dfcbe6d..c199549 100644 --- a/dlls/shell32/tests/shelldispatch.c +++ b/dlls/shell32/tests/shelldispatch.c @@ -31,6 +31,8 @@ #define EXPECT_HR(hr,hr_exp) \ ok(hr == hr_exp, "got 0x%08x, expected 0x%08x\n", hr, hr_exp) +static const WCHAR winetestW[] = {'w','i','n','e','t','e','s','t',0}; + static HRESULT (WINAPI *pSHGetFolderPathW)(HWND, int, HANDLE, DWORD, LPWSTR); static HRESULT (WINAPI *pSHGetNameFromIDList)(PCIDLIST_ABSOLUTE,SIGDN,PWSTR*); static HRESULT (WINAPI *pSHGetSpecialFolderLocation)(HWND, int, LPITEMIDLIST *); @@ -51,7 +53,6 @@ static void init_function_pointers(void) static void test_namespace(void) { - static const WCHAR winetestW[] = {'w','i','n','e','t','e','s','t',0}; static const WCHAR backslashW[] = {'\\',0}; static const WCHAR clsidW[] = { ':',':','{','6','4','5','F','F','0','4','0','-','5','0','8','1','-', @@ -307,6 +308,136 @@ static void test_namespace(void) IShellDispatch_Release(sd); } +static void test_items(void) +{ + WCHAR wstr[MAX_PATH], orig_dir[MAX_PATH]; + HRESULT r; + IShellDispatch *sd = NULL; + Folder *folder = NULL; + FolderItems *items = NULL; + FolderItems2 *items2 = NULL; + FolderItems3 *items3 = NULL; + FolderItem *item = (FolderItem*)0xdeadbeef; + IDispatch *disp = NULL; + IUnknown *unk = NULL; + FolderItemVerbs *verbs = (FolderItemVerbs*)0xdeadbeef; + VARIANT var; + LONG lcount = -1; + + r = CoCreateInstance(&CLSID_Shell, NULL, CLSCTX_INPROC_SERVER, &IID_IShellDispatch, (void**)&sd); + ok(SUCCEEDED(r), "CoCreateInstance failed: %08x\n", r); + ok(!!sd, "sd is null\n"); + + GetTempPathW(MAX_PATH, wstr); + GetCurrentDirectoryW(MAX_PATH, orig_dir); + SetCurrentDirectoryW(wstr); + CreateDirectoryW(winetestW, NULL); + GetFullPathNameW(winetestW, MAX_PATH, wstr, NULL); + V_VT(&var) = VT_BSTR; + V_BSTR(&var) = SysAllocString(wstr); + r = IShellDispatch_NameSpace(sd, var, &folder); + ok(r == S_OK, "IShellDispatch::NameSpace failed: %08x\n", r); + ok(!!folder, "folder is null\n"); + IShellDispatch_Release(sd); + SetCurrentDirectoryW(winetestW); + + r = Folder_Items(folder, &items); + ok(r == S_OK, "Folder::Items failed: %08x\n", r); + ok(!!items, "items is null\n"); + r = FolderItems_QueryInterface(items, &IID_FolderItems2, (void**)&items2); + ok(r == S_OK || broken(E_NOINTERFACE) /* xp and later */, "FolderItems::QueryInterface failed: %08x\n", r); + ok(!!items2 || broken(!items2) /* xp and later */, "items2 is null\n"); + r = FolderItems_QueryInterface(items, &IID_FolderItems3, (void**)&items3); + ok(r == S_OK, "FolderItems::QueryInterface failed: %08x\n", r); + ok(!!items3, "items3 is null\n"); + Folder_Release(folder); + + if (0) /* crashes on all versions of Windows */ + r = FolderItems_get_Count(items, NULL); + + r = FolderItems_get_Count(items, &lcount); +todo_wine + ok(r == S_OK, "FolderItems::get_Count failed: %08x\n", r); +todo_wine + ok(!lcount, "expected 0 files, got %d\n", lcount); + + V_VT(&var) = VT_I4; + V_I4(&var) = 0; + + if (0) /* crashes on all versions of Windows */ + r = FolderItems_Item(items, var, NULL); + + r = FolderItems_Item(items, var, &item); +todo_wine + ok(r == S_FALSE, "expected S_FALSE, got %08x\n", r); +todo_wine + ok(!item, "item is not null\n"); + + if (0) /* crashes on xp */ + { + r = FolderItems_get_Application(items, NULL); + ok(r == E_INVALIDARG, "expected E_INVALIDARG, got %08x\n", r); + } + + r = FolderItems_get_Application(items, &disp); +todo_wine + ok(r == S_OK, "FolderItems::get_Application failed: %08x\n", r); +todo_wine + ok(!!disp, "disp is null\n"); + if (disp) IDispatch_Release(disp); + + if (0) /* crashes on xp */ + { + r = FolderItems_get_Parent(items, NULL); + ok(r == E_NOTIMPL, "expected E_NOTIMPL, got %08x\n", r); + } + + disp = (IDispatch*)0xdeadbeef; + r = FolderItems_get_Parent(items, &disp); + ok(r == E_NOTIMPL, "expected E_NOTIMPL, got %08x\n", r); +todo_wine + ok(!disp, "disp is not null\n"); + + if (0) /* crashes on xp */ + { + r = FolderItems__NewEnum(items, NULL); + ok(r == E_INVALIDARG, "expected E_INVALIDARG, got %08x\n", r); + } + + r = FolderItems__NewEnum(items, &unk); +todo_wine + ok(r == S_OK, "FolderItems::_NewEnum failed: %08x\n", r); +todo_wine + ok(!!unk, "unk is null\n"); + if (unk) IUnknown_Release(unk); + + if (items3) + { + r = FolderItems3_Filter(items3, 0, NULL); +todo_wine + ok(r == S_OK, "expected S_OK, got %08x\n", r); + + if (0) /* crashes on xp */ + { + r = FolderItems3_get_Verbs(items3, NULL); + ok(r == E_INVALIDARG, "expected E_INVALIDARG, got %08x\n", r); + } + + r = FolderItems3_get_Verbs(items3, &verbs); +todo_wine + ok(r == S_FALSE, "expected S_FALSE, got %08x\n", r); +todo_wine + ok(!verbs, "verbs is not null\n"); + } + + RemoveDirectoryW(winetestW); + SetCurrentDirectoryW(orig_dir); + + FolderItems_Release(items); + if (items2) FolderItems2_Release(items2); + if (items3) FolderItems3_Release(items3); +} + static void test_service(void) { static const WCHAR spooler[] = {'S','p','o','o','l','e','r',0}; @@ -880,6 +1011,7 @@ START_TEST(shelldispatch) init_function_pointers(); test_namespace(); + test_items(); test_service(); test_ShellFolderViewDual(); test_ShellWindows();

1 0

Alex Henrie : shell32: Stub out FolderItems.
by Alexandre Julliard 19 Jul '16

19 Jul '16

Module: wine Branch: master Commit: 1ed577384321596f0026faf9eb5e30b24797cfcf URL: http://source.winehq.org/git/wine.git/?a=commit;h=1ed577384321596f0026faf9e… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Wed Jul 6 23:35:42 2016 -0600 shell32: Stub out FolderItems. Signed-off-by: Alex Henrie <alexhenrie24(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/shell32/shell32_main.h | 1 + dlls/shell32/shelldispatch.c | 204 ++++++++++++++++++++++++++++++++++++++++++- 2 files changed, 203 insertions(+), 2 deletions(-) diff --git a/dlls/shell32/shell32_main.h b/dlls/shell32/shell32_main.h index 492f79f..0178e10 100644 --- a/dlls/shell32/shell32_main.h +++ b/dlls/shell32/shell32_main.h @@ -235,6 +235,7 @@ enum tid_t { IShellFolderViewDual3_tid, Folder3_tid, FolderItem2_tid, + FolderItems3_tid, FolderItemVerb_tid, FolderItemVerbs_tid, LAST_tid diff --git a/dlls/shell32/shelldispatch.c b/dlls/shell32/shelldispatch.c index 62c1bd3..93c851f 100644 --- a/dlls/shell32/shelldispatch.c +++ b/dlls/shell32/shelldispatch.c @@ -48,6 +48,7 @@ static const IID * const tid_ids[] = &IID_IShellFolderViewDual3, &IID_Folder3, &IID_FolderItem2, + &IID_FolderItems3, &IID_FolderItemVerb, &IID_FolderItemVerbs }; @@ -65,6 +66,11 @@ typedef struct { } FolderImpl; typedef struct { + FolderItems3 FolderItems3_iface; + LONG ref; +} FolderItemsImpl; + +typedef struct { FolderItem2 FolderItem2_iface; LONG ref; VARIANT dir; @@ -97,6 +103,11 @@ static inline FolderImpl *impl_from_Folder(Folder3 *iface) return CONTAINING_RECORD(iface, FolderImpl, Folder3_iface); } +static inline FolderItemsImpl *impl_from_FolderItems(FolderItems3 *iface) +{ + return CONTAINING_RECORD(iface, FolderItemsImpl, FolderItems3_iface); +} + static inline FolderItemImpl *impl_from_FolderItem(FolderItem2 *iface) { return CONTAINING_RECORD(iface, FolderItemImpl, FolderItem2_iface); @@ -939,6 +950,196 @@ static HRESULT FolderItem_Constructor(VARIANT *dir, FolderItem **ppfi) return ret; } +static HRESULT WINAPI FolderItemsImpl_QueryInterface(FolderItems3 *iface, + REFIID riid, LPVOID *ppv) +{ + FolderItemsImpl *This = impl_from_FolderItems(iface); + + TRACE("(%p,%s,%p)\n", iface, shdebugstr_guid(riid), ppv); + + if (!ppv) return E_INVALIDARG; + + if (IsEqualIID(&IID_IUnknown, riid) || + IsEqualIID(&IID_IDispatch, riid) || + IsEqualIID(&IID_FolderItems, riid) || + IsEqualIID(&IID_FolderItems2, riid) || + IsEqualIID(&IID_FolderItems3, riid)) + *ppv = &This->FolderItems3_iface; + else + { + FIXME("not implemented for %s\n", shdebugstr_guid(riid)); + *ppv = NULL; + return E_NOINTERFACE; + } + IUnknown_AddRef((IUnknown*)*ppv); + return S_OK; +} + +static ULONG WINAPI FolderItemsImpl_AddRef(FolderItems3 *iface) +{ + FolderItemsImpl *This = impl_from_FolderItems(iface); + ULONG ref = InterlockedIncrement(&This->ref); + + TRACE("(%p), new refcount=%i\n", iface, ref); + + return ref; +} + +static ULONG WINAPI FolderItemsImpl_Release(FolderItems3 *iface) +{ + FolderItemsImpl *This = impl_from_FolderItems(iface); + ULONG ref = InterlockedDecrement(&This->ref); + + TRACE("(%p), new refcount=%i\n", iface, ref); + + if (!ref) + HeapFree(GetProcessHeap(), 0, This); + return ref; +} + +static HRESULT WINAPI FolderItemsImpl_GetTypeInfoCount(FolderItems3 *iface, + UINT *count) +{ + TRACE("(%p,%p)\n", iface, count); + + *count = 1; + return S_OK; +} + +static HRESULT WINAPI FolderItemsImpl_GetTypeInfo(FolderItems3 *iface, + UINT type, LCID lcid, ITypeInfo **ppti) +{ + HRESULT hr; + + TRACE("(%p,%u,%d,%p)\n", iface, type, lcid, ppti); + + hr = get_typeinfo(FolderItems3_tid, ppti); + if (SUCCEEDED(hr)) + ITypeInfo_AddRef(*ppti); + return hr; +} + +static HRESULT WINAPI FolderItemsImpl_GetIDsOfNames(FolderItems3 *iface, + REFIID riid, LPOLESTR *names, UINT count, LCID lcid, DISPID *dispid) +{ + ITypeInfo *ti; + HRESULT hr; + + TRACE("(%p,%s,%p,%u,%d,%p)\n", iface, shdebugstr_guid(riid), names, count, lcid, dispid); + + hr = get_typeinfo(FolderItems3_tid, &ti); + if (SUCCEEDED(hr)) + hr = ITypeInfo_GetIDsOfNames(ti, names, count, dispid); + return hr; +} + +static HRESULT WINAPI FolderItemsImpl_Invoke(FolderItems3 *iface, + DISPID dispid, REFIID riid, LCID lcid, WORD flags, DISPPARAMS *params, + VARIANT *result, EXCEPINFO *ei, UINT *err) +{ + FolderItemsImpl *This = impl_from_FolderItems(iface); + ITypeInfo *ti; + HRESULT hr; + + TRACE("(%p,%d,%s,%d,%u,%p,%p,%p,%p)\n", iface, dispid, shdebugstr_guid(riid), lcid, flags, params, result, ei, err); + + hr = get_typeinfo(FolderItems3_tid, &ti); + if (SUCCEEDED(hr)) + hr = ITypeInfo_Invoke(ti, This, dispid, flags, params, result, ei, err); + return hr; +} + +static HRESULT WINAPI FolderItemsImpl_get_Count(FolderItems3 *iface, LONG *count) +{ + FIXME("(%p,%p)\n", iface, count); + + return E_NOTIMPL; +} + +static HRESULT WINAPI FolderItemsImpl_get_Application(FolderItems3 *iface, IDispatch **ppid) +{ + FIXME("(%p,%p)\n", iface, ppid); + + return E_NOTIMPL; +} + +static HRESULT WINAPI FolderItemsImpl_get_Parent(FolderItems3 *iface, IDispatch **ppid) +{ + FIXME("(%p,%p)\n", iface, ppid); + + return E_NOTIMPL; +} + +static HRESULT WINAPI FolderItemsImpl_Item(FolderItems3 *iface, VARIANT index, FolderItem **ppid) +{ + FIXME("(%p,%s,%p)\n", iface, debugstr_variant(&index), ppid); + + return E_NOTIMPL; +} + +static HRESULT WINAPI FolderItemsImpl__NewEnum(FolderItems3 *iface, IUnknown **ppunk) +{ + FIXME("(%p,%p)\n", iface, ppunk); + + return E_NOTIMPL; +} + +static HRESULT WINAPI FolderItemsImpl_InvokeVerbEx(FolderItems3 *iface, VARIANT verb, VARIANT args) +{ + FIXME("(%p,%s,%s)\n", iface, debugstr_variant(&verb), debugstr_variant(&args)); + + return E_NOTIMPL; +} + +static HRESULT WINAPI FolderItemsImpl_Filter(FolderItems3 *iface, LONG flags, BSTR spec) +{ + FIXME("(%p,%d,%s)\n", iface, flags, wine_dbgstr_w(spec)); + + return E_NOTIMPL; +} + +static HRESULT WINAPI FolderItemsImpl_get_Verbs(FolderItems3 *iface, FolderItemVerbs **ppfic) +{ + FIXME("(%p,%p)\n", iface, ppfic); + + return E_NOTIMPL; +} + +static const FolderItems3Vtbl FolderItemsImpl_Vtbl = { + FolderItemsImpl_QueryInterface, + FolderItemsImpl_AddRef, + FolderItemsImpl_Release, + FolderItemsImpl_GetTypeInfoCount, + FolderItemsImpl_GetTypeInfo, + FolderItemsImpl_GetIDsOfNames, + FolderItemsImpl_Invoke, + FolderItemsImpl_get_Count, + FolderItemsImpl_get_Application, + FolderItemsImpl_get_Parent, + FolderItemsImpl_Item, + FolderItemsImpl__NewEnum, + FolderItemsImpl_InvokeVerbEx, + FolderItemsImpl_Filter, + FolderItemsImpl_get_Verbs +}; + +static HRESULT FolderItems_Constructor(FolderItems **ppfi) +{ + FolderItemsImpl *This; + + TRACE("\n"); + + *ppfi = NULL; + + This = HeapAlloc(GetProcessHeap(), 0, sizeof(FolderItemsImpl)); + if (!This) return E_OUTOFMEMORY; + This->FolderItems3_iface.lpVtbl = &FolderItemsImpl_Vtbl; + This->ref = 1; + + *ppfi = (FolderItems*)&This->FolderItems3_iface; + return S_OK; +} + static HRESULT WINAPI FolderImpl_QueryInterface(Folder3 *iface, REFIID riid, LPVOID *ppv) { @@ -1093,8 +1294,7 @@ static HRESULT WINAPI FolderImpl_Items(Folder3 *iface, FolderItems **ppid) { FIXME("(%p,%p)\n", iface, ppid); - *ppid = NULL; - return E_NOTIMPL; + return FolderItems_Constructor(ppid); } static HRESULT WINAPI FolderImpl_ParseName(Folder3 *iface, BSTR name, FolderItem **item)

1 0

Alex Henrie : shell32: Print RIIDs as strings in shell dispatch TRACE statements.
by Alexandre Julliard 19 Jul '16

19 Jul '16

Module: wine Branch: master Commit: ebf480c430c7d390b09386ceef0126c422ec229f URL: http://source.winehq.org/git/wine.git/?a=commit;h=ebf480c430c7d390b09386cee… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Wed Jul 6 23:35:41 2016 -0600 shell32: Print RIIDs as strings in shell dispatch TRACE statements. Signed-off-by: Alex Henrie <alexhenrie24(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/shell32/shelldispatch.c | 30 +++++++++++++++--------------- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/dlls/shell32/shelldispatch.c b/dlls/shell32/shelldispatch.c index 6dece6d..62c1bd3 100644 --- a/dlls/shell32/shelldispatch.c +++ b/dlls/shell32/shelldispatch.c @@ -176,7 +176,7 @@ static HRESULT WINAPI FolderItemVerbImpl_QueryInterface(FolderItemVerb *iface, { FolderItemVerbImpl *This = impl_from_FolderItemVerb(iface); - TRACE("(%p,%p,%p)\n", iface, riid, ppv); + TRACE("(%p,%s,%p)\n", iface, shdebugstr_guid(riid), ppv); *ppv = NULL; @@ -248,7 +248,7 @@ static HRESULT WINAPI FolderItemVerbImpl_GetIDsOfNames(FolderItemVerb *iface, ITypeInfo *ti; HRESULT hr; - TRACE("(%p,%p,%p,%u,%d,%p)\n", iface, riid, rgszNames, cNames, lcid, + TRACE("(%p,%s,%p,%u,%d,%p)\n", iface, shdebugstr_guid(riid), rgszNames, cNames, lcid, rgDispId); hr = get_typeinfo(FolderItemVerb_tid, &ti); @@ -265,7 +265,7 @@ static HRESULT WINAPI FolderItemVerbImpl_Invoke(FolderItemVerb *iface, ITypeInfo *ti; HRESULT hr; - TRACE("(%p,%d,%p,%d,%u,%p,%p,%p,%p)\n", iface, dispIdMember, riid, lcid, + TRACE("(%p,%d,%s,%d,%u,%p,%p,%p,%p)\n", iface, dispIdMember, shdebugstr_guid(riid), lcid, wFlags, pDispParams, pVarResult, pExcepInfo, puArgErr); hr = get_typeinfo(FolderItemVerb_tid, &ti); @@ -342,7 +342,7 @@ static HRESULT WINAPI FolderItemVerbsImpl_QueryInterface(FolderItemVerbs *iface, { FolderItemVerbsImpl *This = impl_from_FolderItemVerbs(iface); - TRACE("(%p,%p,%p)\n", iface, riid, ppv); + TRACE("(%p,%s,%p)\n", iface, shdebugstr_guid(riid), ppv); *ppv = NULL; @@ -414,7 +414,7 @@ static HRESULT WINAPI FolderItemVerbsImpl_GetIDsOfNames(FolderItemVerbs *iface, ITypeInfo *ti; HRESULT hr; - TRACE("(%p,%p,%p,%u,%d,%p)\n", iface, riid, rgszNames, cNames, lcid, + TRACE("(%p,%s,%p,%u,%d,%p)\n", iface, shdebugstr_guid(riid), rgszNames, cNames, lcid, rgDispId); hr = get_typeinfo(FolderItemVerbs_tid, &ti); @@ -431,7 +431,7 @@ static HRESULT WINAPI FolderItemVerbsImpl_Invoke(FolderItemVerbs *iface, ITypeInfo *ti; HRESULT hr; - TRACE("(%p,%d,%p,%d,%u,%p,%p,%p,%p)\n", iface, dispIdMember, riid, lcid, + TRACE("(%p,%d,%s,%d,%u,%p,%p,%p,%p)\n", iface, dispIdMember, shdebugstr_guid(riid), lcid, wFlags, pDispParams, pVarResult, pExcepInfo, puArgErr); hr = get_typeinfo(FolderItemVerbs_tid, &ti); @@ -592,7 +592,7 @@ static HRESULT WINAPI FolderItemImpl_QueryInterface(FolderItem2 *iface, { FolderItemImpl *This = impl_from_FolderItem(iface); - TRACE("(%p,%p,%p)\n", iface, riid, ppv); + TRACE("(%p,%s,%p)\n", iface, shdebugstr_guid(riid), ppv); if (!ppv) return E_INVALIDARG; @@ -665,7 +665,7 @@ static HRESULT WINAPI FolderItemImpl_GetIDsOfNames(FolderItem2 *iface, ITypeInfo *ti; HRESULT hr; - TRACE("(%p,%p,%p,%u,%d,%p)\n", iface, riid, rgszNames, cNames, lcid, + TRACE("(%p,%s,%p,%u,%d,%p)\n", iface, shdebugstr_guid(riid), rgszNames, cNames, lcid, rgDispId); hr = get_typeinfo(FolderItem2_tid, &ti); @@ -683,7 +683,7 @@ static HRESULT WINAPI FolderItemImpl_Invoke(FolderItem2 *iface, ITypeInfo *ti; HRESULT hr; - TRACE("(%p,%d,%p,%d,%u,%p,%p,%p,%p)\n", iface, dispIdMember, riid, lcid, + TRACE("(%p,%d,%s,%d,%u,%p,%p,%p,%p)\n", iface, dispIdMember, shdebugstr_guid(riid), lcid, wFlags, pDispParams, pVarResult, pExcepInfo, puArgErr); hr = get_typeinfo(FolderItem2_tid, &ti); @@ -944,7 +944,7 @@ static HRESULT WINAPI FolderImpl_QueryInterface(Folder3 *iface, REFIID riid, { FolderImpl *This = impl_from_Folder(iface); - TRACE("(%p,%p,%p)\n", iface, riid, ppv); + TRACE("(%p,%s,%p)\n", iface, shdebugstr_guid(riid), ppv); if (!ppv) return E_INVALIDARG; @@ -1017,7 +1017,7 @@ static HRESULT WINAPI FolderImpl_GetIDsOfNames(Folder3 *iface, REFIID riid, ITypeInfo *ti; HRESULT hr; - TRACE("(%p,%p,%p,%u,%d,%p)\n", iface, riid, rgszNames, cNames, lcid, + TRACE("(%p,%s,%p,%u,%d,%p)\n", iface, shdebugstr_guid(riid), rgszNames, cNames, lcid, rgDispId); hr = get_typeinfo(Folder3_tid, &ti); @@ -1034,7 +1034,7 @@ static HRESULT WINAPI FolderImpl_Invoke(Folder3 *iface, DISPID dispIdMember, ITypeInfo *ti; HRESULT hr; - TRACE("(%p,%d,%p,%d,%u,%p,%p,%p,%p)\n", iface, dispIdMember, riid, lcid, + TRACE("(%p,%d,%s,%d,%u,%p,%p,%p,%p)\n", iface, dispIdMember, shdebugstr_guid(riid), lcid, wFlags, pDispParams, pVarResult, pExcepInfo, puArgErr); hr = get_typeinfo(Folder3_tid, &ti); @@ -1289,7 +1289,7 @@ static HRESULT WINAPI ShellDispatch_QueryInterface(IShellDispatch6 *iface, { ShellDispatch *This = impl_from_IShellDispatch6(iface); - TRACE("(%p,%p,%p)\n", iface, riid, ppv); + TRACE("(%p,%s,%p)\n", iface, shdebugstr_guid(riid), ppv); if (!ppv) return E_INVALIDARG; @@ -1364,7 +1364,7 @@ static HRESULT WINAPI ShellDispatch_GetIDsOfNames(IShellDispatch6 *iface, ITypeInfo *ti; HRESULT hr; - TRACE("(%p,%p,%p,%u,%d,%p)\n", iface, riid, rgszNames, cNames, lcid, + TRACE("(%p,%s,%p,%u,%d,%p)\n", iface, shdebugstr_guid(riid), rgszNames, cNames, lcid, rgDispId); hr = get_typeinfo(IShellDispatch6_tid, &ti); @@ -1382,7 +1382,7 @@ static HRESULT WINAPI ShellDispatch_Invoke(IShellDispatch6 *iface, ITypeInfo *ti; HRESULT hr; - TRACE("(%p,%d,%p,%d,%u,%p,%p,%p,%p)\n", iface, dispIdMember, riid, lcid, + TRACE("(%p,%d,%s,%d,%u,%p,%p,%p,%p)\n", iface, dispIdMember, shdebugstr_guid(riid), lcid, wFlags, pDispParams, pVarResult, pExcepInfo, puArgErr); hr = get_typeinfo(IShellDispatch6_tid, &ti);

1 0

Alex Henrie : include: Add FolderItems2 and FolderItems3.
by Alexandre Julliard 19 Jul '16

19 Jul '16

Module: wine Branch: master Commit: 96072f4b6ab99c5d1214f78362a346859e262c26 URL: http://source.winehq.org/git/wine.git/?a=commit;h=96072f4b6ab99c5d1214f7836… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Wed Jul 6 23:35:40 2016 -0600 include: Add FolderItems2 and FolderItems3. Signed-off-by: Alex Henrie <alexhenrie24(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- include/shldisp.idl | 30 ++++++++++++++++++++++++++++++ 1 file changed, 30 insertions(+) diff --git a/include/shldisp.idl b/include/shldisp.idl index 56be778..f5372c4 100644 --- a/include/shldisp.idl +++ b/include/shldisp.idl @@ -350,6 +350,36 @@ interface FolderItems : IDispatch } /***************************************************************************** + * FolderItems2 interface + */ +[ + uuid(c94f0ad0-f363-11d2-a327-00c04f8eec7f), + oleautomation, + dual +] +interface FolderItems2 : FolderItems +{ + HRESULT InvokeVerbEx( [in, optional] VARIANT verb, + [in, optional] VARIANT args ); +} + +/***************************************************************************** + * FolderItems3 interface + */ +[ + uuid(eaa7c309-bbec-49d5-821d-64d966cb667f), + oleautomation, + dual +] +interface FolderItems3 : FolderItems2 +{ + HRESULT Filter( [in] long flags, [in] BSTR spec ); + + [id(0), propget] + HRESULT Verbs( [out, retval] FolderItemVerbs **ppfic ); +} + +/***************************************************************************** * FolderItemVerb interface */ [

1 0

Austin English : shell32: Add SHGetPropertyStoreForWindow stub.
by Alexandre Julliard 19 Jul '16

19 Jul '16

Module: wine Branch: master Commit: ebfe169c4389240dc57d3e1bcdc697e3ed4f5cfc URL: http://source.winehq.org/git/wine.git/?a=commit;h=ebfe169c4389240dc57d3e1bc… Author: Austin English <austinenglish(a)gmail.com> Date: Tue Jul 19 00:20:07 2016 -0500 shell32: Add SHGetPropertyStoreForWindow stub. Signed-off-by: Austin English <austinenglish(a)gmail.com> Signed-off-by: Andrew Eikum <aeikum(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/shell32/shell32.spec | 1 + dlls/shell32/shell32_main.c | 9 +++++++++ include/shellapi.h | 1 + 3 files changed, 11 insertions(+) diff --git a/dlls/shell32/shell32.spec b/dlls/shell32/shell32.spec index 7ebd032..f95aaa7 100644 --- a/dlls/shell32/shell32.spec +++ b/dlls/shell32/shell32.spec @@ -390,6 +390,7 @@ @ stdcall SHGetPathFromIDList(ptr ptr) SHGetPathFromIDListA @ stdcall SHGetPathFromIDListA(ptr ptr) @ stdcall SHGetPathFromIDListW(ptr ptr) +@ stdcall SHGetPropertyStoreForWindow(long ptr ptr) @ stdcall SHGetPropertyStoreFromParsingName(wstr ptr long ptr ptr) @ stdcall SHGetSettings(ptr long) @ stdcall SHGetSpecialFolderLocation(long long ptr) diff --git a/dlls/shell32/shell32_main.c b/dlls/shell32/shell32_main.c index 74deecd..dce1896 100644 --- a/dlls/shell32/shell32_main.c +++ b/dlls/shell32/shell32_main.c @@ -943,6 +943,15 @@ VOID WINAPI Printers_UnregisterWindow(HANDLE hClassPidl, HWND hwnd) } /************************************************************************* + * SHGetPropertyStoreForWindow [SHELL32.@] + */ +HRESULT WINAPI SHGetPropertyStoreForWindow(HWND hwnd, REFIID riid, void **ppv) +{ + FIXME("(%p %p %p) stub!\n", hwnd, riid, ppv); + return E_NOTIMPL; +} + +/************************************************************************* * SHGetPropertyStoreFromParsingName [SHELL32.@] */ HRESULT WINAPI SHGetPropertyStoreFromParsingName(PCWSTR pszPath, IBindCtx *pbc, GETPROPERTYSTOREFLAGS flags, REFIID riid, void **ppv) diff --git a/include/shellapi.h b/include/shellapi.h index b94626f..d1aad6b 100644 --- a/include/shellapi.h +++ b/include/shellapi.h @@ -660,6 +660,7 @@ HRESULT WINAPI SHEnumerateUnreadMailAccountsA(HKEY,DWORD,LPSTR,INT); HRESULT WINAPI SHEnumerateUnreadMailAccountsW(HKEY,DWORD,LPWSTR,INT); #define SHEnumerateUnreadMailAccounts WINELIB_NAME_AW(SHEnumerateUnreadMailAccounts) +HRESULT WINAPI SHGetPropertyStoreForWindow(HWND,REFIID,void **); #ifdef __cplusplus } /* extern "C" */

1 0

Michael Müller : kernel32: In Win9x mode UnmapViewOfFile requires base address of a mapping.
by Alexandre Julliard 19 Jul '16

19 Jul '16

Module: wine Branch: master Commit: 1b26d38095e7d710375657b64db80c7c858ebea6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1b26d38095e7d710375657b64… Author: Michael Müller <michael(a)fds-team.de> Date: Tue Jul 19 13:00:30 2016 +0200 kernel32: In Win9x mode UnmapViewOfFile requires base address of a mapping. Signed-off-by: Michael Müller <michael(a)fds-team.de> Signed-off-by: Sebastian Lackner <sebastian(a)fds-team.de> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/virtual.c | 14 +++++++++++++- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/dlls/kernel32/virtual.c b/dlls/kernel32/virtual.c index 37ab264..5733a42 100644 --- a/dlls/kernel32/virtual.c +++ b/dlls/kernel32/virtual.c @@ -576,7 +576,19 @@ LPVOID WINAPI MapViewOfFileEx( HANDLE handle, DWORD access, */ BOOL WINAPI UnmapViewOfFile( LPCVOID addr ) { - NTSTATUS status = NtUnmapViewOfSection( GetCurrentProcess(), (void *)addr ); + NTSTATUS status; + + if (GetVersion() & 0x80000000) + { + MEMORY_BASIC_INFORMATION info; + if (!VirtualQuery( addr, &info, sizeof(info) ) || info.AllocationBase != addr) + { + SetLastError( ERROR_INVALID_ADDRESS ); + return FALSE; + } + } + + status = NtUnmapViewOfSection( GetCurrentProcess(), (void *)addr ); if (status) SetLastError( RtlNtStatusToDosError(status) ); return !status; }

1 0

← Newer
1
...
19
20
21
22
23
24
25
...
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 July 2016