[PATCH] shell32: Return as soon as requested CLSID is located (Coverity)
Nikolay Sivov
nsivov at codeweavers.com
Wed Nov 11 05:01:37 CST 2015
Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
---
dlls/shell32/shellole.c | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/dlls/shell32/shellole.c b/dlls/shell32/shellole.c
index 637a101..f034951 100644
--- a/dlls/shell32/shellole.c
+++ b/dlls/shell32/shellole.c
@@ -59,7 +59,7 @@ static IClassFactory * IDefClF_fnConstructor(LPFNCREATEINSTANCE lpfnCI, PLONG pc
/* this table contains all CLSIDs of shell32 objects */
static const struct {
- REFIID riid;
+ REFIID clsid;
LPFNCREATEINSTANCE lpfnCI;
} InterfaceTable[] = {
@@ -204,8 +204,8 @@ end:
*/
HRESULT WINAPI DllGetClassObject(REFCLSID rclsid, REFIID iid, LPVOID *ppv)
{
- HRESULT hres = E_OUTOFMEMORY;
IClassFactory * pcf = NULL;
+ HRESULT hres;
int i;
TRACE("CLSID:%s,IID:%s\n",shdebugstr_guid(rclsid),shdebugstr_guid(iid));
@@ -214,10 +214,11 @@ HRESULT WINAPI DllGetClassObject(REFCLSID rclsid, REFIID iid, LPVOID *ppv)
*ppv = NULL;
/* search our internal interface table */
- for(i=0;InterfaceTable[i].riid;i++) {
- if(IsEqualIID(InterfaceTable[i].riid, rclsid)) {
+ for(i=0;InterfaceTable[i].clsid;i++) {
+ if(IsEqualIID(InterfaceTable[i].clsid, rclsid)) {
TRACE("index[%u]\n", i);
pcf = IDefClF_fnConstructor(InterfaceTable[i].lpfnCI, NULL, NULL);
+ break;
}
}
--
2.6.2
More information about the wine-patches
mailing list