FIX: must clear auxiliary ID array when finished

CruiseControl · Nov 1, 2018 · b02ecf5 · b02ecf5
1 parent 85cd0e1
commit b02ecf5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/main/checkstyleSuppressions.xml b/main/checkstyleSuppressions.xml
@@ -9,7 +9,7 @@
     <suppress checks="AnonInnerLength" files="NantBuilderTest.java"/>
 
     <suppress checks="ModifiedControlVariable" files="Launcher.java"/>
-    <suppress checks="ModifiedControlVariable" files="PipedExecBuilder.java" lines="178"/>  <!-- iterator re-assignment in for loop within validate() -->
+    <suppress checks="ModifiedControlVariable" files="PipedExecBuilder.java" lines="179"/>  <!-- iterator re-assignment in for loop within validate() -->
 
     <!-- main/src/net/sourceforge/cruisecontrol/builders/PipedScript.java:10:8: Unused import - net.sourceforge.cruisecontrol.Builder
          But the import is required by javadoc ...  -->

diff --git a/main/src/net/sourceforge/cruisecontrol/builders/PipedExecBuilder.java b/main/src/net/sourceforge/cruisecontrol/builders/PipedExecBuilder.java
@@ -173,6 +173,7 @@ public void validate() throws CruiseControlException {
                            if (auxIDs.size() != p.getWaitFor().length) {
                                p.setWaitFor(PipedScript.Helpers.join(auxIDs));
                            }
+                           auxIDs.clear();
                       }
                       /* Re-assign the iterator (checkstyle suppresses in main/checkstyleSuppressions.xml) */
                       siter = scripts.iterator();