Sven Baars : scrobj: Add a missing return value check (Coverity).

Alexandre Julliard julliard at winehq.org
Mon Oct 21 15:28:38 CDT 2019


Module: wine
Branch: master
Commit: 43eb22c57f3a6227f23178205139c775199db2c8
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=43eb22c57f3a6227f23178205139c775199db2c8

Author: Sven Baars <sven.wine at gmail.com>
Date:   Sat Oct 19 21:20:54 2019 +0200

scrobj: Add a missing return value check (Coverity).

Signed-off-by: Sven Baars <sven.wine at gmail.com>
Signed-off-by: Jacek Caban <jacek at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/scrobj/scrobj.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/dlls/scrobj/scrobj.c b/dlls/scrobj/scrobj.c
index 1d37864039..427e4e5439 100644
--- a/dlls/scrobj/scrobj.c
+++ b/dlls/scrobj/scrobj.c
@@ -1695,6 +1695,8 @@ static HRESULT parse_scriptlet_file(struct scriptlet_factory *factory, const WCH
     hres = next_xml_node(factory, &node_type);
     if (hres == S_OK && node_type == XmlNodeType_XmlDeclaration)
         hres = next_xml_node(factory, &node_type);
+    if (FAILED(hres))
+        return hres;
 
     if (node_type != XmlNodeType_Element || !is_xml_name(factory, L"component"))
     {




More information about the wine-cvs mailing list