• 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

          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

          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

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

              Created:
              Updated:
              Resolved: