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

PKCS7.java is needlessly creating SHA1PRNG SecureRandom instances when timestamping is not done

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: P3 P3
    • 8
    • 8
    • security-libs
    • b27
    • generic
    • generic
    • Not verified

        PKCS7.java has a static initializer that creates a SHA1PRNG, which is only used for generating timestamp nonces. This is not even used during basic JDK startup, this class is getting pulled in during the signed Jar Verification of providers like sunpkcs11, etc. I think it would be better to delay this selection until when it's actually needed. This is today's code.

            static {
                SecureRandom tmp = null;
                try {
                    tmp = SecureRandom.getInstance("SHA1PRNG");
                } catch (NoSuchAlgorithmException e) {
                    // should not happen
                }
                RANDOM = tmp;
            }

            private static byte[] generatorTimestampToken(...) {
            ...deleted...
            if (RANDOM != null) {
                nonce = new BigInteger(64, RANDOM);
            }

              vinnie Vincent Ryan
              wetmore Bradford Wetmore
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved:
                Imported:
                Indexed: