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

NPE at hudson.triggers.SCMTrigger$Runner.hashCode when run is called with job == null

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved (View Workflow)
    • Major
    • Resolution: Fixed
    • core
    • Jenkins v1.622
      Mercurial plugin v1.54

    Description

      I have several projects on my Jenkins server.
      My projects stored in Mercurial SCM.
      For trigger build I use notifyCommit as mentioned in plugin WiKi:

      http://localhost:8080/mercurial/notifyCommit?url=<repository remote url>

      Most of my projects builds after trigger, but one project when get notification send me error report (see attached file jenkins-response.html).

      Attachments

        Issue Links

          is blocking

          Bug - A problem which impairs or prevents the functions of the product. JENKINS-29790 NPE in TimerTrigger after enabling a job previously in disabled state

          • Major - Major loss of function.
          • Resolved
          is duplicated by

          Bug - A problem which impairs or prevents the functions of the product. JENKINS-30366 NPE on webhook request to trigger git polling in Jenkins

          • Critical - Crashes, loss of data, severe memory leak.
          • Closed

          Activity

            Ascending order - Click to sort in descending order
            aleksei_badyaev Aleksei Badyaev created issue -
            jglick Jesse Glick added a comment - 
            java.lang.NullPointerException
	at hudson.triggers.SCMTrigger$Runner.hashCode(SCMTrigger.java:572)
	at java.util.HashMap.hash(HashMap.java:362)
	at java.util.HashMap.put(HashMap.java:492)
	at hudson.util.SequentialExecutionQueue.execute(SequentialExecutionQueue.java:65)
	at hudson.triggers.SCMTrigger.run(SCMTrigger.java:143)
	at hudson.triggers.SCMTrigger.run(SCMTrigger.java:120)
	at hudson.plugins.mercurial.MercurialStatus.handleNotifyCommit(MercurialStatus.java:117)
	at hudson.plugins.mercurial.MercurialStatus.doNotifyCommit(MercurialStatus.java:75)

            Code

            jglick Jesse Glick added a comment - java.lang.NullPointerException at hudson.triggers.SCMTrigger$Runner.hashCode(SCMTrigger.java:572) at java.util.HashMap.hash(HashMap.java:362) at java.util.HashMap.put(HashMap.java:492) at hudson.util.SequentialExecutionQueue.execute(SequentialExecutionQueue.java:65) at hudson.triggers.SCMTrigger.run(SCMTrigger.java:143) at hudson.triggers.SCMTrigger.run(SCMTrigger.java:120) at hudson.plugins.mercurial.MercurialStatus.handleNotifyCommit(MercurialStatus.java:117) at hudson.plugins.mercurial.MercurialStatus.doNotifyCommit(MercurialStatus.java:75) Code
            jglick Jesse Glick made changes -
            Field Original Value New Value
            Link This issue is blocking JENKINS-29790 [ JENKINS-29790 ]
            jglick Jesse Glick made changes -
            Component/s core [ 15593 ]
            Component/s mercurial-plugin [ 15502 ]
            Assignee Jesse Glick [ jglick ] Kanstantsin Shautsou [ integer ]
            integer Kanstantsin Shautsou added a comment -

            First of all bad objects must not to be send to queue, object/Runner has no sense when job is null, so nothing can be changed in hashCode, because scm plugins usually send it through SequentialExecutionQueue to exclude duplicated runs for the same job.

            Added annotations, stubs and tests in https://github.com/jenkinsci/jenkins/pull/1783

            integer Kanstantsin Shautsou added a comment - First of all bad objects must not to be send to queue, object/Runner has no sense when job is null, so nothing can be changed in hashCode, because scm plugins usually send it through SequentialExecutionQueue to exclude duplicated runs for the same job. Added annotations, stubs and tests in https://github.com/jenkinsci/jenkins/pull/1783
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Kanstantsin Shautsou
            Path:
            core/src/main/java/hudson/triggers/SCMTrigger.java
            core/src/main/java/hudson/triggers/TimerTrigger.java
            core/src/main/java/hudson/triggers/Trigger.java
            core/src/test/java/hudson/triggers/SCMTriggerTest.java
            core/src/test/java/hudson/triggers/TimerTriggerTest.java
            http://jenkins-ci.org/commit/jenkins/79c0bbf76680c8491e601be76d062299b7170971
            Log:
            Protect from race condition in Triggers

            [FIXED JENKINS-29790] [FIXED JENKINS-29945]

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Kanstantsin Shautsou Path: core/src/main/java/hudson/triggers/SCMTrigger.java core/src/main/java/hudson/triggers/TimerTrigger.java core/src/main/java/hudson/triggers/Trigger.java core/src/test/java/hudson/triggers/SCMTriggerTest.java core/src/test/java/hudson/triggers/TimerTriggerTest.java http://jenkins-ci.org/commit/jenkins/79c0bbf76680c8491e601be76d062299b7170971 Log: Protect from race condition in Triggers [FIXED JENKINS-29790] [FIXED JENKINS-29945]
            integer Kanstantsin Shautsou made changes -
            Resolution Fixed [ 1 ]
            Status Open [ 1 ] Resolved [ 5 ]
            integer Kanstantsin Shautsou made changes -
            Summary Mercurial notifyCommit cause java.lang.NullPointerException NPE in
            integer Kanstantsin Shautsou made changes -
            Summary NPE in NPE at hudson.triggers.SCMTrigger$Runner.hashCode when run is called with job == null
            integer Kanstantsin Shautsou made changes -
            Link This issue is duplicated by JENKINS-30366 [ JENKINS-30366 ]
            rtyler R. Tyler Croy made changes -
            Workflow JNJira [ 165023 ] JNJira + In-Review [ 197601 ]

            People

              integer Kanstantsin Shautsou
              aleksei_badyaev Aleksei Badyaev
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: