This is an automated email from the ASF dual-hosted git repository.
remm 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 83c4c97415 Cleanup
83c4c97415 is described below
commit 83c4c97415dfb3b6e1e5986bfc773c93ca36282a
Author: remm <[email protected]>
AuthorDate: Mon Sep 11 16:35:26 2023 +0200
Cleanup
Although concurrency is hard to imagine while starting, sync for
consistency.
Found by coverity.
---
java/org/apache/catalina/session/ManagerBase.java | 12 ++++++++----
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/java/org/apache/catalina/session/ManagerBase.java
b/java/org/apache/catalina/session/ManagerBase.java
index 129c4f842a..5645bf652c 100644
--- a/java/org/apache/catalina/session/ManagerBase.java
+++ b/java/org/apache/catalina/session/ManagerBase.java
@@ -628,11 +628,15 @@ public abstract class ManagerBase extends
LifecycleMBeanBase implements Manager
// Ensure caches for timing stats are the right size by filling with
// nulls.
- while (sessionCreationTiming.size() < TIMING_STATS_CACHE_SIZE) {
- sessionCreationTiming.add(null);
+ synchronized (sessionCreationTiming) {
+ while (sessionCreationTiming.size() < TIMING_STATS_CACHE_SIZE) {
+ sessionCreationTiming.add(null);
+ }
}
- while (sessionExpirationTiming.size() < TIMING_STATS_CACHE_SIZE) {
- sessionExpirationTiming.add(null);
+ synchronized (sessionExpirationTiming) {
+ while (sessionExpirationTiming.size() < TIMING_STATS_CACHE_SIZE) {
+ sessionExpirationTiming.add(null);
+ }
}
/* Create sessionIdGenerator if not explicitly configured */
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
