This is an automated email from the ASF dual-hosted git repository.
remm pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/main by this push:
new 4331d9122b Cleanup
4331d9122b is described below
commit 4331d9122b9aaf8dd60ec9ec108ed56b6c5a8e62
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 06aa148d5e..e9ee8c81ba 100644
--- a/java/org/apache/catalina/session/ManagerBase.java
+++ b/java/org/apache/catalina/session/ManagerBase.java
@@ -605,11 +605,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]
