[EXEC-40] NPE in DefaultExecutor.setExitValues()

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.0
• Fix Version/s: 1.0.1

Environment

Ubuntu Linux, Sun Java 1.6.0_13

• Assignee: Siegfried Goeschl
• Reporter: Peter Henderson
• Created: 16/May/09 16:20
• Updated: 24/Oct/10 19:58
• Resolved: 28/Sep/09 20:57

Description

The setExitValues() of DefaultExecutor does not throws a NPE when passed a null argument.
The documentation says passing a null should disable exit value checking.

Solution, modify the body to correctly handle nulls.

public void setExitValues(final int[] values) {
if (values==null)

{ this.extiValues = null; }

else

{ this.exitValues = (int[]) values.clone(); }

}