You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by re...@apache.org on 2020/12/14 08:21:55 UTC

[tomcat] branch master updated: Avoid InaccessibleObjectException on Java 16

This is an automated email from the ASF dual-hosted git repository.

remm pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tomcat.git


The following commit(s) were added to refs/heads/master by this push:
     new efc6af6  Avoid InaccessibleObjectException on Java 16
efc6af6 is described below

commit efc6af6778ff3c1605d8b053f6fd2a4d9fd8e0d3
Author: remm <re...@apache.org>
AuthorDate: Mon Dec 14 09:16:46 2020 +0100

    Avoid InaccessibleObjectException on Java 16
---
 java/org/apache/catalina/loader/WebappClassLoaderBase.java | 6 ++++--
 webapps/docs/changelog.xml                                 | 4 ++++
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/java/org/apache/catalina/loader/WebappClassLoaderBase.java b/java/org/apache/catalina/loader/WebappClassLoaderBase.java
index e105452..2bc07a1 100644
--- a/java/org/apache/catalina/loader/WebappClassLoaderBase.java
+++ b/java/org/apache/catalina/loader/WebappClassLoaderBase.java
@@ -1802,8 +1802,10 @@ public abstract class WebappClassLoaderBase extends URLClassLoader
                                 usingExecutor = true;
                             }
                         }
-                    } catch (SecurityException | NoSuchFieldException | IllegalArgumentException |
-                            IllegalAccessException e) {
+                    } catch (/*SecurityException |*/ NoSuchFieldException | /*IllegalArgumentException |*/
+                            IllegalAccessException | RuntimeException e) {
+                        // FIXME: InaccessibleObjectException is only available in Java 9+,
+                        // swapped for RuntimeException for now
                         log.warn(sm.getString("webappClassLoader.stopThreadFail",
                                 thread.getName(), getContextName()), e);
                     }
diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index caff865..89c8d18 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -138,6 +138,10 @@
         be logged in the access log using the new <code>%{peer}a</code>
         syntax. (rjung)
       </add>
+      <fix>
+        Avoid uncaught InaccessibleObjectException on Java 16 trying to clear
+        references threads. (remm)
+      </fix>
     </changelog>
   </subsection>
   <subsection name="Coyote">


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org