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