Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8165101

AnchorCertificates throws NPE when cacerts file not found

    Details

    • Subcomponent:
    • Resolved In Build:
      b140
    • Verification:
      Not verified

      Backports

        Description

        AnchorCertificates throws NPE when cacerts file doesn't exists. Since this class only throws debug message "Error parsing cacerts" when file is not found, I expect JDK to proceed without NPE.

          Issue Links

            Activity

            Hide
            ascarpino Anthony Scarpino added a comment - - edited
            contains() expects the HashSet object, certs, to be created, but when there is no cacerts file the certs object is still null. Since the cacerts file is always created, this is not a big problem and is appropriately a P4, but contains() should be more careful.
            Show
            ascarpino Anthony Scarpino added a comment - - edited contains() expects the HashSet object, certs, to be created, but when there is no cacerts file the certs object is still null. Since the cacerts file is always created, this is not a big problem and is appropriately a P4, but contains() should be more careful.
            Hide
            ascarpino Anthony Scarpino added a comment -
            While I have not been able to reproduce the failure because the test fails because the cacerts file was not found rather than a null pointer. I am proceeding with the fix as it possible this could occur. There is not regression test because it's removing the cacerts file can't be done concurrently and this is a corner case that it's trivial.
            Show
            ascarpino Anthony Scarpino added a comment - While I have not been able to reproduce the failure because the test fails because the cacerts file was not found rather than a null pointer. I am proceeding with the fix as it possible this could occur. There is not regression test because it's removing the cacerts file can't be done concurrently and this is a corner case that it's trivial.
            Hide
            hgupdate HG Updates added a comment -
            URL: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/b76c2c377883
            User: ascarpino
            Date: 2016-10-10 20:30:20 +0000
            Show
            hgupdate HG Updates added a comment - URL: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/b76c2c377883 User: ascarpino Date: 2016-10-10 20:30:20 +0000
            Hide
            hgupdate HG Updates added a comment -
            URL: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/b76c2c377883
            User: lana
            Date: 2016-10-12 19:59:33 +0000
            Show
            hgupdate HG Updates added a comment - URL: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/b76c2c377883 User: lana Date: 2016-10-12 19:59:33 +0000

              People

              • Assignee:
                ascarpino Anthony Scarpino
                Reporter:
                rhalade Rajan Halade
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: