mshtml: Handle failure and cleanup properly when enumerating nodes.

Signed-off-by: Gabriel Ivăncescu <gabrielopcode@gmail.com>

mshtml: Handle failure and cleanup properly when enumerating nodes.
7d8fd7df · Gabriel Ivăncescu · Alexandre Julliard · 1bce7be3 · 7d8fd7df
Commit 7d8fd7df authored Jun 21, 2023 by Gabriel Ivăncescu Committed by Alexandre Julliard Jun 22, 2023
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 2 deletions

htmlnode.c dlls/mshtml/htmlnode.c +3 -2

No files found.
--- a/dlls/mshtml/htmlnode.c
+++ b/dlls/mshtml/htmlnode.c
@@ -135,8 +135,9 @@ static HRESULT WINAPI HTMLDOMChildrenCollectionEnum_Next(IEnumVARIANT *iface, UL
        hres = get_node(nsnode, TRUE, &node);
        nsIDOMNode_Release(nsnode);
        if(FAILED(hres)) {
-            ERR("get_node failed: %08lx\n", hres);
-            break;
+            while(fetched--)
+                VariantClear(rgVar+fetched);
+            return hres;
        }

        V_VT(rgVar+fetched) = VT_DISPATCH;