You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mina.apache.org by el...@apache.org on 2017/04/14 12:26:34 UTC
mina-ftpserver git commit: Applied patch for FTPSERVER-424
Repository: mina-ftpserver
Updated Branches:
refs/heads/master 3e391f57b -> 9557d1d5f
Applied patch for FTPSERVER-424
Project: http://git-wip-us.apache.org/repos/asf/mina-ftpserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/mina-ftpserver/commit/9557d1d5
Tree: http://git-wip-us.apache.org/repos/asf/mina-ftpserver/tree/9557d1d5
Diff: http://git-wip-us.apache.org/repos/asf/mina-ftpserver/diff/9557d1d5
Branch: refs/heads/master
Commit: 9557d1d5f9ce7e29dd0dac6bc9c64483ed4a20cb
Parents: 3e391f5
Author: Emmanuel L�charny <el...@symas.com>
Authored: Fri Apr 14 14:26:17 2017 +0200
Committer: Emmanuel L�charny <el...@symas.com>
Committed: Fri Apr 14 14:26:17 2017 +0200
----------------------------------------------------------------------
.../java/org/apache/ftpserver/impl/PassivePorts.java | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mina-ftpserver/blob/9557d1d5/core/src/main/java/org/apache/ftpserver/impl/PassivePorts.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/ftpserver/impl/PassivePorts.java b/core/src/main/java/org/apache/ftpserver/impl/PassivePorts.java
index 4d00f12..d812061 100644
--- a/core/src/main/java/org/apache/ftpserver/impl/PassivePorts.java
+++ b/core/src/main/java/org/apache/ftpserver/impl/PassivePorts.java
@@ -154,8 +154,8 @@ public class PassivePorts {
if (passivePorts == null) {
throw new NullPointerException("passivePorts can not be null");
} else if(passivePorts.isEmpty()) {
- passivePorts = new HashSet<Integer>();
- passivePorts.add(0);
+ passivePorts = new HashSet<Integer>();
+ passivePorts.add(0);
}
this.freeList = new ArrayList<Integer>(passivePorts);
@@ -199,9 +199,9 @@ public class PassivePorts {
}
public synchronized int reserveNextPort() {
- // create a copy of the free ports, so that we can keep track of the tested ports
- List<Integer> freeCopy = new ArrayList<Integer>(freeList);
-
+ // create a copy of the free ports, so that we can keep track of the tested ports
+ List<Integer> freeCopy = new ArrayList<Integer>(freeList);
+
// Loop until we have found a port, or exhausted all available ports
while (freeCopy.size() > 0) {
// Otherwise, pick one at random
@@ -215,12 +215,12 @@ public class PassivePorts {
} else if (checkPortUnbound(ret)) {
// Not used by someone else, so lets reserve it and return it
- freeList.remove(i);
+ freeList.remove(ret);
usedList.add(ret);
return ret;
} else {
- freeCopy.remove(i);
+ freeCopy.remove(i);
// log port unavailable, but left in pool
log.warn("Passive port in use by another process: " + ret);
}