Merge pull request #29 from cesarhernandezgt/tomcat-10.0.x-TT.x-patch

Fix BZ 66331
tomitribe · Oct 25, 2023 · 040e7c2 · 040e7c2
2 parents 7f02284 + a905514
commit 040e7c2
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/java/org/apache/tomcat/util/log/SystemLogHandler.java b/java/org/apache/tomcat/util/log/SystemLogHandler.java
@@ -20,7 +20,7 @@
 import java.io.PrintStream;
 import java.util.ArrayDeque;
 import java.util.Deque;
-import java.util.EmptyStackException;
+import java.util.NoSuchElementException;
 import java.util.Queue;
 import java.util.concurrent.ConcurrentLinkedQueue;
 
@@ -83,7 +83,7 @@ public static void startCapture() {
         if (!reuse.isEmpty()) {
             try {
                 log = reuse.remove();
-            } catch (EmptyStackException e) {
+            } catch (NoSuchElementException e) {
                 log = new CaptureLog();
             }
         } else {

diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
@@ -112,6 +112,11 @@
         use of fully qualified class names in 10.1.14 that broke the jdbc-pool.
         (markt)
       </fix>
+      <fix>
+        <bug>66331</bug>: Fix a regression in refactoring for <code>Stack</code>
+        on the <code>SystemLogHandler</code> which caught incorrect exception.
+        (lihan)
+      </fix>
     </changelog>
   </subsection>
   <subsection name="Other">