commit da70148afb107c4e47e9b2e7f33c39118e6a92df
author Huw D M Davies <hdavies@codeweavers.com> Mon Jan 14 18:35:44 2002 +0000
committer Alexandre Julliard <julliard@winehq.org> Mon Jan 14 18:35:44 2002 +0000
tree 57aacdfc7d8133c7ff0031bbe9af57220a8cb13e
parent a8322d0b022f1354f08946bf0c32096b1573810c

IShellFolder_BindToObject should respect riid.

dlls/shell32/shlfolder.c

diff --git a/dlls/shell32/shlfolder.c b/dlls/shell32/shlfolder.c
index 46aa71e..b5e8f13 100644
--- a/dlls/shell32/shlfolder.c
+++ b/dlls/shell32/shlfolder.c
@@ -736,6 +736,7 @@
 	IShellFolder	*pShellFolder, *pSubFolder;
 	IPersistFolder 	*pPersistFolder;
 	LPITEMIDLIST	absPidl;
+	HRESULT         hr;
 	
 	TRACE("(%p)->(pidl=%p,%p,\n\tIID:\t%s,%p)\n",This,pidl,pbcReserved,debugstr_guid(riid),ppvOut);
 
@@ -773,18 +774,28 @@
 	
 	if (_ILIsPidlSimple(pidl))
 	{
-	  *ppvOut = pShellFolder;
+	  if(IsEqualIID(riid, &IID_IShellFolder))
+	  {
+	    *ppvOut = pShellFolder;
+	    hr = S_OK;
+	  }
+	  else
+	  {
+	    hr = IShellFolder_QueryInterface(pShellFolder, riid, ppvOut);
+	    IShellFolder_Release(pShellFolder);
+	  }
 	}
 	else
 	{
-	  IShellFolder_BindToObject(pShellFolder, ILGetNext(pidl), NULL, &IID_IShellFolder, (LPVOID)&pSubFolder);
+	  hr = IShellFolder_BindToObject(pShellFolder, ILGetNext(pidl), NULL,
+					 riid, (LPVOID)&pSubFolder);
 	  IShellFolder_Release(pShellFolder);
 	  *ppvOut = pSubFolder;
 	}
 
-	TRACE("-- (%p) returning (%p)\n",This, *ppvOut);
+	TRACE("-- (%p) returning (%p) %08lx\n",This, *ppvOut, hr);
 
-	return S_OK;
+	return hr;
 }
 
 /**************************************************************************
@@ -1764,7 +1775,8 @@
 	_ICOM_THIS_From_IShellFolder2(IGenericSFImpl, iface)
 	GUID		const * clsid;
 	IShellFolder	*pShellFolder, *pSubFolder;
-	
+	HRESULT         hr;
+
 	TRACE("(%p)->(pidl=%p,%p,\n\tIID:\t%s,%p)\n",
               This,pidl,pbcReserved,debugstr_guid(riid),ppvOut);
 
@@ -1810,17 +1822,18 @@
 	if (_ILIsPidlSimple(pidl))	/* no sub folders */
 	{
 	  *ppvOut = pShellFolder;
+	  hr = S_OK;
 	}
 	else				/* go deeper */
 	{
-	  IShellFolder_BindToObject(pShellFolder, ILGetNext(pidl), NULL, riid, (LPVOID)&pSubFolder);
+	  hr = IShellFolder_BindToObject(pShellFolder, ILGetNext(pidl), NULL, riid, (LPVOID)&pSubFolder);
 	  IShellFolder_Release(pShellFolder);
 	  *ppvOut = pSubFolder;
 	}
 
-	TRACE("-- (%p) returning (%p)\n",This, *ppvOut);
+	TRACE("-- (%p) returning (%p) %08lx\n",This, *ppvOut, hr);
 
-	return S_OK;
+	return hr;
 }
 
 /**************************************************************************
@@ -2206,7 +2219,8 @@
 	GUID		const * clsid;
 	IShellFolder	*pShellFolder, *pSubFolder;
 	LPITEMIDLIST	pidltemp;
-	
+	HRESULT         hr;
+
 	TRACE("(%p)->(pidl=%p,%p,\n\tIID:\t%s,%p)\n",
               This,pidl,pbcReserved,debugstr_guid(riid),ppvOut);
 
@@ -2233,17 +2247,19 @@
 	if (_ILIsPidlSimple(pidl))	/* no sub folders */
 	{
 	  *ppvOut = pShellFolder;
+	  hr = S_OK;
 	}
 	else				/* go deeper */
 	{
-	  IShellFolder_BindToObject(pShellFolder, ILGetNext(pidl), NULL, &IID_IShellFolder, (LPVOID)&pSubFolder);
+	  hr = IShellFolder_BindToObject(pShellFolder, ILGetNext(pidl), NULL,
+					 riid, (LPVOID)&pSubFolder);
 	  IShellFolder_Release(pShellFolder);
 	  *ppvOut = pSubFolder;
 	}
 
-	TRACE("-- (%p) returning (%p)\n",This, *ppvOut);
+	TRACE("-- (%p) returning (%p) %08lx\n",This, *ppvOut, hr);
 
-	return S_OK;
+	return hr;
 }
 
 /**************************************************************************