Jacek Caban : mshtml: Fixed error handling of IBindCtx_GetObjectParam ( coverity).

Alexandre Julliard julliard at winehq.org
Fri Oct 19 13:33:54 CDT 2012


Module: wine
Branch: master
Commit: 3b6615686b1bb349f796b9163389cf611f9e33b8
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=3b6615686b1bb349f796b9163389cf611f9e33b8

Author: Jacek Caban <jacek at codeweavers.com>
Date:   Fri Oct 19 11:59:57 2012 +0200

mshtml: Fixed error handling of IBindCtx_GetObjectParam (coverity).

---

 dlls/mshtml/persist.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/dlls/mshtml/persist.c b/dlls/mshtml/persist.c
index b68f79a..d92989e 100644
--- a/dlls/mshtml/persist.c
+++ b/dlls/mshtml/persist.c
@@ -520,8 +520,8 @@ static HRESULT WINAPI PersistMoniker_Load(IPersistMoniker *iface, BOOL fFullyAva
          * "_EnumFORMATETC_"
          */
 
-        IBindCtx_GetObjectParam(pibc, (LPOLESTR)SZ_HTML_CLIENTSITE_OBJECTPARAM, &unk);
-        if(unk) {
+        hres = IBindCtx_GetObjectParam(pibc, (LPOLESTR)SZ_HTML_CLIENTSITE_OBJECTPARAM, &unk);
+        if(SUCCEEDED(hres) && unk) {
             IOleClientSite *client = NULL;
 
             hres = IUnknown_QueryInterface(unk, &IID_IOleClientSite, (void**)&client);




More information about the wine-cvs mailing list