fix: Changed the behavior of the nesting counter ever so slightly

cure53 · Apr 23, 2024 · c725ce0 · c725ce0
1 parent c5369f2
commit c725ce0
Show file tree

Hide file tree

Showing 9 changed files with 29 additions and 13 deletions.
diff --git a/dist/purify.cjs.js b/dist/purify.cjs.js
diff --git a/dist/purify.cjs.js.map b/dist/purify.cjs.js.map
diff --git a/dist/purify.es.mjs b/dist/purify.es.mjs
@@ -1379,8 +1379,12 @@ function createDOMPurify() {
       }
 
       /* Count the nesting depth of an element */
-      if (currentNode.hasChildNodes()) {
-        depth++;
+      if (currentNode.nodeType === 1 && currentNode.hasChildNodes()) {
+        if (currentNode.hasChildNodes()) {
+          depth++;
+        } else {
+          depth--;
+        }
       }
 
       /* Remove an element if nested too deeply to avoid mXSS */

diff --git a/dist/purify.es.mjs.map b/dist/purify.es.mjs.map
diff --git a/dist/purify.js b/dist/purify.js
diff --git a/dist/purify.js.map b/dist/purify.js.map
diff --git a/dist/purify.min.js b/dist/purify.min.js
diff --git a/dist/purify.min.js.map b/dist/purify.min.js.map
diff --git a/src/purify.js b/src/purify.js
@@ -1502,8 +1502,12 @@ function createDOMPurify(window = getGlobal()) {
       }
 
       /* Count the nesting depth of an element */
-      if (currentNode.hasChildNodes()) {
-        depth++;
+      if (currentNode.nodeType === 1 && currentNode.hasChildNodes()) {
+        if (currentNode.hasChildNodes()) {
+          depth++;
+        } else {
+          depth--;
+        }
       }
 
       /* Remove an element if nested too deeply to avoid mXSS */