This is an automated email from the ASF dual-hosted git repository.
markt 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 5edb5a1560 Consistent synchronization - reported by Coverity
5edb5a1560 is described below
commit 5edb5a156083a30b5158dcba027f86e7c625ccf5
Author: Mark Thomas <[email protected]>
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: [email protected]
For additional commands, e-mail: [email protected]
