winehq.org
Sign In
Sign Up
Sign In
Sign Up
Manage this list
×
Keyboard Shortcuts
Thread View
j
: Next unread message
k
: Previous unread message
j a
: Jump to all threads
j l
: Jump to MailingList overview
2025
February
January
2024
December
November
October
September
August
July
June
May
April
March
February
January
2023
December
November
October
September
August
July
June
May
April
March
February
January
2022
December
November
October
September
August
July
June
May
April
March
February
January
2021
December
November
October
September
August
July
June
May
April
March
February
January
2020
December
November
October
September
August
July
June
May
April
March
February
January
2019
December
November
October
September
August
July
June
May
April
March
February
January
2018
December
November
October
September
August
July
June
May
April
March
February
January
2017
December
November
October
September
August
July
June
May
April
March
February
January
2016
December
November
October
September
August
July
June
May
April
March
February
January
2015
December
November
October
September
August
July
June
May
April
March
February
January
2014
December
November
October
September
August
July
June
May
April
March
February
January
2013
December
November
October
September
August
July
June
May
April
March
February
January
2012
December
November
October
September
August
July
June
May
April
March
February
January
2011
December
November
October
September
August
July
June
May
April
March
February
January
2010
December
November
October
September
August
July
June
May
April
March
February
January
2009
December
November
October
September
August
July
June
May
April
March
February
January
2008
December
November
October
September
August
July
June
May
April
March
February
January
2007
December
November
October
September
August
July
June
May
April
March
February
January
2006
December
November
October
September
August
July
June
May
April
March
February
January
2005
December
November
October
September
August
July
June
May
April
March
February
January
2004
December
November
October
September
August
July
June
May
April
March
February
January
2003
December
November
October
September
August
July
June
May
April
March
February
January
2002
December
November
October
September
August
July
June
May
April
March
February
January
2001
December
November
October
September
August
July
June
May
April
March
February
List overview
wine-commits
May 2006
----- 2025 -----
February 2025
January 2025
----- 2024 -----
December 2024
November 2024
October 2024
September 2024
August 2024
July 2024
June 2024
May 2024
April 2024
March 2024
February 2024
January 2024
----- 2023 -----
December 2023
November 2023
October 2023
September 2023
August 2023
July 2023
June 2023
May 2023
April 2023
March 2023
February 2023
January 2023
----- 2022 -----
December 2022
November 2022
October 2022
September 2022
August 2022
July 2022
June 2022
May 2022
April 2022
March 2022
February 2022
January 2022
----- 2021 -----
December 2021
November 2021
October 2021
September 2021
August 2021
July 2021
June 2021
May 2021
April 2021
March 2021
February 2021
January 2021
----- 2020 -----
December 2020
November 2020
October 2020
September 2020
August 2020
July 2020
June 2020
May 2020
April 2020
March 2020
February 2020
January 2020
----- 2019 -----
December 2019
November 2019
October 2019
September 2019
August 2019
July 2019
June 2019
May 2019
April 2019
March 2019
February 2019
January 2019
----- 2018 -----
December 2018
November 2018
October 2018
September 2018
August 2018
July 2018
June 2018
May 2018
April 2018
March 2018
February 2018
January 2018
----- 2017 -----
December 2017
November 2017
October 2017
September 2017
August 2017
July 2017
June 2017
May 2017
April 2017
March 2017
February 2017
January 2017
----- 2016 -----
December 2016
November 2016
October 2016
September 2016
August 2016
July 2016
June 2016
May 2016
April 2016
March 2016
February 2016
January 2016
----- 2015 -----
December 2015
November 2015
October 2015
September 2015
August 2015
July 2015
June 2015
May 2015
April 2015
March 2015
February 2015
January 2015
----- 2014 -----
December 2014
November 2014
October 2014
September 2014
August 2014
July 2014
June 2014
May 2014
April 2014
March 2014
February 2014
January 2014
----- 2013 -----
December 2013
November 2013
October 2013
September 2013
August 2013
July 2013
June 2013
May 2013
April 2013
March 2013
February 2013
January 2013
----- 2012 -----
December 2012
November 2012
October 2012
September 2012
August 2012
July 2012
June 2012
May 2012
April 2012
March 2012
February 2012
January 2012
----- 2011 -----
December 2011
November 2011
October 2011
September 2011
August 2011
July 2011
June 2011
May 2011
April 2011
March 2011
February 2011
January 2011
----- 2010 -----
December 2010
November 2010
October 2010
September 2010
August 2010
July 2010
June 2010
May 2010
April 2010
March 2010
February 2010
January 2010
----- 2009 -----
December 2009
November 2009
October 2009
September 2009
August 2009
July 2009
June 2009
May 2009
April 2009
March 2009
February 2009
January 2009
----- 2008 -----
December 2008
November 2008
October 2008
September 2008
August 2008
July 2008
June 2008
May 2008
April 2008
March 2008
February 2008
January 2008
----- 2007 -----
December 2007
November 2007
October 2007
September 2007
August 2007
July 2007
June 2007
May 2007
April 2007
March 2007
February 2007
January 2007
----- 2006 -----
December 2006
November 2006
October 2006
September 2006
August 2006
July 2006
June 2006
May 2006
April 2006
March 2006
February 2006
January 2006
----- 2005 -----
December 2005
November 2005
October 2005
September 2005
August 2005
July 2005
June 2005
May 2005
April 2005
March 2005
February 2005
January 2005
----- 2004 -----
December 2004
November 2004
October 2004
September 2004
August 2004
July 2004
June 2004
May 2004
April 2004
March 2004
February 2004
January 2004
----- 2003 -----
December 2003
November 2003
October 2003
September 2003
August 2003
July 2003
June 2003
May 2003
April 2003
March 2003
February 2003
January 2003
----- 2002 -----
December 2002
November 2002
October 2002
September 2002
August 2002
July 2002
June 2002
May 2002
April 2002
March 2002
February 2002
January 2002
----- 2001 -----
December 2001
November 2001
October 2001
September 2001
August 2001
July 2001
June 2001
May 2001
April 2001
March 2001
February 2001
wine-commits@winehq.org
3 participants
863 discussions
Start a n
N
ew thread
Robert Shearman : ole32: The type check in AntiMonikerImpl_CommonPrefixWith should be for anti-monikers , not item-monikers.
by Alexandre Julliard
08 May '06
08 May '06
Module: wine Branch: refs/heads/master Commit: 1c6f16a384d356e3c3fb0e03bd17b38e4b6e3b62 URL:
http://source.winehq.org/git/?p=wine.git;a=commit;h=1c6f16a384d356e3c3fb0e0…
Author: Robert Shearman <rob(a)codeweavers.com> Date: Mon May 8 15:55:52 2006 +0100 ole32: The type check in AntiMonikerImpl_CommonPrefixWith should be for anti-monikers, not item-monikers. --- dlls/ole32/antimoniker.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/ole32/antimoniker.c b/dlls/ole32/antimoniker.c index 77d883a..472683f 100644 --- a/dlls/ole32/antimoniker.c +++ b/dlls/ole32/antimoniker.c @@ -409,7 +409,7 @@ AntiMonikerImpl_CommonPrefixWith(IMonike IMoniker_IsSystemMoniker(pmkOther,&mkSys); - if(mkSys==MKSYS_ITEMMONIKER){ + if(mkSys==MKSYS_ANTIMONIKER){ IMoniker_AddRef(iface);
1
0
0
0
Robert Shearman : ole32: Implement several class moniker functions using the description provided on MSDN .
by Alexandre Julliard
08 May '06
08 May '06
Module: wine Branch: refs/heads/master Commit: ae62ab039f4a1594aadf820b62084945f90d1066 URL:
http://source.winehq.org/git/?p=wine.git;a=commit;h=ae62ab039f4a1594aadf820…
Author: Robert Shearman <rob(a)codeweavers.com> Date: Mon May 8 15:55:40 2006 +0100 ole32: Implement several class moniker functions using the description provided on MSDN. --- dlls/ole32/classmoniker.c | 58 +++++++++++++++++++++++++++++++++++--------- dlls/ole32/tests/moniker.c | 3 -- 2 files changed, 46 insertions(+), 15 deletions(-) diff --git a/dlls/ole32/classmoniker.c b/dlls/ole32/classmoniker.c index 74a0952..afdf48a 100644 --- a/dlls/ole32/classmoniker.c +++ b/dlls/ole32/classmoniker.c @@ -242,8 +242,33 @@ static HRESULT WINAPI ClassMoniker_BindT REFIID riid, VOID** ppvResult) { - FIXME("(%p,%p,%p,%p)\n", pbc, pmkToLeft, riid, ppvResult); - return E_NOTIMPL; + ClassMoniker *This = (ClassMoniker *)iface; + BIND_OPTS2 bindopts; + IClassActivator *pActivator; + HRESULT hr; + + TRACE("(%p,%p,%p,%p)\n", pbc, pmkToLeft, riid, ppvResult); + + bindopts.cbStruct = sizeof(bindopts); + IBindCtx_GetBindOptions(pbc, (BIND_OPTS *)&bindopts); + + if (!pmkToLeft) + return CoGetClassObject(&This->clsid, bindopts.dwClassContext, NULL, + riid, ppvResult); + else + { + hr = IMoniker_BindToObject(pmkToLeft, pbc, NULL, &IID_IClassActivator, + (void **)&pActivator); + if (FAILED(hr)) return hr; + + hr = IClassActivator_GetClassObject(pActivator, &This->clsid, + bindopts.dwClassContext, + bindopts.locale, riid, ppvResult); + + IClassActivator_Release(pActivator); + + return hr; + } } /****************************************************************************** @@ -255,8 +280,8 @@ static HRESULT WINAPI ClassMoniker_BindT REFIID riid, VOID** ppvResult) { - FIXME("(%p,%p,%p,%p)\n",pbc, pmkToLeft, riid, ppvResult); - return E_NOTIMPL; + TRACE("(%p,%p,%p,%p)\n",pbc, pmkToLeft, riid, ppvResult); + return ClassMoniker_BindToObject(iface, pbc, pmkToLeft, riid, ppvResult); } /****************************************************************************** @@ -424,8 +449,9 @@ static HRESULT WINAPI ClassMoniker_IsRun IMoniker* pmkToLeft, IMoniker* pmkNewlyRunning) { - FIXME("(%p, %p, %p)\n", pbc, pmkToLeft, pmkNewlyRunning); + TRACE("(%p, %p, %p)\n", pbc, pmkToLeft, pmkNewlyRunning); + /* as in native */ return E_NOTIMPL; } @@ -437,8 +463,9 @@ static HRESULT WINAPI ClassMoniker_GetTi IMoniker* pmkToLeft, FILETIME* pItemTime) { - FIXME("(%p, %p, %p)\n", pbc, pmkToLeft, pItemTime); - return E_NOTIMPL; + TRACE("(%p, %p, %p)\n", pbc, pmkToLeft, pItemTime); + + return MK_E_UNAVAILABLE; } /****************************************************************************** @@ -463,18 +490,25 @@ static HRESULT WINAPI ClassMoniker_Commo TRACE("(%p, %p)\n", pmkOther, ppmkPrefix); + *ppmkPrefix = NULL; + IMoniker_IsSystemMoniker(pmkOther, &mkSys); /* If the other moniker is an class moniker that is equal to this moniker, this method sets *ppmkPrefix */ /* to this moniker and returns MK_S_US */ - if((mkSys == MKSYS_CLASSMONIKER) && (IMoniker_IsEqual(iface, pmkOther) == S_OK) ){ - - *ppmkPrefix = iface; + if (mkSys == MKSYS_CLASSMONIKER) + { + if (IMoniker_IsEqual(iface, pmkOther) == S_OK) + { + *ppmkPrefix = iface; - IMoniker_AddRef(iface); + IMoniker_AddRef(iface); - return MK_S_US; + return MK_S_US; + } + else + return MK_E_NOPREFIX; } else /* otherwise, the method calls the MonikerCommonPrefixWith function. This function correctly handles */ diff --git a/dlls/ole32/tests/moniker.c b/dlls/ole32/tests/moniker.c index 14dcda3..6ae8cf1 100644 --- a/dlls/ole32/tests/moniker.c +++ b/dlls/ole32/tests/moniker.c @@ -523,16 +523,13 @@ static void test_class_moniker(void) ok(hr == E_NOTIMPL, "IMoniker_IsRunning should return E_NOTIMPL, not 0x%08lx\n", hr); hr = IMoniker_GetTimeOfLastChange(moniker, bindctx, NULL, &filetime); - todo_wine ok(hr == MK_E_UNAVAILABLE, "IMoniker_GetTimeOfLastChange should return MK_E_UNAVAILABLE, not 0x%08lx\n", hr); hr = IMoniker_BindToObject(moniker, bindctx, NULL, &IID_IUnknown, (void **)&unknown); - todo_wine ok_ole_success(hr, IMoniker_BindToStorage); IUnknown_Release(unknown); hr = IMoniker_BindToStorage(moniker, bindctx, NULL, &IID_IUnknown, (void **)&unknown); - todo_wine ok_ole_success(hr, IMoniker_BindToStorage); IUnknown_Release(unknown);
1
0
0
0
Robert Shearman : ole32: Remove duplicated CLSIDs.
by Alexandre Julliard
08 May '06
08 May '06
Module: wine Branch: refs/heads/master Commit: b155f233f30cb2dfc4b2bd20e870e8b6bda9dff2 URL:
http://source.winehq.org/git/?p=wine.git;a=commit;h=b155f233f30cb2dfc4b2bd2…
Author: Robert Shearman <rob(a)codeweavers.com> Date: Mon May 8 15:55:25 2006 +0100 ole32: Remove duplicated CLSIDs. --- dlls/ole32/compobj_private.h | 3 +++ dlls/ole32/regsvr.c | 24 +----------------------- 2 files changed, 4 insertions(+), 23 deletions(-) diff --git a/dlls/ole32/compobj_private.h b/dlls/ole32/compobj_private.h index 66a7c1c..f362d16 100644 --- a/dlls/ole32/compobj_private.h +++ b/dlls/ole32/compobj_private.h @@ -41,6 +41,9 @@ #include "winternl.h" struct apartment; typedef struct apartment APARTMENT; +extern const CLSID CLSID_PSFactoryBuffer; +extern const CLSID CLSID_DfMarshal; + /* Thread-safety Annotation Legend: * * RO - The value is read only. It never changes after creation, so no diff --git a/dlls/ole32/regsvr.c b/dlls/ole32/regsvr.c index 31f0e22..741821f 100644 --- a/dlls/ole32/regsvr.c +++ b/dlls/ole32/regsvr.c @@ -33,6 +33,7 @@ #include "objbase.h" #include "compobj_private.h" #include "ole2.h" #include "olectl.h" +#include "moniker.h" #include "wine/debug.h" @@ -398,35 +399,12 @@ static LONG recursive_delete_key(HKEY ke /*********************************************************************** * coclass list */ -static GUID const CLSID_FileMoniker = { - 0x00000303, 0x0000, 0x0000, {0xC0,0x00,0x00,0x00,0x00,0x00,0x00,0x46} }; - -static GUID const CLSID_ItemMoniker = { - 0x00000304, 0x0000, 0x0000, {0xC0,0x00,0x00,0x00,0x00,0x00,0x00,0x46} }; - -static GUID const CLSID_AntiMoniker = { - 0x00000305, 0x0000, 0x0000, {0xC0,0x00,0x00,0x00,0x00,0x00,0x00,0x46} }; - static GUID const CLSID_PointerMoniker = { 0x00000306, 0x0000, 0x0000, {0xC0,0x00,0x00,0x00,0x00,0x00,0x00,0x46} }; static GUID const CLSID_PackagerMoniker = { 0x00000308, 0x0000, 0x0000, {0xC0,0x00,0x00,0x00,0x00,0x00,0x00,0x46} }; -static GUID const CLSID_CompositeMoniker = { - 0x00000309, 0x0000, 0x0000, {0xC0,0x00,0x00,0x00,0x00,0x00,0x00,0x46} }; - -/* FIXME: DfMarshal and PSFactoryBuffer are defined elsewhere too */ - -static GUID const CLSID_DfMarshal = { - 0x0000030B, 0x0000, 0x0000, {0xC0,0x00,0x00,0x00,0x00,0x00,0x00,0x46} }; - -static GUID const CLSID_ClassMoniker = { - 0x0000031A, 0x0000, 0x0000, {0xC0,0x00,0x00,0x00,0x00,0x00,0x00,0x46} }; - -static GUID const CLSID_PSFactoryBuffer = { - 0x00000320, 0x0000, 0x0000, {0xC0,0x00,0x00,0x00,0x00,0x00,0x00,0x46} }; - static struct regsvr_coclass const coclass_list[] = { { &CLSID_FileMoniker, "FileMoniker",
1
0
0
0
Robert Shearman : ole32: Add an implementation of class monikers.
by Alexandre Julliard
08 May '06
08 May '06
Module: wine Branch: refs/heads/master Commit: 8a4f71c61e4c3e9668245f2e39be5b24065f1a80 URL:
http://source.winehq.org/git/?p=wine.git;a=commit;h=8a4f71c61e4c3e9668245f2…
Author: Robert Shearman <rob(a)codeweavers.com> Date: Mon May 8 15:55:14 2006 +0100 ole32: Add an implementation of class monikers. --- dlls/ole32/Makefile.in | 1 dlls/ole32/classmoniker.c | 771 ++++++++++++++++++++++++++++++++++++++++++++ dlls/ole32/moniker.c | 11 - dlls/ole32/moniker.h | 2 dlls/ole32/oleproxy.c | 2 dlls/ole32/tests/moniker.c | 5 6 files changed, 780 insertions(+), 12 deletions(-) create mode 100644 dlls/ole32/classmoniker.c Diff:
http://source.winehq.org/git/?p=wine.git;a=commitdiff;h=8a4f71c61e4c3e96682…
1
0
0
0
Robert Shearman : ole32: Add tests for IsRunning, GetTimeOfLastChange, BindToObject, BindToStorage and Inverse.
by Alexandre Julliard
08 May '06
08 May '06
Module: wine Branch: refs/heads/master Commit: 71a7393f451cca87d0c1625dbc294fa84e0c094e URL:
http://source.winehq.org/git/?p=wine.git;a=commit;h=71a7393f451cca87d0c1625…
Author: Robert Shearman <rob(a)codeweavers.com> Date: Mon May 8 15:55:01 2006 +0100 ole32: Add tests for IsRunning, GetTimeOfLastChange, BindToObject, BindToStorage and Inverse. --- dlls/ole32/tests/moniker.c | 102 ++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 101 insertions(+), 1 deletions(-) diff --git a/dlls/ole32/tests/moniker.c b/dlls/ole32/tests/moniker.c index 2e7cd7d..bb22b74 100644 --- a/dlls/ole32/tests/moniker.c +++ b/dlls/ole32/tests/moniker.c @@ -482,6 +482,10 @@ static void test_class_moniker(void) IMoniker *moniker; DWORD moniker_type; DWORD hash; + IBindCtx *bindctx; + IMoniker *inverse; + IUnknown *unknown; + FILETIME filetime; hr = CreateClassMoniker(&CLSID_StdComponentCategoriesMgr, &moniker); todo_wine ok_ole_success(hr, CreateClassMoniker); @@ -511,6 +515,30 @@ static void test_class_moniker(void) "dwMkSys != MKSYS_CLASSMONIKER, instead was 0x%08lx", moniker_type); + hr = CreateBindCtx(0, &bindctx); + ok_ole_success(hr, CreateBindCtx); + + /* IsRunning test */ + hr = IMoniker_IsRunning(moniker, bindctx, NULL, NULL); + ok(hr == E_NOTIMPL, "IMoniker_IsRunning should return E_NOTIMPL, not 0x%08lx\n", hr); + + hr = IMoniker_GetTimeOfLastChange(moniker, bindctx, NULL, &filetime); + ok(hr == MK_E_UNAVAILABLE, "IMoniker_GetTimeOfLastChange should return MK_E_UNAVAILABLE, not 0x%08lx\n", hr); + + hr = IMoniker_BindToObject(moniker, bindctx, NULL, &IID_IUnknown, (void **)&unknown); + ok_ole_success(hr, IMoniker_BindToStorage); + IUnknown_Release(unknown); + + hr = IMoniker_BindToStorage(moniker, bindctx, NULL, &IID_IUnknown, (void **)&unknown); + ok_ole_success(hr, IMoniker_BindToStorage); + IUnknown_Release(unknown); + + IBindCtx_Release(bindctx); + + hr = IMoniker_Inverse(moniker, &inverse); + ok_ole_success(hr, IMoniker_Inverse); + IMoniker_Release(inverse); + IMoniker_Release(moniker); } @@ -582,6 +610,9 @@ static void test_item_moniker(void) IMoniker *moniker; DWORD moniker_type; DWORD hash; + IBindCtx *bindctx; + IMoniker *inverse; + IUnknown *unknown; static const WCHAR wszDelimeter[] = {'!',0}; static const WCHAR wszObjectName[] = {'T','e','s','t',0}; static const WCHAR expected_display_name[] = { '!','T','e','s','t',0 }; @@ -613,6 +644,25 @@ static void test_item_moniker(void) "dwMkSys != MKSYS_ITEMMONIKER, instead was 0x%08lx", moniker_type); + hr = CreateBindCtx(0, &bindctx); + ok_ole_success(hr, CreateBindCtx); + + /* IsRunning test */ + hr = IMoniker_IsRunning(moniker, bindctx, NULL, NULL); + ok(hr == S_FALSE, "IMoniker_IsRunning should return S_FALSE, not 0x%08lx\n", hr); + + hr = IMoniker_BindToObject(moniker, bindctx, NULL, &IID_IUnknown, (void **)&unknown); + ok(hr == E_INVALIDARG, "IMoniker_BindToStorage should return E_INVALIDARG, not 0x%08lx\n", hr); + + hr = IMoniker_BindToStorage(moniker, bindctx, NULL, &IID_IUnknown, (void **)&unknown); + ok(hr == E_INVALIDARG, "IMoniker_BindToObject should return E_INVALIDARG, not 0x%08lx\n", hr); + + IBindCtx_Release(bindctx); + + hr = IMoniker_Inverse(moniker, &inverse); + ok_ole_success(hr, IMoniker_Inverse); + IMoniker_Release(inverse); + IMoniker_Release(moniker); } @@ -620,9 +670,12 @@ static void test_anti_moniker(void) { HRESULT hr; IMoniker *moniker; - IMoniker *inverse; DWORD moniker_type; DWORD hash; + IBindCtx *bindctx; + FILETIME filetime; + IMoniker *inverse; + IUnknown *unknown; static const WCHAR expected_display_name[] = { '\\','.','.',0 }; hr = CreateAntiMoniker(&moniker); @@ -653,6 +706,24 @@ static void test_anti_moniker(void) ok(hr == MK_E_NOINVERSE, "IMoniker_Inverse should have returned MK_E_NOINVERSE instead of 0x%08lx\n", hr); ok(inverse == NULL, "inverse should have been set to NULL instead of %p\n", inverse); + hr = CreateBindCtx(0, &bindctx); + ok_ole_success(hr, CreateBindCtx); + + /* IsRunning test */ + hr = IMoniker_IsRunning(moniker, bindctx, NULL, NULL); + ok(hr == S_FALSE, "IMoniker_IsRunning should return S_FALSE, not 0x%08lx\n", hr); + + hr = IMoniker_GetTimeOfLastChange(moniker, bindctx, NULL, &filetime); + ok(hr == E_NOTIMPL, "IMoniker_GetTimeOfLastChange should return E_NOTIMPL, not 0x%08lx\n", hr); + + hr = IMoniker_BindToObject(moniker, bindctx, NULL, &IID_IUnknown, (void **)&unknown); + ok(hr == E_NOTIMPL, "IMoniker_BindToObject should return E_NOTIMPL, not 0x%08lx\n", hr); + + hr = IMoniker_BindToStorage(moniker, bindctx, NULL, &IID_IUnknown, (void **)&unknown); + ok(hr == E_NOTIMPL, "IMoniker_BindToStorage should return E_NOTIMPL, not 0x%08lx\n", hr); + + IBindCtx_Release(bindctx); + IMoniker_Release(moniker); } @@ -664,6 +735,10 @@ static void test_generic_composite_monik IMoniker *moniker2; DWORD moniker_type; DWORD hash; + IBindCtx *bindctx; + FILETIME filetime; + IMoniker *inverse; + IUnknown *unknown; static const WCHAR wszDelimeter1[] = {'!',0}; static const WCHAR wszObjectName1[] = {'T','e','s','t',0}; static const WCHAR wszDelimeter2[] = {'#',0}; @@ -701,6 +776,31 @@ static void test_generic_composite_monik "dwMkSys != MKSYS_GENERICCOMPOSITE, instead was 0x%08lx", moniker_type); + hr = CreateBindCtx(0, &bindctx); + ok_ole_success(hr, CreateBindCtx); + + /* IsRunning test */ + hr = IMoniker_IsRunning(moniker, bindctx, NULL, NULL); + todo_wine + ok(hr == S_FALSE, "IMoniker_IsRunning should return S_FALSE, not 0x%08lx\n", hr); + + hr = IMoniker_GetTimeOfLastChange(moniker, bindctx, NULL, &filetime); + ok(hr == MK_E_NOTBINDABLE, "IMoniker_GetTimeOfLastChange should return MK_E_NOTBINDABLE, not 0x%08lx\n", hr); + + hr = IMoniker_BindToObject(moniker, bindctx, NULL, &IID_IUnknown, (void **)&unknown); + todo_wine + ok(hr == E_INVALIDARG, "IMoniker_BindToObject should return E_INVALIDARG, not 0x%08lx\n", hr); + + todo_wine + hr = IMoniker_BindToStorage(moniker, bindctx, NULL, &IID_IUnknown, (void **)&unknown); + ok(hr == E_INVALIDARG, "IMoniker_BindToStorage should return E_INVALIDARG, not 0x%08lx\n", hr); + + IBindCtx_Release(bindctx); + + hr = IMoniker_Inverse(moniker, &inverse); + ok_ole_success(hr, IMoniker_Inverse); + IMoniker_Release(inverse); + IMoniker_Release(moniker); }
1
0
0
0
Robert Shearman : ole32: Fix regression caused by an earlier patch that removed the
by Alexandre Julliard
08 May '06
08 May '06
Module: wine Branch: refs/heads/master Commit: 288fb09b314f6d3e761323092a7520772775bf7f URL:
http://source.winehq.org/git/?p=wine.git;a=commit;h=288fb09b314f6d3e7613230…
Author: Robert Shearman <rob(a)codeweavers.com> Date: Mon May 8 15:54:52 2006 +0100 ole32: Fix regression caused by an earlier patch that removed the check for the composite moniker having only one moniker in it. In this case, return the one moniker and free the memory associate with the composite moniker. --- dlls/ole32/compositemoniker.c | 39 ++++++++++++++++++++++++++++----------- 1 files changed, 28 insertions(+), 11 deletions(-) diff --git a/dlls/ole32/compositemoniker.c b/dlls/ole32/compositemoniker.c index 059eabf..8912e6d 100644 --- a/dlls/ole32/compositemoniker.c +++ b/dlls/ole32/compositemoniker.c @@ -1764,7 +1764,7 @@ static const IMarshalVtbl VT_MarshalImpl * Composite-Moniker_Construct (local function) *******************************************************************************/ static HRESULT -CompositeMonikerImpl_Construct(CompositeMonikerImpl** ppThis, +CompositeMonikerImpl_Construct(IMoniker** ppMoniker, LPMONIKER pmkFirst, LPMONIKER pmkRest) { DWORD mkSys; @@ -1773,7 +1773,7 @@ CompositeMonikerImpl_Construct(Composite HRESULT res; CompositeMonikerImpl *This; - *ppThis = This = HeapAlloc(GetProcessHeap(), 0, sizeof(*This)); + This = HeapAlloc(GetProcessHeap(), 0, sizeof(*This)); if (!This) return E_OUTOFMEMORY; @@ -1784,7 +1784,7 @@ CompositeMonikerImpl_Construct(Composite This->lpvtbl1 = &VT_CompositeMonikerImpl; This->lpvtbl2 = &VT_ROTDataImpl; This->lpvtblMarshal= &VT_MarshalImpl; - This->ref = 0; + This->ref = 1; This->tabSize=BLOCK_TAB_SIZE; This->tabLastIndex=0; @@ -1794,7 +1794,10 @@ CompositeMonikerImpl_Construct(Composite return E_OUTOFMEMORY; if (!pmkFirst && !pmkRest) + { + *ppMoniker = (IMoniker *)This; return S_OK; + } IMoniker_IsSystemMoniker(pmkFirst,&mkSys); @@ -1913,6 +1916,16 @@ CompositeMonikerImpl_Construct(Composite IEnumMoniker_Release(enumMoniker); } + /* only one moniker, then just return it */ + if (This->tabLastIndex == 1) + { + *ppMoniker = This->tabMoniker[0]; + IMoniker_AddRef(*ppMoniker); + IMoniker_Release((IMoniker *)This); + } + else + *ppMoniker = (IMoniker *)This; + return S_OK; } @@ -1923,7 +1936,7 @@ HRESULT WINAPI CreateGenericComposite(LPMONIKER pmkFirst, LPMONIKER pmkRest, LPMONIKER* ppmkComposite) { - CompositeMonikerImpl* newCompositeMoniker = 0; + IMoniker* moniker = 0; HRESULT hr = S_OK; TRACE("(%p,%p,%p)\n",pmkFirst,pmkRest,ppmkComposite); @@ -1945,12 +1958,15 @@ CreateGenericComposite(LPMONIKER pmkFirs else if (pmkFirst==NULL && pmkRest==NULL) return S_OK; - hr = CompositeMonikerImpl_Construct(&newCompositeMoniker,pmkFirst,pmkRest); + hr = CompositeMonikerImpl_Construct(&moniker,pmkFirst,pmkRest); if (FAILED(hr)) return hr; - return IMoniker_QueryInterface((IMoniker*)newCompositeMoniker,&IID_IMoniker,(void**)ppmkComposite); + hr = IMoniker_QueryInterface(moniker,&IID_IMoniker,(void**)ppmkComposite); + IMoniker_Release(moniker); + + return hr; } /****************************************************************************** @@ -1989,7 +2005,7 @@ static ULONG WINAPI CompositeMonikerCF_R static HRESULT WINAPI CompositeMonikerCF_CreateInstance(LPCLASSFACTORY iface, LPUNKNOWN pUnk, REFIID riid, LPVOID *ppv) { - CompositeMonikerImpl* newCompositeMoniker; + IMoniker* pMoniker; HRESULT hr; TRACE("(%p, %s, %p)\n", pUnk, debugstr_guid(riid), ppv); @@ -1999,12 +2015,13 @@ static HRESULT WINAPI CompositeMonikerCF if (pUnk) return CLASS_E_NOAGGREGATION; - hr = CompositeMonikerImpl_Construct(&newCompositeMoniker, NULL, NULL); + hr = CompositeMonikerImpl_Construct(&pMoniker, NULL, NULL); if (SUCCEEDED(hr)) - hr = CompositeMonikerImpl_QueryInterface((IMoniker*)newCompositeMoniker, riid, ppv); - if (FAILED(hr)) - HeapFree(GetProcessHeap(),0,newCompositeMoniker); + { + hr = IMoniker_QueryInterface(pMoniker, riid, ppv); + IMoniker_Release(pMoniker); + } return hr; }
1
0
0
0
Robert Shearman : ole32: Fix infinite recursion in CompositeMonikerImpl_GetTimeOfLastChange by handling pmkToLeft properly .
by Alexandre Julliard
08 May '06
08 May '06
Module: wine Branch: refs/heads/master Commit: 96646d532ff072414afd8315b43ffcd3e81eb358 URL:
http://source.winehq.org/git/?p=wine.git;a=commit;h=96646d532ff072414afd831…
Author: Robert Shearman <rob(a)codeweavers.com> Date: Mon May 8 15:54:43 2006 +0100 ole32: Fix infinite recursion in CompositeMonikerImpl_GetTimeOfLastChange by handling pmkToLeft properly. Fix a typo where CompositeMonikerImpl_GetTimeOfLastChange was called instead of the virtual implementation of it on mostRightMk. --- dlls/ole32/compositemoniker.c | 50 ++++++++++++++++++++++++----------------- 1 files changed, 29 insertions(+), 21 deletions(-) diff --git a/dlls/ole32/compositemoniker.c b/dlls/ole32/compositemoniker.c index fbeaa74..059eabf 100644 --- a/dlls/ole32/compositemoniker.c +++ b/dlls/ole32/compositemoniker.c @@ -727,9 +727,8 @@ static HRESULT WINAPI CompositeMonikerImpl_GetTimeOfLastChange(IMoniker* iface, IBindCtx* pbc, IMoniker* pmkToLeft, FILETIME* pCompositeTime) { - IRunningObjectTable* rot; HRESULT res; - IMoniker *tempMk,*antiMk,*mostRigthMk; + IMoniker *tempMk,*antiMk,*mostRigthMk,*leftMk; IEnumMoniker *enumMoniker; TRACE("(%p,%p,%p,%p)\n",iface,pbc,pmkToLeft,pCompositeTime); @@ -741,36 +740,45 @@ CompositeMonikerImpl_GetTimeOfLastChange /* retrieve the time of last change. If the object is not in the ROT, the method recursively calls */ /* IMoniker::GetTimeOfLastChange on the rightmost component of the composite, passing the remainder */ /* of the composite as the pmkToLeft parameter for that call. */ - if (pmkToLeft!=NULL){ - - res=CreateGenericComposite(pmkToLeft,iface,&tempMk); + if (pmkToLeft) + { + IRunningObjectTable* rot; - res=IBindCtx_GetRunningObjectTable(pbc,&rot); + res = IMoniker_ComposeWith(pmkToLeft, iface, FALSE, &leftMk); + res = IBindCtx_GetRunningObjectTable(pbc,&rot); if (FAILED(res)) + { + IMoniker_Release(leftMk); return res; + } - if (IRunningObjectTable_GetTimeOfLastChange(rot,tempMk,pCompositeTime)==S_OK) + if (IRunningObjectTable_GetTimeOfLastChange(rot,leftMk,pCompositeTime)==S_OK) + { + IMoniker_Release(leftMk); return res; - else + } + } + else + leftMk = iface; - IMoniker_Enum(iface,FALSE,&enumMoniker); - IEnumMoniker_Next(enumMoniker,1,&mostRigthMk,NULL); - IEnumMoniker_Release(enumMoniker); + IMoniker_Enum(iface, FALSE, &enumMoniker); + IEnumMoniker_Next(enumMoniker, 1, &mostRigthMk, NULL); + IEnumMoniker_Release(enumMoniker); - res=CreateAntiMoniker(&antiMk); - res=IMoniker_ComposeWith(iface,antiMk,0,&tempMk); - IMoniker_Release(antiMk); + res = CreateAntiMoniker(&antiMk); + res = IMoniker_ComposeWith(leftMk, antiMk, 0, &tempMk); + IMoniker_Release(antiMk); - res=CompositeMonikerImpl_GetTimeOfLastChange(mostRigthMk,pbc,tempMk,pCompositeTime); + res = IMoniker_GetTimeOfLastChange(mostRigthMk, pbc, tempMk, pCompositeTime); - IMoniker_Release(tempMk); - IMoniker_Release(mostRigthMk); + IMoniker_Release(tempMk); + IMoniker_Release(mostRigthMk); - return res; - } - else - return IMoniker_GetTimeOfLastChange(iface,pbc,NULL,pCompositeTime); + if (pmkToLeft) + IMoniker_Release(leftMk); + + return res; } /******************************************************************************
1
0
0
0
Robert Shearman : ole32: Fix infinite recursion in CompositeMonikerImpl_BindToStorage.
by Alexandre Julliard
08 May '06
08 May '06
Module: wine Branch: refs/heads/master Commit: 91afe981e0571e7f2b3904b701dde166ed7dd178 URL:
http://source.winehq.org/git/?p=wine.git;a=commit;h=91afe981e0571e7f2b3904b…
Author: Robert Shearman <rob(a)codeweavers.com> Date: Mon May 8 15:54:34 2006 +0100 ole32: Fix infinite recursion in CompositeMonikerImpl_BindToStorage. --- dlls/ole32/compositemoniker.c | 38 +++++++++++++++++++++++--------------- 1 files changed, 23 insertions(+), 15 deletions(-) diff --git a/dlls/ole32/compositemoniker.c b/dlls/ole32/compositemoniker.c index 439fbca..fbeaa74 100644 --- a/dlls/ole32/compositemoniker.c +++ b/dlls/ole32/compositemoniker.c @@ -402,7 +402,7 @@ CompositeMonikerImpl_BindToStorage(IMoni IMoniker* pmkToLeft, REFIID riid, VOID** ppvResult) { HRESULT res; - IMoniker *tempMk,*antiMk,*mostRigthMk; + IMoniker *tempMk,*antiMk,*mostRigthMk,*leftMk; IEnumMoniker *enumMoniker; TRACE("(%p,%p,%p,%p,%p)\n",iface,pbc,pmkToLeft,riid,ppvResult); @@ -412,26 +412,34 @@ CompositeMonikerImpl_BindToStorage(IMoni /* This method recursively calls BindToStorage on the rightmost component of the composite, */ /* passing the rest of the composite as the pmkToLeft parameter for that call. */ - if (pmkToLeft!=NULL){ + if (pmkToLeft) + { + res = IMoniker_ComposeWith(pmkToLeft, iface, FALSE, &leftMk); + if (FAILED(res)) return res; + } + else + leftMk = iface; - IMoniker_Enum(iface,FALSE,&enumMoniker); - IEnumMoniker_Next(enumMoniker,1,&mostRigthMk,NULL); - IEnumMoniker_Release(enumMoniker); + IMoniker_Enum(iface, FALSE, &enumMoniker); + IEnumMoniker_Next(enumMoniker, 1, &mostRigthMk, NULL); + IEnumMoniker_Release(enumMoniker); - res=CreateAntiMoniker(&antiMk); - res=IMoniker_ComposeWith(iface,antiMk,0,&tempMk); - IMoniker_Release(antiMk); + res = CreateAntiMoniker(&antiMk); + if (FAILED(res)) return res; + res = IMoniker_ComposeWith(leftMk, antiMk, 0, &tempMk); + if (FAILED(res)) return res; + IMoniker_Release(antiMk); - res=IMoniker_BindToStorage(mostRigthMk,pbc,tempMk,riid,ppvResult); + res = IMoniker_BindToStorage(mostRigthMk, pbc, tempMk, riid, ppvResult); - IMoniker_Release(tempMk); + IMoniker_Release(tempMk); - IMoniker_Release(mostRigthMk); + IMoniker_Release(mostRigthMk); - return res; - } - else - return IMoniker_BindToStorage(iface,pbc,NULL,riid,ppvResult); + if (pmkToLeft) + IMoniker_Release(leftMk); + + return res; } /******************************************************************************
1
0
0
0
Roderick Colenbrander : dinput8: DllGetClassObject support.
by Alexandre Julliard
08 May '06
08 May '06
Module: wine Branch: refs/heads/master Commit: 969ae13ebb2c177508b2c052c3f352c73ec5b3c6 URL:
http://source.winehq.org/git/?p=wine.git;a=commit;h=969ae13ebb2c177508b2c05…
Author: Roderick Colenbrander <thunderbird2k(a)gmx.net> Date: Sat May 6 20:45:11 2006 +0200 dinput8: DllGetClassObject support. --- dlls/dinput8/Makefile.in | 2 - dlls/dinput8/dinput8_main.c | 96 +++++++++++++++++++++++++++++++++++++++---- 2 files changed, 87 insertions(+), 11 deletions(-) diff --git a/dlls/dinput8/Makefile.in b/dlls/dinput8/Makefile.in index cd23d04..34135c1 100644 --- a/dlls/dinput8/Makefile.in +++ b/dlls/dinput8/Makefile.in @@ -5,7 +5,7 @@ VPATH = @srcdir@ MODULE = dinput8.dll IMPORTLIB = libdinput8.$(IMPLIBEXT) IMPORTS = dinput kernel32 -EXTRALIBS = -luuid +EXTRALIBS = -luuid -ldxguid C_SRCS = \ dinput8_main.c diff --git a/dlls/dinput8/dinput8_main.c b/dlls/dinput8/dinput8_main.c index 432d3f1..ad9b2eb 100644 --- a/dlls/dinput8/dinput8_main.c +++ b/dlls/dinput8/dinput8_main.c @@ -1,6 +1,7 @@ /* DirectInput 8 * * Copyright 2002 TransGaming Technologies Inc. + * Copyright 2006 Roderick Colenbrander * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public @@ -22,6 +23,8 @@ #include <assert.h> #include <stdarg.h> #include <string.h> +#define COBJMACROS + #include "wine/debug.h" #include "windef.h" #include "winbase.h" @@ -29,25 +32,93 @@ #include "winerror.h" #include "dinput.h" WINE_DEFAULT_DEBUG_CHANNEL(dinput); +static LONG dll_count; + +/* + * Dll lifetime tracking declaration + */ +static void LockModule(void) +{ + InterlockedIncrement(&dll_count); +} + +static void UnlockModule(void) +{ + InterlockedDecrement(&dll_count); +} /****************************************************************************** * DirectInput8Create (DINPUT8.@) */ -HRESULT WINAPI DirectInput8Create( - HINSTANCE hinst, DWORD dwVersion, REFIID riid, LPVOID *ppDI, - LPUNKNOWN punkOuter -) { - return DirectInputCreateEx(hinst, dwVersion, riid, ppDI, punkOuter); +HRESULT WINAPI DirectInput8Create(HINSTANCE hinst, DWORD dwVersion, REFIID riid, LPVOID *ppDI, LPUNKNOWN punkOuter) { + /* TODO: Create the interface using CoCreateInstance as that's what windows does too and check if the version number >= 0x800 */ + return DirectInputCreateEx(hinst, dwVersion, riid, ppDI, punkOuter); +} + +/******************************************************************************* + * DirectInput8 ClassFactory + */ +typedef struct +{ + /* IUnknown fields */ + const IClassFactoryVtbl *lpVtbl; +} IClassFactoryImpl; + +static HRESULT WINAPI DI8CF_QueryInterface(LPCLASSFACTORY iface,REFIID riid,LPVOID *ppobj) { + IClassFactoryImpl *This = (IClassFactoryImpl *)iface; + FIXME("%p %s %p\n",This,debugstr_guid(riid),ppobj); + return E_NOINTERFACE; +} + +static ULONG WINAPI DI8CF_AddRef(LPCLASSFACTORY iface) { + LockModule(); + return 2; } +static ULONG WINAPI DI8CF_Release(LPCLASSFACTORY iface) { + UnlockModule(); + return 1; +} + +static HRESULT WINAPI DI8CF_CreateInstance(LPCLASSFACTORY iface,LPUNKNOWN pOuter,REFIID riid,LPVOID *ppobj) { + IClassFactoryImpl *This = (IClassFactoryImpl *)iface; + + TRACE("(%p)->(%p,%s,%p)\n",This,pOuter,debugstr_guid(riid),ppobj); + if( IsEqualGUID( &IID_IDirectInput8A, riid ) || IsEqualGUID( &IID_IDirectInput8W, riid ) ) { + return DirectInput8Create(0, DIRECTINPUT_VERSION, riid, ppobj, pOuter); + } + + ERR("(%p,%p,%s,%p) Interface not found!\n",This,pOuter,debugstr_guid(riid),ppobj); + return E_NOINTERFACE; +} + +static HRESULT WINAPI DI8CF_LockServer(LPCLASSFACTORY iface,BOOL dolock) { + TRACE("(%p)->(%d)\n", iface, dolock); + + if(dolock) + LockModule(); + else + UnlockModule(); + + return S_OK; +} + +static const IClassFactoryVtbl DI8CF_Vtbl = { + DI8CF_QueryInterface, + DI8CF_AddRef, + DI8CF_Release, + DI8CF_CreateInstance, + DI8CF_LockServer +}; +static IClassFactoryImpl DINPUT8_CF = { &DI8CF_Vtbl }; + + /*********************************************************************** * DllCanUnloadNow (DINPUT8.@) */ HRESULT WINAPI DllCanUnloadNow(void) { - FIXME("(void): stub\n"); - - return S_FALSE; + return dll_count == 0 ? S_OK : S_FALSE; } /*********************************************************************** @@ -55,9 +126,14 @@ HRESULT WINAPI DllCanUnloadNow(void) */ HRESULT WINAPI DllGetClassObject(REFCLSID rclsid, REFIID riid, LPVOID *ppv) { - FIXME("(%p, %p, %p): stub\n", debugstr_guid(rclsid), - debugstr_guid(riid), ppv); + TRACE("(%s,%s,%p)\n", debugstr_guid(rclsid), debugstr_guid(riid), ppv); + if ( IsEqualCLSID( &IID_IClassFactory, riid ) ) { + *ppv = (LPVOID)&DINPUT8_CF; + IClassFactory_AddRef((IClassFactory*)*ppv); + return S_OK; + } + FIXME("(%s,%s,%p): no interface found.\n", debugstr_guid(rclsid), debugstr_guid(riid), ppv); return CLASS_E_CLASSNOTAVAILABLE; }
1
0
0
0
Hans Leidekker : dnsapi: Implement and test DnsRecordSetCompare.
by Alexandre Julliard
08 May '06
08 May '06
Module: wine Branch: refs/heads/master Commit: efe9cedab282c6028ab970a74fe4e1580c22fa94 URL:
http://source.winehq.org/git/?p=wine.git;a=commit;h=efe9cedab282c6028ab970a…
Author: Hans Leidekker <hans(a)it.vu.nl> Date: Mon May 8 16:39:23 2006 +0200 dnsapi: Implement and test DnsRecordSetCompare. --- dlls/dnsapi/dnsapi.spec | 2 + dlls/dnsapi/record.c | 82 ++++++++++++++++++++++++++++++++++++++++++++ dlls/dnsapi/tests/record.c | 75 +++++++++++++++++++++++++++++++++------- 3 files changed, 144 insertions(+), 15 deletions(-) diff --git a/dlls/dnsapi/dnsapi.spec b/dlls/dnsapi/dnsapi.spec index a2502b8..e6a42b0 100644 --- a/dlls/dnsapi/dnsapi.spec +++ b/dlls/dnsapi/dnsapi.spec @@ -86,7 +86,7 @@ @ stdcall DnsRecordCompare(ptr ptr) @ stdcall DnsRecordCopyEx(ptr long long) @ stdcall DnsRecordListFree(ptr long) -@ stub DnsRecordSetCompare +@ stdcall DnsRecordSetCompare(ptr ptr ptr ptr) @ stdcall DnsRecordSetCopyEx(ptr long long) @ stub DnsRecordSetDetach @ stub DnsRecordStringForType diff --git a/dlls/dnsapi/record.c b/dlls/dnsapi/record.c index 08c3cd8..8b68314 100644 --- a/dlls/dnsapi/record.c +++ b/dlls/dnsapi/record.c @@ -640,6 +640,88 @@ void WINAPI DnsRecordListFree( PDNS_RECO } /****************************************************************************** + * DnsRecordSetCompare [DNSAPI.@] + * + */ +BOOL WINAPI DnsRecordSetCompare( PDNS_RECORD set1, PDNS_RECORD set2, + PDNS_RECORD *diff1, PDNS_RECORD *diff2 ) +{ + BOOL ret = TRUE; + DNS_RECORD *r, *s1, *s2, *t, *u; + DNS_RRSET rr1, rr2; + + TRACE( "(%p,%p,%p,%p)\n", set1, set2, diff1, diff2 ); + + if (!set1 && !set2) return FALSE; + + if (diff1) *diff1 = NULL; + if (diff2) *diff2 = NULL; + + if (set1 && !set2) + { + if (diff1) *diff1 = DnsRecordSetCopyEx( set1, 0, set1->Flags.S.CharSet ); + return FALSE; + } + if (!set1 && set2) + { + if (diff2) *diff2 = DnsRecordSetCopyEx( set2, 0, set2->Flags.S.CharSet ); + return FALSE; + } + + DNS_RRSET_INIT( rr1 ); + DNS_RRSET_INIT( rr2 ); + + for (r = s1 = set1; (s1 = r); r = r->pNext) + { + for (t = s2 = set2; (s2 = t); t = t->pNext) + { + u = DnsRecordCopyEx( r, r->Flags.S.CharSet, t->Flags.S.CharSet ); + if (!u) goto error; + + if (!DnsRecordCompare( t, u )) + { + DNS_RRSET_ADD( rr1, u ); + ret = FALSE; + } + else dns_free( u ); + } + } + + for (t = s2 = set2; (s2 = t); t = t->pNext) + { + for (r = s1 = set1; (s1 = r); r = r->pNext) + { + u = DnsRecordCopyEx( t, t->Flags.S.CharSet, r->Flags.S.CharSet ); + if (!u) goto error; + + if (!DnsRecordCompare( r, u )) + { + DNS_RRSET_ADD( rr2, u ); + ret = FALSE; + } + else dns_free( u ); + } + } + + DNS_RRSET_TERMINATE( rr1 ); + DNS_RRSET_TERMINATE( rr2 ); + + if (diff1) *diff1 = rr1.pFirstRR; + if (diff2) *diff2 = rr2.pFirstRR; + + return ret; + +error: + DNS_RRSET_TERMINATE( rr1 ); + DNS_RRSET_TERMINATE( rr2 ); + + DnsRecordListFree( rr1.pFirstRR, DnsFreeRecordList ); + DnsRecordListFree( rr2.pFirstRR, DnsFreeRecordList ); + + return FALSE; +} + +/****************************************************************************** * DnsRecordSetCopyEx [DNSAPI.@] * */ diff --git a/dlls/dnsapi/tests/record.c b/dlls/dnsapi/tests/record.c index 307ecaa..6173e57 100644 --- a/dlls/dnsapi/tests/record.c +++ b/dlls/dnsapi/tests/record.c @@ -19,6 +19,7 @@ */ #include <stdarg.h> +#include <stdio.h> #include "windef.h" #include "winbase.h" @@ -30,6 +31,7 @@ #include "wine/test.h" static HMODULE dnsapi; static BOOL (WINAPI *pDnsRecordCompare)(PDNS_RECORD,PDNS_RECORD); +static BOOL (WINAPI *pDnsRecordSetCompare)(PDNS_RECORD,PDNS_RECORD,PDNS_RECORD*,PDNS_RECORD*); #define GETFUNCPTR(func) p##func = (void *)GetProcAddress( dnsapi, #func ); \ if (!p##func) return FALSE; @@ -37,25 +39,18 @@ #define GETFUNCPTR(func) p##func = (void static BOOL init_function_ptrs( void ) { GETFUNCPTR( DnsRecordCompare ) + GETFUNCPTR( DnsRecordSetCompare ) return TRUE; } +static char name1[] = "localhost"; +static char name2[] = "LOCALHOST"; + +static DNS_RECORDA r1 = { NULL, name1, DNS_TYPE_A, sizeof(DNS_A_DATA), { 0 }, 1200, 0, { { 0xffffffff } } }; +static DNS_RECORDA r2 = { NULL, name1, DNS_TYPE_A, sizeof(DNS_A_DATA), { 0 }, 1200, 0, { { 0xffffffff } } }; + static void test_DnsRecordCompare( void ) { - char name1[] = "localhost"; - char name2[] = "LOCALHOST"; - static DNS_RECORDA r1, r2; - - r1.pName = name1; - r1.wType = DNS_TYPE_A; - r1.wDataLength = sizeof(DNS_A_DATA); - r1.Data.A.IpAddress = 0xffffffff; - - r2.pName = name1; - r2.wType = DNS_TYPE_A; - r2.wDataLength = sizeof(DNS_A_DATA); - r2.Data.A.IpAddress = 0xffffffff; - ok( pDnsRecordCompare( (PDNS_RECORD)&r1, (PDNS_RECORD)&r1 ) == TRUE, "failed unexpectedly\n" ); r2.pName = name2; @@ -74,6 +69,57 @@ static void test_DnsRecordCompare( void ok( pDnsRecordCompare( (PDNS_RECORD)&r1, (PDNS_RECORD)&r2 ) == FALSE, "succeeded unexpectedly\n" ); } +static void test_DnsRecordSetCompare( void ) +{ + DNS_RECORD *diff1; + DNS_RECORD *diff2; + DNS_RRSET rr1, rr2; + + r1.Flags.DW = 0x2019; + r2.Flags.DW = 0x2019; + r2.Data.A.IpAddress = 0xffffffff; + + DNS_RRSET_INIT( rr1 ); + DNS_RRSET_INIT( rr2 ); + + DNS_RRSET_ADD( rr1, &r1 ); + DNS_RRSET_ADD( rr2, &r2 ); + + DNS_RRSET_TERMINATE( rr1 ); + DNS_RRSET_TERMINATE( rr2 ); + + ok( pDnsRecordSetCompare( NULL, NULL, NULL, NULL ) == FALSE, "succeeded unexpectedly\n" ); + ok( pDnsRecordSetCompare( rr1.pFirstRR, NULL, NULL, NULL ) == FALSE, "succeeded unexpectedly\n" ); + ok( pDnsRecordSetCompare( NULL, rr2.pFirstRR, NULL, NULL ) == FALSE, "succeeded unexpectedly\n" ); + + diff1 = NULL; + diff2 = NULL; + + ok( pDnsRecordSetCompare( NULL, NULL, &diff1, &diff2 ) == FALSE, "succeeded unexpectedly\n" ); + ok( diff1 == NULL && diff2 == NULL, "unexpected result: %p, %p\n", diff1, diff2 ); + + ok( pDnsRecordSetCompare( rr1.pFirstRR, NULL, &diff1, &diff2 ) == FALSE, "succeeded unexpectedly\n" ); + ok( diff1 != NULL && diff2 == NULL, "unexpected result: %p, %p\n", diff1, diff2 ); + + ok( pDnsRecordSetCompare( NULL, rr2.pFirstRR, &diff1, &diff2 ) == FALSE, "succeeded unexpectedly\n" ); + ok( diff1 == NULL && diff2 != NULL, "unexpected result: %p, %p\n", diff1, diff2 ); + + ok( pDnsRecordSetCompare( rr1.pFirstRR, rr2.pFirstRR, NULL, &diff2 ) == TRUE, "failed unexpectedly\n" ); + ok( diff2 == NULL, "unexpected result: %p\n", diff2 ); + + ok( pDnsRecordSetCompare( rr1.pFirstRR, rr2.pFirstRR, &diff1, NULL ) == TRUE, "failed unexpectedly\n" ); + ok( diff1 == NULL, "unexpected result: %p\n", diff1 ); + + ok( pDnsRecordSetCompare( rr1.pFirstRR, rr2.pFirstRR, &diff1, &diff2 ) == TRUE, "failed unexpectedly\n" ); + ok( diff1 == NULL && diff2 == NULL, "unexpected result: %p, %p\n", diff1, diff2 ); + + r2.Data.A.IpAddress = 0; + + ok( pDnsRecordSetCompare( rr1.pFirstRR, rr2.pFirstRR, NULL, &diff2 ) == FALSE, "succeeded unexpectedly\n" ); + ok( pDnsRecordSetCompare( rr1.pFirstRR, rr2.pFirstRR, &diff1, NULL ) == FALSE, "succeeded unexpectedly\n" ); + ok( pDnsRecordSetCompare( rr1.pFirstRR, rr2.pFirstRR, &diff1, &diff2 ) == FALSE, "succeeded unexpectedly\n" ); +} + START_TEST(record) { dnsapi = LoadLibraryA( "dnsapi.dll" ); @@ -86,6 +132,7 @@ START_TEST(record) } test_DnsRecordCompare(); + test_DnsRecordSetCompare(); FreeLibrary( dnsapi ); }
1
0
0
0
← Newer
1
...
65
66
67
68
69
70
71
...
87
Older →
Jump to page:
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
Results per page:
10
25
50
100
200