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

Possible NPE in AbstractProject.resolveForCLI()

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved (View Workflow)
    • Priority: Minor
    • Resolution: Fixed
    • Component/s: core
    • Labels:
      None
    • Similar Issues:

      Description

      Possible NPE can occur when findNearest() returns a null value.

      The issue is similar to AbstractItem.resolveForCLI tracked at JENKINS-30629.

        Attachments

          Issue Links

            Activity

            Hide
            pajasoft Pavel Janoušek added a comment -

            PR sent.

            Show
            pajasoft Pavel Janoušek added a comment - PR sent.
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: tfennelly
            Path:
            .gitignore
            changelog.html
            cli/pom.xml
            core/pom.xml
            core/src/main/java/hudson/model/AbstractProject.java
            core/src/main/java/hudson/model/Run.java
            core/src/main/resources/hudson/widgets/HistoryWidget/index.jelly
            core/src/main/resources/lib/form/confirm.js
            core/src/main/resources/lib/form/select/select.js
            core/src/main/resources/lib/layout/layout.jelly
            plugins/pom.xml
            pom.xml
            test/pom.xml
            test/src/test/groovy/hudson/model/AbstractProjectTest.groovy
            war/pom.xml
            http://jenkins-ci.org/commit/jenkins/95ca3da67d217c90d31819ec92e521e2072acd5a
            Log:
            Merge branch 'master' into plugin-manager-dependants

            • master:
              Update the changelog by new merges:
              [FIXED JENKINS-30569] HistoryWidget: fix JS syntax error
              [FIXED JENKINS-29014] render API link conditional on getApi() presence
              JENKINS-21720 JS alert preventig to leave a configuration page even without formulary changes
              add ctags file 'tags' to .gitignore
              [maven-release-plugin] prepare release jenkins-1.632
              [FIXED JENKINS-29888] Handling all exceptions returned by logRotator
              JENKINS-30742 Fixed possible NPE in AbstractProject.resolveForCLI()
            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: tfennelly Path: .gitignore changelog.html cli/pom.xml core/pom.xml core/src/main/java/hudson/model/AbstractProject.java core/src/main/java/hudson/model/Run.java core/src/main/resources/hudson/widgets/HistoryWidget/index.jelly core/src/main/resources/lib/form/confirm.js core/src/main/resources/lib/form/select/select.js core/src/main/resources/lib/layout/layout.jelly plugins/pom.xml pom.xml test/pom.xml test/src/test/groovy/hudson/model/AbstractProjectTest.groovy war/pom.xml http://jenkins-ci.org/commit/jenkins/95ca3da67d217c90d31819ec92e521e2072acd5a Log: Merge branch 'master' into plugin-manager-dependants master: Update the changelog by new merges: [FIXED JENKINS-30569] HistoryWidget: fix JS syntax error [FIXED JENKINS-29014] render API link conditional on getApi() presence JENKINS-21720 JS alert preventig to leave a configuration page even without formulary changes add ctags file 'tags' to .gitignore [maven-release-plugin] prepare release jenkins-1.632 [FIXED JENKINS-29888] Handling all exceptions returned by logRotator JENKINS-30742 Fixed possible NPE in AbstractProject.resolveForCLI()
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Ing. Pavel Janousek
            Path:
            core/src/main/java/hudson/model/AbstractProject.java
            test/src/test/groovy/hudson/model/AbstractProjectTest.groovy
            http://jenkins-ci.org/commit/jenkins/37c5656f2cf969bbce8f1f0e0ff6a0462f348a8e
            Log:
            JENKINS-30742 Fixed possible NPE in AbstractProject.resolveForCLI()

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ing. Pavel Janousek Path: core/src/main/java/hudson/model/AbstractProject.java test/src/test/groovy/hudson/model/AbstractProjectTest.groovy http://jenkins-ci.org/commit/jenkins/37c5656f2cf969bbce8f1f0e0ff6a0462f348a8e Log: JENKINS-30742 Fixed possible NPE in AbstractProject.resolveForCLI()
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Oleg Nenashev
            Path:
            core/src/main/java/hudson/model/AbstractProject.java
            test/src/test/groovy/hudson/model/AbstractProjectTest.groovy
            http://jenkins-ci.org/commit/jenkins/236198883b598048488c5c80202e85470012021c
            Log:
            Merge pull request #1849 from pjanouse/JENKINS-30742

            JENKINS-30742 Fixed possible NPE in AbstractProject.resolveForCLI()

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: core/src/main/java/hudson/model/AbstractProject.java test/src/test/groovy/hudson/model/AbstractProjectTest.groovy http://jenkins-ci.org/commit/jenkins/236198883b598048488c5c80202e85470012021c Log: Merge pull request #1849 from pjanouse/ JENKINS-30742 JENKINS-30742 Fixed possible NPE in AbstractProject.resolveForCLI()
            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: changelog.html http://jenkins-ci.org/commit/jenkins/a79e2f144a932fe0a97b524f79133d1ea5d0fef0 Log: Update the changelog by new merges: JENKINS-29888 - https://github.com/jenkinsci/jenkins/pull/1790 JENKINS-30742 - https://github.com/jenkinsci/jenkins/pull/1849 JENKINS-29014 - https://github.com/jenkinsci/jenkins/pull/1850 JENKINS-21720 - https://github.com/jenkinsci/jenkins/pull/1854 JENKINS-30569 - https://github.com/jenkinsci/jenkins/pull/1856
            Hide
            danielbeck Daniel Beck added a comment -

            Assuming this is resolved.

            Show
            danielbeck Daniel Beck added a comment - Assuming this is resolved.

              People

              Assignee:
              pajasoft Pavel Janoušek
              Reporter:
              pajasoft Pavel Janoušek
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: