From b80a78d885e2568af41fc133b0185399385eda0c Mon Sep 17 00:00:00 2001 From: Clebert Suconic Date: Tue, 12 Feb 2019 09:42:28 +0100 Subject: [PATCH] ARTEMIS-1058 Fixing testsuite Possible NPE that will happen on ReplicatedFailoverTest only --- .../artemis/utils/SpawnedVMSupport.java | 18 ++++++++++++------ 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/artemis-commons/src/main/java/org/apache/activemq/artemis/utils/SpawnedVMSupport.java b/artemis-commons/src/main/java/org/apache/activemq/artemis/utils/SpawnedVMSupport.java index 6f47c8247b..82724c1811 100644 --- a/artemis-commons/src/main/java/org/apache/activemq/artemis/utils/SpawnedVMSupport.java +++ b/artemis-commons/src/main/java/org/apache/activemq/artemis/utils/SpawnedVMSupport.java @@ -141,13 +141,19 @@ public class SpawnedVMSupport { StringBuilder stringBuilder = new StringBuilder(); boolean empty = true; - for (File f : libfolder.listFiles()) { - if (f.getName().endsWith(".jar") || f.getName().endsWith(".zip")) { - if (!empty) { - stringBuilder.append(File.pathSeparator); + File[] files = libfolder.listFiles(); + + if (files == null) { + return getClassPath(); + } else { + for (File f : libfolder.listFiles()) { + if (f.getName().endsWith(".jar") || f.getName().endsWith(".zip")) { + if (!empty) { + stringBuilder.append(File.pathSeparator); + } + empty = false; + stringBuilder.append(f.toString()); } - empty = false; - stringBuilder.append(f.toString()); } }