From 46d133e90663100c1747a7b231e8827d50e83f25 Mon Sep 17 00:00:00 2001 From: Witold Filipczyk Date: Wed, 30 Oct 2024 17:24:54 +0100 Subject: [PATCH] [quickjs] Fix in fragment.removeChild Allow other types of Nodes, not only DocumentFragment. --- src/js/quickjs/fragment.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/js/quickjs/fragment.c b/src/js/quickjs/fragment.c index d72d1a284..accf7a04c 100644 --- a/src/js/quickjs/fragment.c +++ b/src/js/quickjs/fragment.c @@ -1508,9 +1508,13 @@ js_fragment_removeChild(JSContext *ctx, JSValueConst this_val, int argc, JSValue } //dom_node_ref(el); JSValue node = argv[0]; - dom_node *el2 = (dom_node *)(js_getopaque_fragment(node, js_fragment_class_id)); + dom_node *el2 = (dom_node *)js_getopaque_any(node); + + if (!el2) { + return JS_NULL; + } dom_exception exc; - dom_node *spare; + dom_node *spare = NULL; exc = dom_node_remove_child(el, el2, &spare); if (exc == DOM_NO_ERR && spare) {