• Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • remoting
    • None
    • Platform: All, OS: All

      There is a bug in line 150 of hudson.remoting.Engine which reads:

      String[] tokens = tunnel.split(":");
      if(tokens.length!=2)
      throw new IOException("Illegal tunneling parameter: "+tunnel);

      This does not work as documented since String.split ignores delimiters at the
      end of the string, thus when tunnel="HOST:" tokens.length is 1. The fix is trivial

      String[] tokens = tunnel.split(":",2);
      if(tokens.length!=2)
      throw new IOException("Illegal tunneling parameter: "+tunnel);

          [JENKINS-2869] Bug in hudson.remoting.Engine

          tonyj created issue -

          Alan Harder added a comment -

          using "3" instead of "2" for param, to actually a bad case with too many :

          Alan Harder added a comment - using "3" instead of "2" for param, to actually a bad case with too many :

          Code changed in hudson
          User: : mindless
          Path:
          trunk/hudson/main/remoting/src/main/java/hudson/remoting/Engine.java
          http://fisheye4.cenqua.com/changelog/hudson/?cs=16147
          Log:
          [FIXED JENKINS-2869] accept "host:" (trailing as valid tunnel

          SCM/JIRA link daemon added a comment - Code changed in hudson User: : mindless Path: trunk/hudson/main/remoting/src/main/java/hudson/remoting/Engine.java http://fisheye4.cenqua.com/changelog/hudson/?cs=16147 Log: [FIXED JENKINS-2869] accept "host:" (trailing as valid tunnel
          SCM/JIRA link daemon made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]

          Code changed in hudson
          User: : kohsuke
          Path:
          trunk/www/changelog.html
          http://fisheye4.cenqua.com/changelog/hudson/?cs=16152
          Log:
          noting JENKINS-2869 as fixed in 1.291

          SCM/JIRA link daemon added a comment - Code changed in hudson User: : kohsuke Path: trunk/www/changelog.html http://fisheye4.cenqua.com/changelog/hudson/?cs=16152 Log: noting JENKINS-2869 as fixed in 1.291
          Andrew Bayer made changes -
          Status Original: Resolved [ 5 ] New: Closed [ 6 ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 132942 ] New: JNJira + In-Review [ 201796 ]

            mindless Alan Harder
            tonyj tonyj
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Created:
              Updated:
              Resolved: