Code Review
/
nonrtric.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
Fix potential risk of race condition
[nonrtric.git]
/
policy-agent
/
src
/
main
/
java
/
org
/
oransc
/
policyagent
/
repository
/
Services.java
diff --git
a/policy-agent/src/main/java/org/oransc/policyagent/repository/Services.java
b/policy-agent/src/main/java/org/oransc/policyagent/repository/Services.java
index
f6c55dc
..
1c0e15a
100644
(file)
--- a/
policy-agent/src/main/java/org/oransc/policyagent/repository/Services.java
+++ b/
policy-agent/src/main/java/org/oransc/policyagent/repository/Services.java
@@
-20,9
+20,9
@@
package org.oransc.policyagent.repository;
package org.oransc.policyagent.repository;
-import java.util.Collections;
import java.util.HashMap;
import java.util.Map;
import java.util.HashMap;
import java.util.Map;
+import java.util.Vector;
import org.oransc.policyagent.exceptions.ServiceException;
import org.slf4j.Logger;
import org.oransc.policyagent.exceptions.ServiceException;
import org.slf4j.Logger;
@@
-47,11
+47,12
@@
public class Services {
public synchronized void put(Service service) {
logger.debug("Put service: {}", service.getName());
public synchronized void put(Service service) {
logger.debug("Put service: {}", service.getName());
+ service.keepAlive();
registeredServices.put(service.getName(), service);
}
public synchronized Iterable<Service> getAll() {
registeredServices.put(service.getName(), service);
}
public synchronized Iterable<Service> getAll() {
- return
Collections.unmodifiableCollection
(registeredServices.values());
+ return
new Vector<>
(registeredServices.values());
}
public synchronized void remove(String name) {
}
public synchronized void remove(String name) {
@@
-62,7
+63,7
@@
public class Services {
return registeredServices.size();
}
return registeredServices.size();
}
- public void clear() {
+ public
synchronized
void clear() {
registeredServices.clear();
}
}
registeredServices.clear();
}
}