remove recursion to make the fuzzer happy

HtmlUnit · Jan 19, 2023 · 4ee1862 · 4ee1862
1 parent 71f7623
commit 4ee1862
Showing 1 changed file with 24 additions and 21 deletions.
diff --git a/src/main/java/com/gargoylesoftware/htmlunit/html/DomNode.java b/src/main/java/com/gargoylesoftware/htmlunit/html/DomNode.java
@@ -1704,14 +1704,15 @@ public void removeDomChangeListener(final DomChangeListener listener) {
      * @param event the DomChangeEvent to be propagated
      */
     protected void fireNodeAdded(final DomChangeEvent event) {
-        final List<DomChangeListener> listeners = safeGetDomListeners();
-        if (listeners != null) {
-            for (final DomChangeListener listener : listeners) {
-                listener.nodeAdded(event);
+        DomNode toInform = this;
+        while (toInform != null) {
+            final List<DomChangeListener> listeners = toInform.safeGetDomListeners();
+            if (listeners != null) {
+                for (final DomChangeListener listener : listeners) {
+                    listener.nodeAdded(event);
+                }
             }
-        }
-        if (parent_ != null) {
-            parent_.fireNodeAdded(event);
+            toInform = toInform.getParentNode();
         }
     }
 
@@ -1760,14 +1761,15 @@ public void removeCharacterDataChangeListener(final CharacterDataChangeListener
      * @param event the CharacterDataChangeEvent to be propagated
      */
     protected void fireCharacterDataChanged(final CharacterDataChangeEvent event) {
-        final List<CharacterDataChangeListener> listeners = safeGetCharacterDataListeners();
-        if (listeners != null) {
-            for (final CharacterDataChangeListener listener : listeners) {
-                listener.characterDataChanged(event);
+        DomNode toInform = this;
+        while (toInform != null) {
+            final List<CharacterDataChangeListener> listeners = safeGetCharacterDataListeners();
+            if (listeners != null) {
+                for (final CharacterDataChangeListener listener : listeners) {
+                    listener.characterDataChanged(event);
+                }
             }
-        }
-        if (parent_ != null) {
-            parent_.fireCharacterDataChanged(event);
+            toInform = toInform.getParentNode();
         }
     }
 
@@ -1780,14 +1782,15 @@ protected void fireCharacterDataChanged(final CharacterDataChangeEvent event) {
      * @param event the DomChangeEvent to be propagated
      */
     protected void fireNodeDeleted(final DomChangeEvent event) {
-        final List<DomChangeListener> listeners = safeGetDomListeners();
-        if (listeners != null) {
-            for (final DomChangeListener listener : listeners) {
-                listener.nodeDeleted(event);
+        DomNode toInform = this;
+        while (toInform != null) {
+            final List<DomChangeListener> listeners = toInform.safeGetDomListeners();
+            if (listeners != null) {
+                for (final DomChangeListener listener : listeners) {
+                    listener.nodeDeleted(event);
+                }
             }
-        }
-        if (parent_ != null) {
-            parent_.fireNodeDeleted(event);
+            toInform = toInform.getParentNode();
         }
     }