diff options
author | Daniel Silverstone <dsilvers@netsurf-browser.org> | 2012-03-25 13:21:11 +0000 |
---|---|---|
committer | Daniel Silverstone <dsilvers@netsurf-browser.org> | 2012-03-25 13:21:11 +0000 |
commit | 2da3eed0601f392a810753b152055f6ddea18111 (patch) | |
tree | 6f09c26f9e270ea793a3486b9863a4702b3b8d52 /src/core/node.c | |
parent | 9a2c16fcbc08558f993d11b2bcb23d4b98f696a4 (diff) | |
download | libdom-2da3eed0601f392a810753b152055f6ddea18111.tar.gz libdom-2da3eed0601f392a810753b152055f6ddea18111.tar.bz2 |
Squash scan-build issues
svn path=/trunk/libdom/; revision=13686
Diffstat (limited to 'src/core/node.c')
-rw-r--r-- | src/core/node.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/src/core/node.c b/src/core/node.c index 1bd7630..4509f10 100644 --- a/src/core/node.c +++ b/src/core/node.c @@ -720,7 +720,9 @@ dom_exception _dom_node_insert_before(dom_node_internal *node, { dom_exception err; dom_node_internal *n; - + + assert(node != NULL); + /* Ensure that new_child and node are owned by the same document */ if ((new_child->type == DOM_DOCUMENT_TYPE_NODE && new_child->owner != NULL && @@ -736,7 +738,7 @@ dom_exception _dom_node_insert_before(dom_node_internal *node, /* Ensure that ref_child (if any) is a child of node */ if (ref_child != NULL && ref_child->parent != node) return DOM_NOT_FOUND_ERR; - + /* Ensure that new_child is not an ancestor of node, nor node itself */ for (n = node; n != NULL; n = n->parent) { if (n == new_child) @@ -2428,6 +2430,10 @@ dom_exception _dom_node_dispatch_event(dom_event_target *et, evt->current = et; err = _dom_event_target_dispatch(et, &((dom_node_internal *) et)->eti, evt, DOM_AT_TARGET, success); + if (err != DOM_NO_ERR) { + ret = err; + goto cleanup; + } if (evt->stop_now == true || evt->stop == true) goto cleanup; |