You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@commons.apache.org by sg...@apache.org on 2009/07/14 17:25:10 UTC

svn commit: r793940 - /commons/proper/exec/trunk/src/main/java/org/apache/commons/exec/DefaultExecutor.java

Author: sgoeschl
Date: Tue Jul 14 15:25:10 2009
New Revision: 793940

URL: http://svn.apache.org/viewvc?rev=793940&view=rev
Log:
[EXEC-40] Fixes NPE in DefaultExecutor.setExitValues().

Modified:
    commons/proper/exec/trunk/src/main/java/org/apache/commons/exec/DefaultExecutor.java

Modified: commons/proper/exec/trunk/src/main/java/org/apache/commons/exec/DefaultExecutor.java
URL: http://svn.apache.org/viewvc/commons/proper/exec/trunk/src/main/java/org/apache/commons/exec/DefaultExecutor.java?rev=793940&r1=793939&r2=793940&view=diff
==============================================================================
--- commons/proper/exec/trunk/src/main/java/org/apache/commons/exec/DefaultExecutor.java (original)
+++ commons/proper/exec/trunk/src/main/java/org/apache/commons/exec/DefaultExecutor.java Tue Jul 14 15:25:10 2009
@@ -198,7 +198,7 @@
 
     /** @see org.apache.commons.exec.Executor#setExitValues(int[]) */
     public void setExitValues(final int[] values) {
-        this.exitValues = (int[]) values.clone();
+        this.exitValues = (values == null ? null : (int[]) values.clone());
     }
 
     /** @see org.apache.commons.exec.Executor#isFailure(int) */