Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-47458

Trilead-SSH2 fails with NPE when calling Connection#getReasonClosedCause()

    XMLWordPrintable

    Details

    • Similar Issues:
    • Released As:
      Jenkins 2.157, TrileadSSH build-217-jenkins-14

      Description

      Just caught it in SSH Slaves plugin tests on my local machine:

      Caused by: java.lang.NullPointerException
      	at com.trilead.ssh2.Connection.getReasonClosedCause(Connection.java:1556)
      	at hudson.plugins.sshslaves.SSHLauncher.reportTransportLoss(SSHLauncher.java:1420)
      	at hudson.plugins.sshslaves.SSHLauncher.afterDisconnect(SSHLauncher.java:1343)
      	at hudson.slaves.SlaveComputer$3.run(SlaveComputer.java:603)
      	at jenkins.util.ContextResettingExecutorService$1.run(ContextResettingExecutorService.java:28)
      	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
      	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
      	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
      	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
      	at java.lang.Thread.run(Thread.java:745)
      

      Tirlead code does not check whether "this.tm" field is null before returning getReasonClosedCause from "this.tm". Just a minor defect

        Attachments

          Activity

          Hide
          oleg_nenashev Oleg Nenashev added a comment -

          https://github.com/jenkinsci/trilead-ssh2/pull/25 was merged and released, but we need to upstream it to the core. CC Ivan Fernandez Calvo

          Show
          oleg_nenashev Oleg Nenashev added a comment - https://github.com/jenkinsci/trilead-ssh2/pull/25 was merged and released, but we need to upstream it to the core. CC Ivan Fernandez Calvo
          Hide
          ifernandezcalvo Ivan Fernandez Calvo added a comment -

          The PRs to the core is in place

          Show
          ifernandezcalvo Ivan Fernandez Calvo added a comment - The PRs to the core is in place

            People

            • Assignee:
              oleg_nenashev Oleg Nenashev
              Reporter:
              oleg_nenashev Oleg Nenashev
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: