check for null to make findbugs happy

jenkinsci · Dec 8, 2017 · ba96994 · ba96994
1 parent bf5a330
commit ba96994
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/main/java/org/jenkinsci/plugins/docker/workflow/WithContainerStep.java b/src/main/java/org/jenkinsci/plugins/docker/workflow/WithContainerStep.java
@@ -220,8 +220,8 @@ private static class Decorator extends LauncherDecorator implements Serializable
             this.envHost = Util.mapToEnv(envHost);
             this.ws = ws;
             this.toolName = toolName;
-            this.hasEnv = dockerVersion.compareTo(new VersionNumber("1.13.0")) >= 0;
-            this.hasWorkdir = dockerVersion.compareTo(new VersionNumber("17.12")) >= 0;
+            this.hasEnv = dockerVersion != null && dockerVersion.compareTo(new VersionNumber("1.13.0")) >= 0;
+            this.hasWorkdir = dockerVersion != null && dockerVersion.compareTo(new VersionNumber("17.12")) >= 0;
         }
 
         @Override public Launcher decorate(final Launcher launcher, final Node node) {