This is an automated email from the ASF dual-hosted git repository.
markt 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 cceb5b9bd1 Consistent synchronization - reported by Coverity
cceb5b9bd1 is described below
commit cceb5b9bd1913a08f4754df921f131c7b6844120
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Thu Aug 10 15:36:28 2023 +0100
Consistent synchronization - reported by Coverity
---
java/org/apache/catalina/core/StandardService.java | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/java/org/apache/catalina/core/StandardService.java
b/java/org/apache/catalina/core/StandardService.java
index c52dca88f4..7b9a0e601b 100644
--- a/java/org/apache/catalina/core/StandardService.java
+++ b/java/org/apache/catalina/core/StandardService.java
@@ -241,11 +241,13 @@ public class StandardService extends LifecycleMBeanBase
implements Service {
public ObjectName[] getConnectorNames() {
- ObjectName results[] = new ObjectName[connectors.length];
- for (int i = 0; i < results.length; i++) {
- results[i] = connectors[i].getObjectName();
+ synchronized (connectorsLock) {
+ ObjectName results[] = new ObjectName[connectors.length];
+ for (int i = 0; i < results.length; i++) {
+ results[i] = connectors[i].getObjectName();
+ }
+ return results;
}
- return results;
}
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
