[java] Choosing a new random port on each invocation of build method …

…in a driver service builder (if there is no port specified explicitly)
SeleniumHQ · Sep 25, 2019 · fd7c919 · fd7c919
1 parent acb61e5
commit fd7c919
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/java/client/src/org/openqa/selenium/remote/service/DriverService.java b/java/client/src/org/openqa/selenium/remote/service/DriverService.java
@@ -356,8 +356,9 @@ protected File getLogFile() {
      * @return The new service object.
      */
     public DS build() {
-      if (port == 0) {
-        port = PortProber.findFreePort();
+      int portToUse = port;
+      if (portToUse == 0) {
+        portToUse = PortProber.findFreePort();
       }
 
       if (exe == null) {
@@ -366,7 +367,7 @@ public DS build() {
 
       ImmutableList<String> args = createArgs();
 
-      return createDriverService(exe, port, args, environment);
+      return createDriverService(exe, portToUse, args, environment);
     }
 
     protected abstract File findDefaultExecutable();