OpenJDK / jdk8u / jdk8u / jdk
changeset 12333:56e0ab47dbec
8174113: Better sourcing of code
Reviewed-by: weijun
author | robm |
---|---|
date | Tue, 28 Mar 2017 15:46:05 +0100 |
parents | 56e011f1384f |
children | 079cd6c5de27 |
files | src/share/classes/java/security/CodeSource.java |
diffstat | 1 files changed, 8 insertions(+), 3 deletions(-) [+] |
line wrap: on
line diff
--- a/src/share/classes/java/security/CodeSource.java Mon Apr 03 13:46:33 2017 -0700 +++ b/src/share/classes/java/security/CodeSource.java Tue Mar 28 15:46:05 2017 +0100 @@ -1,5 +1,5 @@ /* - * Copyright (c) 1997, 2013, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 1997, 2017, Oracle and/or its affiliates. All rights reserved. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * * This code is free software; you can redistribute it and/or modify it @@ -535,6 +535,7 @@ { CertificateFactory cf; Hashtable<String, CertificateFactory> cfs = null; + List<java.security.cert.Certificate> certList = null; ois.defaultReadObject(); // location @@ -544,7 +545,7 @@ // we know of 3 different cert types: X.509, PGP, SDSI, which // could all be present in the stream at the same time cfs = new Hashtable<String, CertificateFactory>(3); - this.certs = new java.security.cert.Certificate[size]; + certList = new ArrayList<>(size > 20 ? 20 : size); } for (int i = 0; i < size; i++) { @@ -575,13 +576,17 @@ ois.readFully(encoded); ByteArrayInputStream bais = new ByteArrayInputStream(encoded); try { - this.certs[i] = cf.generateCertificate(bais); + certList.add(cf.generateCertificate(bais)); } catch (CertificateException ce) { throw new IOException(ce.getMessage()); } bais.close(); } + if (certList != null) { + this.certs = certList.toArray( + new java.security.cert.Certificate[size]); + } // Deserialize array of code signers (if any) try { this.signers = ((CodeSigner[])ois.readObject()).clone();