From c504056adb0706b5eefc3d1606892dda544ef7a0 Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Thu, 21 Apr 2011 20:57:54 -0700
Subject: [PATCH] improving the error diagnostics

---
 core/src/main/java/hudson/util/ProcessTree.java | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/core/src/main/java/hudson/util/ProcessTree.java b/core/src/main/java/hudson/util/ProcessTree.java
index 08ceebe063..14759618eb 100644
--- a/core/src/main/java/hudson/util/ProcessTree.java
+++ b/core/src/main/java/hudson/util/ProcessTree.java
@@ -1053,10 +1053,14 @@ public abstract class ProcessTree implements Iterable<OSProcess>, IProcessTree,
                         */
 
                     int nargs = m.readInt();
-                    m.readString(); // exec path
-                    for( int i=0; i<nargs; i++) {
-                        m.skip0();
-                        arguments.add(m.readString());
+                    String args0 = m.readString(); // exec path
+                    try {
+                        for( int i=0; i<nargs; i++) {
+                            m.skip0();
+                            arguments.add(m.readString());
+                        }
+                    } catch (IndexOutOfBoundsException e) {
+                        throw new IllegalStateException("Failed to parse arguments: arg0="+args0+", arguments="+arguments+", nargs="+nargs,e);
                     }
 
                     // this is how you can read environment variables
-- 
GitLab