This is an automated email from the ASF dual-hosted git repository.
markt-asf pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/9.0.x by this push:
new 4df84a6b86 Additional guards for a null Manager
4df84a6b86 is described below
commit 4df84a6b8638455f8f438dc6169739142a419700
Author: Mark Thomas <[email protected]>
AuthorDate: Fri May 22 19:03:25 2026 +0100
Additional guards for a null Manager
---
java/org/apache/catalina/manager/HTMLManagerServlet.java | 7 ++++++-
java/org/apache/catalina/manager/ManagerServlet.java | 8 ++++++--
2 files changed, 12 insertions(+), 3 deletions(-)
diff --git a/java/org/apache/catalina/manager/HTMLManagerServlet.java
b/java/org/apache/catalina/manager/HTMLManagerServlet.java
index 2457da654d..80dde91961 100644
--- a/java/org/apache/catalina/manager/HTMLManagerServlet.java
+++ b/java/org/apache/catalina/manager/HTMLManagerServlet.java
@@ -890,7 +890,12 @@ public class HTMLManagerServlet extends ManagerServlet {
smClient.getString("managerServlet.noContext",
Escape.htmlElementContent(cn.getDisplayName())));
}
Manager manager = ctxt.getManager();
- List<Session> sessions = new
ArrayList<>(Arrays.asList(manager.findSessions()));
+ List<Session> sessions;
+ if (manager == null) {
+ sessions = Collections.emptyList();
+ } else {
+ sessions = new ArrayList<>(Arrays.asList(manager.findSessions()));
+ }
if (manager instanceof DistributedManager && showProxySessions) {
// Add dummy proxy sessions
Set<String> sessionIds = ((DistributedManager)
manager).getSessionIdsFull();
diff --git a/java/org/apache/catalina/manager/ManagerServlet.java
b/java/org/apache/catalina/manager/ManagerServlet.java
index e45dfaccc8..c86fc302e1 100644
--- a/java/org/apache/catalina/manager/ManagerServlet.java
+++ b/java/org/apache/catalina/manager/ManagerServlet.java
@@ -1050,8 +1050,12 @@ public class ManagerServlet extends HttpServlet
implements ContainerServlet {
}
List<String> parts;
if (context.getState().isAvailable()) {
- parts = Arrays.asList(displayPath, "running", "" +
context.getManager().findSessions().length,
- context.getDocBase());
+ int sessionCount = 0;
+ Manager manager = context.getManager();
+ if (manager != null) {
+ sessionCount = manager.findSessions().length;
+ }
+ parts = Arrays.asList(displayPath, "running", "" +
sessionCount, context.getDocBase());
} else {
parts = Arrays.asList(displayPath, "stopped", "0",
context.getDocBase());
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
