etan39
/
cloudstack
mirror of https://github.com/apache/cloudstack.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix findbugs DMI_RANDOM_USED_ONLY_ONCE warning in ServerTransportProvider.java Changed to recommended SecureRandom instead of Random Moved initializers to constructor 

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>

This closes #415
This commit is contained in:
Rafael da Fonseca 2015-06-14 16:22:08 +02:00 committed by Rohit Yadav
parent 16baa1289b
commit 4e0fe469dd
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
framework/ipc/src/org/apache/cloudstack/framework/server/ServerTransportProvider.java
View File

@ -20,7 +20,7 @@ package org.apache.cloudstack.framework.server;
import java.util.HashMap;
import java.util.Map;
import java.util.Random;
import java.security.SecureRandom;
import java.util.concurrent.ExecutorService;
import java.util.concurrent.Executors;
@ -47,12 +47,14 @@ public class ServerTransportProvider implements TransportProvider {
private Map<String, TransportEndpointSite> _endpointMap = new HashMap<String, TransportEndpointSite>();
private int _poolSize = DEFAULT_WORKER_POOL_SIZE;
private ExecutorService _executor;
private int _nextEndpointId = new Random().nextInt();
private final SecureRandom randomGenerator;
private int _nextEndpointId;
private MessageSerializer _messageSerializer;
public ServerTransportProvider() {
randomGenerator=new SecureRandom();
_nextEndpointId=randomGenerator.nextInt();
}
public String getNodeId() {