jdk/src/jdk.jartool/share/classes/sun/security/tools/jarsigner/Main.java
author weijun
Tue, 08 Aug 2017 21:37:38 +0800
changeset 46139 5196af754957
parent 44597 98f0c39ce3a8
permissions -rw-r--r--
8185934: keytool shows "Signature algorithm: SHA1withECDSA, -1-bit key" Reviewed-by: mullan
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
     1
/*
31060
be4eb6360ee0 8076535: Deprecate the com.sun.jarsigner package
juh
parents: 29369
diff changeset
     2
 * Copyright (c) 1997, 2015, Oracle and/or its affiliates. All rights reserved.
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
90ce3da70b43 Initial load
duke
parents:
diff changeset
     4
 *
90ce3da70b43 Initial load
duke
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
90ce3da70b43 Initial load
duke
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
5506
202f599c92aa 6943119: Rebrand source copyright notices
ohair
parents: 5462
diff changeset
     7
 * published by the Free Software Foundation.  Oracle designates this
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
     8
 * particular file as subject to the "Classpath" exception as provided
5506
202f599c92aa 6943119: Rebrand source copyright notices
ohair
parents: 5462
diff changeset
     9
 * by Oracle in the LICENSE file that accompanied this code.
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
    10
 *
90ce3da70b43 Initial load
duke
parents:
diff changeset
    11
 * This code is distributed in the hope that it will be useful, but WITHOUT
90ce3da70b43 Initial load
duke
parents:
diff changeset
    12
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
90ce3da70b43 Initial load
duke
parents:
diff changeset
    13
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
90ce3da70b43 Initial load
duke
parents:
diff changeset
    14
 * version 2 for more details (a copy is included in the LICENSE file that
90ce3da70b43 Initial load
duke
parents:
diff changeset
    15
 * accompanied this code).
90ce3da70b43 Initial load
duke
parents:
diff changeset
    16
 *
90ce3da70b43 Initial load
duke
parents:
diff changeset
    17
 * You should have received a copy of the GNU General Public License version
90ce3da70b43 Initial load
duke
parents:
diff changeset
    18
 * 2 along with this work; if not, write to the Free Software Foundation,
90ce3da70b43 Initial load
duke
parents:
diff changeset
    19
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
90ce3da70b43 Initial load
duke
parents:
diff changeset
    20
 *
5506
202f599c92aa 6943119: Rebrand source copyright notices
ohair
parents: 5462
diff changeset
    21
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
202f599c92aa 6943119: Rebrand source copyright notices
ohair
parents: 5462
diff changeset
    22
 * or visit www.oracle.com if you need additional information or have any
202f599c92aa 6943119: Rebrand source copyright notices
ohair
parents: 5462
diff changeset
    23
 * questions.
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
    24
 */
90ce3da70b43 Initial load
duke
parents:
diff changeset
    25
14182
3041082abb40 7194449: String resources for Key Tool and Policy Tool should be in their respective packages
sflores
parents: 12046
diff changeset
    26
package sun.security.tools.jarsigner;
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
    27
90ce3da70b43 Initial load
duke
parents:
diff changeset
    28
import java.io.*;
43183
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
    29
import java.security.cert.CertPathValidatorException;
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
    30
import java.security.cert.PKIXBuilderParameters;
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
    31
import java.util.*;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    32
import java.util.zip.*;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    33
import java.util.jar.*;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    34
import java.net.URI;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    35
import java.text.Collator;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    36
import java.text.MessageFormat;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    37
import java.security.cert.Certificate;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    38
import java.security.cert.X509Certificate;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    39
import java.security.cert.CertificateException;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    40
import java.security.*;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    41
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
    42
import java.net.SocketTimeoutException;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
    43
import java.net.URL;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
    44
import java.security.cert.CertPath;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
    45
import java.security.cert.CertificateExpiredException;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
    46
import java.security.cert.CertificateFactory;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
    47
import java.security.cert.CertificateNotYetValidException;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
    48
import java.security.cert.TrustAnchor;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
    49
import java.util.Map.Entry;
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
    50
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
    51
import jdk.security.jarsigner.JarSigner;
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
    52
import jdk.security.jarsigner.JarSignerException;
41590
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
    53
import sun.security.pkcs.PKCS7;
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
    54
import sun.security.pkcs.SignerInfo;
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
    55
import sun.security.timestamp.TimestampToken;
14182
3041082abb40 7194449: String resources for Key Tool and Policy Tool should be in their respective packages
sflores
parents: 12046
diff changeset
    56
import sun.security.tools.KeyStoreUtil;
43183
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
    57
import sun.security.validator.Validator;
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
    58
import sun.security.validator.ValidatorException;
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
    59
import sun.security.x509.*;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    60
import sun.security.util.*;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    61
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
    62
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
    63
/**
90ce3da70b43 Initial load
duke
parents:
diff changeset
    64
 * <p>The jarsigner utility.
90ce3da70b43 Initial load
duke
parents:
diff changeset
    65
 *
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
    66
 * The exit codes for the main method are:
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
    67
 *
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
    68
 * 0: success
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
    69
 * 1: any error that the jar cannot be signed or verified, including:
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
    70
 *      keystore loading error
12046
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
    71
 *      TSP communication error
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
    72
 *      jarsigner command line error...
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
    73
 * otherwise: error codes from -strict
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
    74
 *
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
    75
 * @author Roland Schemers
90ce3da70b43 Initial load
duke
parents:
diff changeset
    76
 * @author Jan Luehe
90ce3da70b43 Initial load
duke
parents:
diff changeset
    77
 */
14182
3041082abb40 7194449: String resources for Key Tool and Policy Tool should be in their respective packages
sflores
parents: 12046
diff changeset
    78
public class Main {
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
    79
90ce3da70b43 Initial load
duke
parents:
diff changeset
    80
    // for i18n
90ce3da70b43 Initial load
duke
parents:
diff changeset
    81
    private static final java.util.ResourceBundle rb =
90ce3da70b43 Initial load
duke
parents:
diff changeset
    82
        java.util.ResourceBundle.getBundle
14182
3041082abb40 7194449: String resources for Key Tool and Policy Tool should be in their respective packages
sflores
parents: 12046
diff changeset
    83
        ("sun.security.tools.jarsigner.Resources");
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
    84
    private static final Collator collator = Collator.getInstance();
90ce3da70b43 Initial load
duke
parents:
diff changeset
    85
    static {
90ce3da70b43 Initial load
duke
parents:
diff changeset
    86
        // this is for case insensitive string comparisions
90ce3da70b43 Initial load
duke
parents:
diff changeset
    87
        collator.setStrength(Collator.PRIMARY);
90ce3da70b43 Initial load
duke
parents:
diff changeset
    88
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
    89
90ce3da70b43 Initial load
duke
parents:
diff changeset
    90
    private static final String NONE = "NONE";
90ce3da70b43 Initial load
duke
parents:
diff changeset
    91
    private static final String P11KEYSTORE = "PKCS11";
90ce3da70b43 Initial load
duke
parents:
diff changeset
    92
90ce3da70b43 Initial load
duke
parents:
diff changeset
    93
    private static final long SIX_MONTHS = 180*24*60*60*1000L; //milliseconds
90ce3da70b43 Initial load
duke
parents:
diff changeset
    94
41590
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
    95
    private static final DisabledAlgorithmConstraints DISABLED_CHECK =
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
    96
            new DisabledAlgorithmConstraints(
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
    97
                    DisabledAlgorithmConstraints.PROPERTY_JAR_DISABLED_ALGS);
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
    98
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
    99
    private static final Set<CryptoPrimitive> DIGEST_PRIMITIVE_SET = Collections
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   100
            .unmodifiableSet(EnumSet.of(CryptoPrimitive.MESSAGE_DIGEST));
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   101
    private static final Set<CryptoPrimitive> SIG_PRIMITIVE_SET = Collections
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   102
            .unmodifiableSet(EnumSet.of(CryptoPrimitive.SIGNATURE));
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   103
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   104
    // Attention:
90ce3da70b43 Initial load
duke
parents:
diff changeset
   105
    // This is the entry that get launched by the security tool jarsigner.
90ce3da70b43 Initial load
duke
parents:
diff changeset
   106
    public static void main(String args[]) throws Exception {
14182
3041082abb40 7194449: String resources for Key Tool and Policy Tool should be in their respective packages
sflores
parents: 12046
diff changeset
   107
        Main js = new Main();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   108
        js.run(args);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   109
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   110
90ce3da70b43 Initial load
duke
parents:
diff changeset
   111
    static final String VERSION = "1.0";
90ce3da70b43 Initial load
duke
parents:
diff changeset
   112
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   113
    static final int IN_KEYSTORE = 0x01;        // signer is in keystore
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   114
    static final int NOT_ALIAS = 0x04;          // alias list is NOT empty and
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   115
                                                // signer is not in alias list
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   116
    static final int SIGNED_BY_ALIAS = 0x08;    // signer is in alias list
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   117
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   118
    X509Certificate[] certChain;    // signer's cert chain (when composing)
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   119
    PrivateKey privateKey;          // private key
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   120
    KeyStore store;                 // the keystore specified by -keystore
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   121
                                    // or the default keystore, never null
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   122
90ce3da70b43 Initial load
duke
parents:
diff changeset
   123
    String keystore; // key store file
90ce3da70b43 Initial load
duke
parents:
diff changeset
   124
    boolean nullStream = false; // null keystore input stream (NONE)
90ce3da70b43 Initial load
duke
parents:
diff changeset
   125
    boolean token = false; // token-based keystore
5462
cb614e59f7f9 6890876: jarsigner can add CRL info into signed jar
weijun
parents: 5461
diff changeset
   126
    String jarfile;  // jar files to sign or verify
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   127
    String alias;    // alias to sign jar with
7977
f47f211cd627 7008713: diamond conversion of kerberos5 and security tools
smarks
parents: 7525
diff changeset
   128
    List<String> ckaliases = new ArrayList<>(); // aliases in -verify
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   129
    char[] storepass; // keystore password
90ce3da70b43 Initial load
duke
parents:
diff changeset
   130
    boolean protectedPath; // protected authentication path
90ce3da70b43 Initial load
duke
parents:
diff changeset
   131
    String storetype; // keystore type
90ce3da70b43 Initial load
duke
parents:
diff changeset
   132
    String providerName; // provider name
39633
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   133
    List<String> providers = null; // list of provider names
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   134
    List<String> providerClasses = null; // list of provider classes
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   135
    // arguments for provider constructors
7977
f47f211cd627 7008713: diamond conversion of kerberos5 and security tools
smarks
parents: 7525
diff changeset
   136
    HashMap<String,String> providerArgs = new HashMap<>();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   137
    char[] keypass; // private key password
90ce3da70b43 Initial load
duke
parents:
diff changeset
   138
    String sigfile; // name of .SF file
90ce3da70b43 Initial load
duke
parents:
diff changeset
   139
    String sigalg; // name of signature algorithm
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
   140
    String digestalg; // name of digest algorithm
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   141
    String signedjar; // output filename
90ce3da70b43 Initial load
duke
parents:
diff changeset
   142
    String tsaUrl; // location of the Timestamping Authority
90ce3da70b43 Initial load
duke
parents:
diff changeset
   143
    String tsaAlias; // alias for the Timestamping Authority's certificate
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   144
    String altCertChain; // file to read alternative cert chain from
17161
df1ec0e2f0e7 8009636: JARSigner including TimeStamp PolicyID (TSAPolicyID) as defined in RFC3161
weijun
parents: 16020
diff changeset
   145
    String tSAPolicyID;
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
   146
    String tSADigestAlg;
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   147
    boolean verify = false; // verify the jar
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   148
    String verbose = null; // verbose output when signing/verifying
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   149
    boolean showcerts = false; // show certs when verifying
90ce3da70b43 Initial load
duke
parents:
diff changeset
   150
    boolean debug = false; // debug
90ce3da70b43 Initial load
duke
parents:
diff changeset
   151
    boolean signManifest = true; // "sign" the whole manifest
90ce3da70b43 Initial load
duke
parents:
diff changeset
   152
    boolean externalSF = true; // leave the .SF out of the PKCS7 block
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   153
    boolean strict = false;  // treat warnings as error
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   154
90ce3da70b43 Initial load
duke
parents:
diff changeset
   155
    // read zip entry raw bytes
90ce3da70b43 Initial load
duke
parents:
diff changeset
   156
    private String altSignerClass = null;
90ce3da70b43 Initial load
duke
parents:
diff changeset
   157
    private String altSignerClasspath = null;
90ce3da70b43 Initial load
duke
parents:
diff changeset
   158
    private ZipFile zipFile = null;
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   159
22315
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   160
    // Informational warnings
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   161
    private boolean hasExpiringCert = false;
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   162
    private boolean noTimestamp = false;
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   163
    private Date expireDate = new Date(0L);     // used in noTimestamp warning
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   164
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   165
    // Severe warnings
34382
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
   166
    private int weakAlg = 0; // 1. digestalg, 2. sigalg, 4. tsadigestalg
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   167
    private boolean hasExpiredCert = false;
90ce3da70b43 Initial load
duke
parents:
diff changeset
   168
    private boolean notYetValidCert = false;
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   169
    private boolean chainNotValidated = false;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   170
    private boolean notSignedByAlias = false;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   171
    private boolean aliasNotInStore = false;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   172
    private boolean hasUnsignedEntry = false;
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   173
    private boolean badKeyUsage = false;
90ce3da70b43 Initial load
duke
parents:
diff changeset
   174
    private boolean badExtendedKeyUsage = false;
90ce3da70b43 Initial load
duke
parents:
diff changeset
   175
    private boolean badNetscapeCertType = false;
34382
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
   176
    private boolean signerSelfSigned = false;
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
   177
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
   178
    private Throwable chainNotValidatedReason = null;
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   179
41590
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   180
    private boolean seeWeak = false;
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   181
43183
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
   182
    PKIXBuilderParameters pkixParameters;
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   183
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   184
    public void run(String args[]) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   185
        try {
24868
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   186
            args = parseArgs(args);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   187
90ce3da70b43 Initial load
duke
parents:
diff changeset
   188
            // Try to load and install the specified providers
90ce3da70b43 Initial load
duke
parents:
diff changeset
   189
            if (providers != null) {
39633
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   190
                for (String provName: providers) {
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   191
                    try {
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   192
                        KeyStoreUtil.loadProviderByName(provName,
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   193
                                providerArgs.get(provName));
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   194
                        if (debug) {
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   195
                            System.out.println("loadProviderByName: " + provName);
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   196
                        }
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   197
                    } catch (IllegalArgumentException e) {
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   198
                        throw new Exception(String.format(rb.getString(
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   199
                                "provider.name.not.found"), provName));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   200
                    }
39633
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   201
                }
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   202
            }
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   203
39633
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   204
            if (providerClasses != null) {
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   205
                ClassLoader cl = ClassLoader.getSystemClassLoader();
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   206
                for (String provClass: providerClasses) {
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   207
                    try {
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   208
                        KeyStoreUtil.loadProviderByClass(provClass,
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   209
                                providerArgs.get(provClass), cl);
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   210
                        if (debug) {
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   211
                            System.out.println("loadProviderByClass: " + provClass);
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   212
                        }
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   213
                    } catch (ClassCastException cce) {
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   214
                        throw new Exception(String.format(rb.getString(
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   215
                                "provclass.not.a.provider"), provClass));
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   216
                    } catch (IllegalArgumentException e) {
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   217
                        throw new Exception(String.format(rb.getString(
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   218
                                "provider.class.not.found"), provClass), e.getCause());
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   219
                    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   220
                }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   221
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   222
90ce3da70b43 Initial load
duke
parents:
diff changeset
   223
            if (verify) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   224
                try {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   225
                    loadKeyStore(keystore, false);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   226
                } catch (Exception e) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   227
                    if ((keystore != null) || (storepass != null)) {
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   228
                        System.out.println(rb.getString("jarsigner.error.") +
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   229
                                        e.getMessage());
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
   230
                        if (debug) {
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
   231
                            e.printStackTrace();
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
   232
                        }
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   233
                        System.exit(1);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   234
                    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   235
                }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   236
                /*              if (debug) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   237
                    SignatureFileVerifier.setDebug(true);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   238
                    ManifestEntryVerifier.setDebug(true);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   239
                }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   240
                */
90ce3da70b43 Initial load
duke
parents:
diff changeset
   241
                verifyJar(jarfile);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   242
            } else {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   243
                loadKeyStore(keystore, true);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   244
                getAliasInfo(alias);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   245
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
   246
                signJar(jarfile, alias);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   247
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   248
        } catch (Exception e) {
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   249
            System.out.println(rb.getString("jarsigner.error.") + e);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   250
            if (debug) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   251
                e.printStackTrace();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   252
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   253
            System.exit(1);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   254
        } finally {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   255
            // zero-out private key password
90ce3da70b43 Initial load
duke
parents:
diff changeset
   256
            if (keypass != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   257
                Arrays.fill(keypass, ' ');
90ce3da70b43 Initial load
duke
parents:
diff changeset
   258
                keypass = null;
90ce3da70b43 Initial load
duke
parents:
diff changeset
   259
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   260
            // zero-out keystore password
90ce3da70b43 Initial load
duke
parents:
diff changeset
   261
            if (storepass != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   262
                Arrays.fill(storepass, ' ');
90ce3da70b43 Initial load
duke
parents:
diff changeset
   263
                storepass = null;
90ce3da70b43 Initial load
duke
parents:
diff changeset
   264
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   265
        }
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   266
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   267
        if (strict) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   268
            int exitCode = 0;
34382
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
   269
            if (weakAlg != 0 || chainNotValidated || hasExpiredCert || notYetValidCert || signerSelfSigned) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   270
                exitCode |= 4;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   271
            }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   272
            if (badKeyUsage || badExtendedKeyUsage || badNetscapeCertType) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   273
                exitCode |= 8;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   274
            }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   275
            if (hasUnsignedEntry) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   276
                exitCode |= 16;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   277
            }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   278
            if (notSignedByAlias || aliasNotInStore) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   279
                exitCode |= 32;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   280
            }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   281
            if (exitCode != 0) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   282
                System.exit(exitCode);
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   283
            }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   284
        }
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   285
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   286
90ce3da70b43 Initial load
duke
parents:
diff changeset
   287
    /*
90ce3da70b43 Initial load
duke
parents:
diff changeset
   288
     * Parse command line arguments.
90ce3da70b43 Initial load
duke
parents:
diff changeset
   289
     */
24868
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   290
    String[] parseArgs(String args[]) throws Exception {
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   291
        /* parse flags */
90ce3da70b43 Initial load
duke
parents:
diff changeset
   292
        int n = 0;
90ce3da70b43 Initial load
duke
parents:
diff changeset
   293
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   294
        if (args.length == 0) fullusage();
24868
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   295
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   296
        String confFile = null;
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   297
        String command = "-sign";
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   298
        for (n=0; n < args.length; n++) {
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   299
            if (collator.compare(args[n], "-verify") == 0) {
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   300
                command = "-verify";
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   301
            } else if (collator.compare(args[n], "-conf") == 0) {
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   302
                if (n == args.length - 1) {
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   303
                    usageNoArg();
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   304
                }
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   305
                confFile = args[++n];
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   306
            }
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   307
        }
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   308
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   309
        if (confFile != null) {
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   310
            args = KeyStoreUtil.expandArgs(
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   311
                    "jarsigner", confFile, command, null, args);
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   312
        }
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   313
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   314
        debug = Arrays.stream(args).anyMatch(
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   315
                x -> collator.compare(x, "-debug") == 0);
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   316
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   317
        if (debug) {
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   318
            // No need to localize debug output
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   319
            System.out.println("Command line args: " +
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   320
                    Arrays.toString(args));
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   321
        }
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   322
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   323
        for (n=0; n < args.length; n++) {
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   324
90ce3da70b43 Initial load
duke
parents:
diff changeset
   325
            String flags = args[n];
3951
e821908c953e 6868579: RFE: jarsigner to support reading password from environment variable
weijun
parents: 3716
diff changeset
   326
            String modifier = null;
19189
a4b8478a2bc5 8021789: jarsigner parses alias as command line option (depending on locale)
weijun
parents: 17161
diff changeset
   327
a4b8478a2bc5 8021789: jarsigner parses alias as command line option (depending on locale)
weijun
parents: 17161
diff changeset
   328
            if (flags.startsWith("-")) {
3951
e821908c953e 6868579: RFE: jarsigner to support reading password from environment variable
weijun
parents: 3716
diff changeset
   329
                int pos = flags.indexOf(':');
e821908c953e 6868579: RFE: jarsigner to support reading password from environment variable
weijun
parents: 3716
diff changeset
   330
                if (pos > 0) {
e821908c953e 6868579: RFE: jarsigner to support reading password from environment variable
weijun
parents: 3716
diff changeset
   331
                    modifier = flags.substring(pos+1);
e821908c953e 6868579: RFE: jarsigner to support reading password from environment variable
weijun
parents: 3716
diff changeset
   332
                    flags = flags.substring(0, pos);
e821908c953e 6868579: RFE: jarsigner to support reading password from environment variable
weijun
parents: 3716
diff changeset
   333
                }
e821908c953e 6868579: RFE: jarsigner to support reading password from environment variable
weijun
parents: 3716
diff changeset
   334
            }
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   335
19189
a4b8478a2bc5 8021789: jarsigner parses alias as command line option (depending on locale)
weijun
parents: 17161
diff changeset
   336
            if (!flags.startsWith("-")) {
a4b8478a2bc5 8021789: jarsigner parses alias as command line option (depending on locale)
weijun
parents: 17161
diff changeset
   337
                if (jarfile == null) {
a4b8478a2bc5 8021789: jarsigner parses alias as command line option (depending on locale)
weijun
parents: 17161
diff changeset
   338
                    jarfile = flags;
a4b8478a2bc5 8021789: jarsigner parses alias as command line option (depending on locale)
weijun
parents: 17161
diff changeset
   339
                } else {
a4b8478a2bc5 8021789: jarsigner parses alias as command line option (depending on locale)
weijun
parents: 17161
diff changeset
   340
                    alias = flags;
a4b8478a2bc5 8021789: jarsigner parses alias as command line option (depending on locale)
weijun
parents: 17161
diff changeset
   341
                    ckaliases.add(alias);
a4b8478a2bc5 8021789: jarsigner parses alias as command line option (depending on locale)
weijun
parents: 17161
diff changeset
   342
                }
24868
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   343
            } else if (collator.compare(flags, "-conf") == 0) {
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   344
                if (++n == args.length) usageNoArg();
19189
a4b8478a2bc5 8021789: jarsigner parses alias as command line option (depending on locale)
weijun
parents: 17161
diff changeset
   345
            } else if (collator.compare(flags, "-keystore") == 0) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   346
                if (++n == args.length) usageNoArg();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   347
                keystore = args[n];
90ce3da70b43 Initial load
duke
parents:
diff changeset
   348
            } else if (collator.compare(flags, "-storepass") ==0) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   349
                if (++n == args.length) usageNoArg();
3951
e821908c953e 6868579: RFE: jarsigner to support reading password from environment variable
weijun
parents: 3716
diff changeset
   350
                storepass = getPass(modifier, args[n]);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   351
            } else if (collator.compare(flags, "-storetype") ==0) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   352
                if (++n == args.length) usageNoArg();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   353
                storetype = args[n];
90ce3da70b43 Initial load
duke
parents:
diff changeset
   354
            } else if (collator.compare(flags, "-providerName") ==0) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   355
                if (++n == args.length) usageNoArg();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   356
                providerName = args[n];
39633
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   357
            } else if (collator.compare(flags, "-provider") == 0 ||
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   358
                        collator.compare(flags, "-providerClass") == 0) {
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   359
                if (++n == args.length) usageNoArg();
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   360
                if (providerClasses == null) {
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   361
                    providerClasses = new ArrayList<>(3);
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   362
                }
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   363
                providerClasses.add(args[n]);
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   364
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   365
                if (args.length > (n+1)) {
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   366
                    flags = args[n+1];
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   367
                    if (collator.compare(flags, "-providerArg") == 0) {
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   368
                        if (args.length == (n+2)) usageNoArg();
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   369
                        providerArgs.put(args[n], args[n+2]);
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   370
                        n += 2;
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   371
                    }
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   372
                }
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   373
            } else if (collator.compare(flags, "-addprovider") == 0) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   374
                if (++n == args.length) usageNoArg();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   375
                if (providers == null) {
39633
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   376
                    providers = new ArrayList<>(3);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   377
                }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   378
                providers.add(args[n]);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   379
90ce3da70b43 Initial load
duke
parents:
diff changeset
   380
                if (args.length > (n+1)) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   381
                    flags = args[n+1];
90ce3da70b43 Initial load
duke
parents:
diff changeset
   382
                    if (collator.compare(flags, "-providerArg") == 0) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   383
                        if (args.length == (n+2)) usageNoArg();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   384
                        providerArgs.put(args[n], args[n+2]);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   385
                        n += 2;
90ce3da70b43 Initial load
duke
parents:
diff changeset
   386
                    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   387
                }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   388
            } else if (collator.compare(flags, "-protected") ==0) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   389
                protectedPath = true;
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   390
            } else if (collator.compare(flags, "-certchain") ==0) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   391
                if (++n == args.length) usageNoArg();
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   392
                altCertChain = args[n];
17161
df1ec0e2f0e7 8009636: JARSigner including TimeStamp PolicyID (TSAPolicyID) as defined in RFC3161
weijun
parents: 16020
diff changeset
   393
            } else if (collator.compare(flags, "-tsapolicyid") ==0) {
df1ec0e2f0e7 8009636: JARSigner including TimeStamp PolicyID (TSAPolicyID) as defined in RFC3161
weijun
parents: 16020
diff changeset
   394
                if (++n == args.length) usageNoArg();
df1ec0e2f0e7 8009636: JARSigner including TimeStamp PolicyID (TSAPolicyID) as defined in RFC3161
weijun
parents: 16020
diff changeset
   395
                tSAPolicyID = args[n];
24034
31fe17eef94a 8038837: Add support to jarsigner for specifying timestamp hash algorithm
weijun
parents: 23912
diff changeset
   396
            } else if (collator.compare(flags, "-tsadigestalg") ==0) {
31fe17eef94a 8038837: Add support to jarsigner for specifying timestamp hash algorithm
weijun
parents: 23912
diff changeset
   397
                if (++n == args.length) usageNoArg();
31fe17eef94a 8038837: Add support to jarsigner for specifying timestamp hash algorithm
weijun
parents: 23912
diff changeset
   398
                tSADigestAlg = args[n];
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   399
            } else if (collator.compare(flags, "-debug") ==0) {
24868
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   400
                // Already processed
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   401
            } else if (collator.compare(flags, "-keypass") ==0) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   402
                if (++n == args.length) usageNoArg();
3951
e821908c953e 6868579: RFE: jarsigner to support reading password from environment variable
weijun
parents: 3716
diff changeset
   403
                keypass = getPass(modifier, args[n]);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   404
            } else if (collator.compare(flags, "-sigfile") ==0) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   405
                if (++n == args.length) usageNoArg();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   406
                sigfile = args[n];
90ce3da70b43 Initial load
duke
parents:
diff changeset
   407
            } else if (collator.compare(flags, "-signedjar") ==0) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   408
                if (++n == args.length) usageNoArg();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   409
                signedjar = args[n];
90ce3da70b43 Initial load
duke
parents:
diff changeset
   410
            } else if (collator.compare(flags, "-tsa") ==0) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   411
                if (++n == args.length) usageNoArg();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   412
                tsaUrl = args[n];
90ce3da70b43 Initial load
duke
parents:
diff changeset
   413
            } else if (collator.compare(flags, "-tsacert") ==0) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   414
                if (++n == args.length) usageNoArg();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   415
                tsaAlias = args[n];
90ce3da70b43 Initial load
duke
parents:
diff changeset
   416
            } else if (collator.compare(flags, "-altsigner") ==0) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   417
                if (++n == args.length) usageNoArg();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   418
                altSignerClass = args[n];
31060
be4eb6360ee0 8076535: Deprecate the com.sun.jarsigner package
juh
parents: 29369
diff changeset
   419
                System.err.println(
be4eb6360ee0 8076535: Deprecate the com.sun.jarsigner package
juh
parents: 29369
diff changeset
   420
                        rb.getString("This.option.is.deprecated") +
be4eb6360ee0 8076535: Deprecate the com.sun.jarsigner package
juh
parents: 29369
diff changeset
   421
                                "-altsigner");
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   422
            } else if (collator.compare(flags, "-altsignerpath") ==0) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   423
                if (++n == args.length) usageNoArg();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   424
                altSignerClasspath = args[n];
31060
be4eb6360ee0 8076535: Deprecate the com.sun.jarsigner package
juh
parents: 29369
diff changeset
   425
                System.err.println(
be4eb6360ee0 8076535: Deprecate the com.sun.jarsigner package
juh
parents: 29369
diff changeset
   426
                        rb.getString("This.option.is.deprecated") +
be4eb6360ee0 8076535: Deprecate the com.sun.jarsigner package
juh
parents: 29369
diff changeset
   427
                                "-altsignerpath");
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   428
            } else if (collator.compare(flags, "-sectionsonly") ==0) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   429
                signManifest = false;
90ce3da70b43 Initial load
duke
parents:
diff changeset
   430
            } else if (collator.compare(flags, "-internalsf") ==0) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   431
                externalSF = false;
90ce3da70b43 Initial load
duke
parents:
diff changeset
   432
            } else if (collator.compare(flags, "-verify") ==0) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   433
                verify = true;
90ce3da70b43 Initial load
duke
parents:
diff changeset
   434
            } else if (collator.compare(flags, "-verbose") ==0) {
3951
e821908c953e 6868579: RFE: jarsigner to support reading password from environment variable
weijun
parents: 3716
diff changeset
   435
                verbose = (modifier != null) ? modifier : "all";
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   436
            } else if (collator.compare(flags, "-sigalg") ==0) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   437
                if (++n == args.length) usageNoArg();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   438
                sigalg = args[n];
90ce3da70b43 Initial load
duke
parents:
diff changeset
   439
            } else if (collator.compare(flags, "-digestalg") ==0) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   440
                if (++n == args.length) usageNoArg();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   441
                digestalg = args[n];
90ce3da70b43 Initial load
duke
parents:
diff changeset
   442
            } else if (collator.compare(flags, "-certs") ==0) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   443
                showcerts = true;
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   444
            } else if (collator.compare(flags, "-strict") ==0) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   445
                strict = true;
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   446
            } else if (collator.compare(flags, "-h") == 0 ||
44597
98f0c39ce3a8 8172422: jarsigner needs to understand -?
weijun
parents: 43183
diff changeset
   447
                        collator.compare(flags, "-?") == 0 ||
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   448
                        collator.compare(flags, "-help") == 0) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   449
                fullusage();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   450
            } else {
19189
a4b8478a2bc5 8021789: jarsigner parses alias as command line option (depending on locale)
weijun
parents: 17161
diff changeset
   451
                System.err.println(
a4b8478a2bc5 8021789: jarsigner parses alias as command line option (depending on locale)
weijun
parents: 17161
diff changeset
   452
                        rb.getString("Illegal.option.") + flags);
a4b8478a2bc5 8021789: jarsigner parses alias as command line option (depending on locale)
weijun
parents: 17161
diff changeset
   453
                usage();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   454
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   455
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   456
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   457
        // -certs must always be specified with -verbose
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   458
        if (verbose == null) showcerts = false;
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   459
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   460
        if (jarfile == null) {
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   461
            System.err.println(rb.getString("Please.specify.jarfile.name"));
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   462
            usage();
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   463
        }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   464
        if (!verify && alias == null) {
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   465
            System.err.println(rb.getString("Please.specify.alias.name"));
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   466
            usage();
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   467
        }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   468
        if (!verify && ckaliases.size() > 1) {
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   469
            System.err.println(rb.getString("Only.one.alias.can.be.specified"));
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   470
            usage();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   471
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   472
90ce3da70b43 Initial load
duke
parents:
diff changeset
   473
        if (storetype == null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   474
            storetype = KeyStore.getDefaultType();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   475
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   476
        storetype = KeyStoreUtil.niceStoreTypeName(storetype);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   477
3481
6ae7a2a6c956 6866479: libzip.so caused JVM to crash when running jarsigner
weijun
parents: 3318
diff changeset
   478
        try {
6ae7a2a6c956 6866479: libzip.so caused JVM to crash when running jarsigner
weijun
parents: 3318
diff changeset
   479
            if (signedjar != null && new File(signedjar).getCanonicalPath().equals(
6ae7a2a6c956 6866479: libzip.so caused JVM to crash when running jarsigner
weijun
parents: 3318
diff changeset
   480
                    new File(jarfile).getCanonicalPath())) {
6ae7a2a6c956 6866479: libzip.so caused JVM to crash when running jarsigner
weijun
parents: 3318
diff changeset
   481
                signedjar = null;
6ae7a2a6c956 6866479: libzip.so caused JVM to crash when running jarsigner
weijun
parents: 3318
diff changeset
   482
            }
6ae7a2a6c956 6866479: libzip.so caused JVM to crash when running jarsigner
weijun
parents: 3318
diff changeset
   483
        } catch (IOException ioe) {
6ae7a2a6c956 6866479: libzip.so caused JVM to crash when running jarsigner
weijun
parents: 3318
diff changeset
   484
            // File system error?
6ae7a2a6c956 6866479: libzip.so caused JVM to crash when running jarsigner
weijun
parents: 3318
diff changeset
   485
            // Just ignore it.
6ae7a2a6c956 6866479: libzip.so caused JVM to crash when running jarsigner
weijun
parents: 3318
diff changeset
   486
        }
6ae7a2a6c956 6866479: libzip.so caused JVM to crash when running jarsigner
weijun
parents: 3318
diff changeset
   487
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   488
        if (P11KEYSTORE.equalsIgnoreCase(storetype) ||
90ce3da70b43 Initial load
duke
parents:
diff changeset
   489
                KeyStoreUtil.isWindowsKeyStore(storetype)) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   490
            token = true;
90ce3da70b43 Initial load
duke
parents:
diff changeset
   491
            if (keystore == null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   492
                keystore = NONE;
90ce3da70b43 Initial load
duke
parents:
diff changeset
   493
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   494
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   495
90ce3da70b43 Initial load
duke
parents:
diff changeset
   496
        if (NONE.equals(keystore)) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   497
            nullStream = true;
90ce3da70b43 Initial load
duke
parents:
diff changeset
   498
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   499
90ce3da70b43 Initial load
duke
parents:
diff changeset
   500
        if (token && !nullStream) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   501
            System.err.println(MessageFormat.format(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   502
                (".keystore.must.be.NONE.if.storetype.is.{0}"), storetype));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   503
            usage();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   504
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   505
90ce3da70b43 Initial load
duke
parents:
diff changeset
   506
        if (token && keypass != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   507
            System.err.println(MessageFormat.format(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   508
                (".keypass.can.not.be.specified.if.storetype.is.{0}"), storetype));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   509
            usage();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   510
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   511
90ce3da70b43 Initial load
duke
parents:
diff changeset
   512
        if (protectedPath) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   513
            if (storepass != null || keypass != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   514
                System.err.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   515
                        ("If.protected.is.specified.then.storepass.and.keypass.must.not.be.specified"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   516
                usage();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   517
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   518
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   519
        if (KeyStoreUtil.isWindowsKeyStore(storetype)) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   520
            if (storepass != null || keypass != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   521
                System.err.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   522
                        ("If.keystore.is.not.password.protected.then.storepass.and.keypass.must.not.be.specified"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   523
                usage();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   524
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   525
        }
24868
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   526
        return args;
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   527
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   528
3951
e821908c953e 6868579: RFE: jarsigner to support reading password from environment variable
weijun
parents: 3716
diff changeset
   529
    static char[] getPass(String modifier, String arg) {
14182
3041082abb40 7194449: String resources for Key Tool and Policy Tool should be in their respective packages
sflores
parents: 12046
diff changeset
   530
        char[] output = KeyStoreUtil.getPassWithModifier(modifier, arg, rb);
3951
e821908c953e 6868579: RFE: jarsigner to support reading password from environment variable
weijun
parents: 3716
diff changeset
   531
        if (output != null) return output;
e821908c953e 6868579: RFE: jarsigner to support reading password from environment variable
weijun
parents: 3716
diff changeset
   532
        usage();
e821908c953e 6868579: RFE: jarsigner to support reading password from environment variable
weijun
parents: 3716
diff changeset
   533
        return null;    // Useless, usage() already exit
e821908c953e 6868579: RFE: jarsigner to support reading password from environment variable
weijun
parents: 3716
diff changeset
   534
    }
e821908c953e 6868579: RFE: jarsigner to support reading password from environment variable
weijun
parents: 3716
diff changeset
   535
e821908c953e 6868579: RFE: jarsigner to support reading password from environment variable
weijun
parents: 3716
diff changeset
   536
    static void usageNoArg() {
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   537
        System.out.println(rb.getString("Option.lacks.argument"));
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   538
        usage();
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   539
    }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   540
3951
e821908c953e 6868579: RFE: jarsigner to support reading password from environment variable
weijun
parents: 3716
diff changeset
   541
    static void usage() {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   542
        System.out.println();
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   543
        System.out.println(rb.getString("Please.type.jarsigner.help.for.usage"));
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   544
        System.exit(1);
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   545
    }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   546
3951
e821908c953e 6868579: RFE: jarsigner to support reading password from environment variable
weijun
parents: 3716
diff changeset
   547
    static void fullusage() {
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   548
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   549
                ("Usage.jarsigner.options.jar.file.alias"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   550
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   551
                (".jarsigner.verify.options.jar.file.alias."));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   552
        System.out.println();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   553
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   554
                (".keystore.url.keystore.location"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   555
        System.out.println();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   556
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   557
                (".storepass.password.password.for.keystore.integrity"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   558
        System.out.println();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   559
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   560
                (".storetype.type.keystore.type"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   561
        System.out.println();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   562
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   563
                (".keypass.password.password.for.private.key.if.different."));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   564
        System.out.println();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   565
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   566
                (".certchain.file.name.of.alternative.certchain.file"));
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   567
        System.out.println();
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   568
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   569
                (".sigfile.file.name.of.SF.DSA.file"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   570
        System.out.println();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   571
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   572
                (".signedjar.file.name.of.signed.JAR.file"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   573
        System.out.println();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   574
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   575
                (".digestalg.algorithm.name.of.digest.algorithm"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   576
        System.out.println();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   577
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   578
                (".sigalg.algorithm.name.of.signature.algorithm"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   579
        System.out.println();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   580
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   581
                (".verify.verify.a.signed.JAR.file"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   582
        System.out.println();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   583
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   584
                (".verbose.suboptions.verbose.output.when.signing.verifying."));
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   585
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   586
                (".suboptions.can.be.all.grouped.or.summary"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   587
        System.out.println();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   588
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   589
                (".certs.display.certificates.when.verbose.and.verifying"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   590
        System.out.println();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   591
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   592
                (".tsa.url.location.of.the.Timestamping.Authority"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   593
        System.out.println();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   594
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   595
                (".tsacert.alias.public.key.certificate.for.Timestamping.Authority"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   596
        System.out.println();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   597
        System.out.println(rb.getString
17161
df1ec0e2f0e7 8009636: JARSigner including TimeStamp PolicyID (TSAPolicyID) as defined in RFC3161
weijun
parents: 16020
diff changeset
   598
                (".tsapolicyid.tsapolicyid.for.Timestamping.Authority"));
df1ec0e2f0e7 8009636: JARSigner including TimeStamp PolicyID (TSAPolicyID) as defined in RFC3161
weijun
parents: 16020
diff changeset
   599
        System.out.println();
df1ec0e2f0e7 8009636: JARSigner including TimeStamp PolicyID (TSAPolicyID) as defined in RFC3161
weijun
parents: 16020
diff changeset
   600
        System.out.println(rb.getString
24034
31fe17eef94a 8038837: Add support to jarsigner for specifying timestamp hash algorithm
weijun
parents: 23912
diff changeset
   601
                (".tsadigestalg.algorithm.of.digest.data.in.timestamping.request"));
31fe17eef94a 8038837: Add support to jarsigner for specifying timestamp hash algorithm
weijun
parents: 23912
diff changeset
   602
        System.out.println();
31fe17eef94a 8038837: Add support to jarsigner for specifying timestamp hash algorithm
weijun
parents: 23912
diff changeset
   603
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   604
                (".altsigner.class.class.name.of.an.alternative.signing.mechanism"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   605
        System.out.println();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   606
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   607
                (".altsignerpath.pathlist.location.of.an.alternative.signing.mechanism"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   608
        System.out.println();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   609
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   610
                (".internalsf.include.the.SF.file.inside.the.signature.block"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   611
        System.out.println();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   612
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   613
                (".sectionsonly.don.t.compute.hash.of.entire.manifest"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   614
        System.out.println();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   615
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   616
                (".protected.keystore.has.protected.authentication.path"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   617
        System.out.println();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   618
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   619
                (".providerName.name.provider.name"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   620
        System.out.println();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   621
        System.out.println(rb.getString
39633
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   622
                (".add.provider.option"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   623
        System.out.println(rb.getString
39633
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   624
                (".providerArg.option.1"));
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   625
        System.out.println();
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   626
        System.out.println(rb.getString
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   627
                (".providerClass.option"));
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   628
        System.out.println(rb.getString
9dc7586be5f0 8130302: jarsigner and keytool -providerClass needs be re-examined for modules
weijun
parents: 34382
diff changeset
   629
                (".providerArg.option.2"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   630
        System.out.println();
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   631
        System.out.println(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   632
                (".strict.treat.warnings.as.errors"));
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   633
        System.out.println();
24868
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   634
        System.out.println(rb.getString
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   635
                (".conf.url.specify.a.pre.configured.options.file"));
89d9bd9eba96 8023197: Pre-configured command line options for keytool and jarsigner
weijun
parents: 24625
diff changeset
   636
        System.out.println();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   637
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   638
        System.exit(0);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   639
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   640
90ce3da70b43 Initial load
duke
parents:
diff changeset
   641
    void verifyJar(String jarName)
90ce3da70b43 Initial load
duke
parents:
diff changeset
   642
        throws Exception
90ce3da70b43 Initial load
duke
parents:
diff changeset
   643
    {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   644
        boolean anySigned = false;  // if there exists entry inside jar signed
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   645
        JarFile jf = null;
41590
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   646
        Map<String,String> digestMap = new HashMap<>();
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   647
        Map<String,PKCS7> sigMap = new HashMap<>();
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   648
        Map<String,String> sigNameMap = new HashMap<>();
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   649
        Map<String,String> unparsableSignatures = new HashMap<>();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   650
90ce3da70b43 Initial load
duke
parents:
diff changeset
   651
        try {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   652
            jf = new JarFile(jarName, true);
7977
f47f211cd627 7008713: diamond conversion of kerberos5 and security tools
smarks
parents: 7525
diff changeset
   653
            Vector<JarEntry> entriesVec = new Vector<>();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   654
            byte[] buffer = new byte[8192];
90ce3da70b43 Initial load
duke
parents:
diff changeset
   655
90ce3da70b43 Initial load
duke
parents:
diff changeset
   656
            Enumeration<JarEntry> entries = jf.entries();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   657
            while (entries.hasMoreElements()) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   658
                JarEntry je = entries.nextElement();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   659
                entriesVec.addElement(je);
41590
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   660
                try (InputStream is = jf.getInputStream(je)) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   661
                    String name = je.getName();
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   662
                    if (signatureRelated(name)
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   663
                            && SignatureFileVerifier.isBlockOrSF(name)) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   664
                        String alias = name.substring(name.lastIndexOf('/') + 1,
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   665
                                name.lastIndexOf('.'));
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   666
                        try {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   667
                            if (name.endsWith(".SF")) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   668
                                Manifest sf = new Manifest(is);
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   669
                                boolean found = false;
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   670
                                for (Object obj : sf.getMainAttributes().keySet()) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   671
                                    String key = obj.toString();
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   672
                                    if (key.endsWith("-Digest-Manifest")) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   673
                                        digestMap.put(alias,
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   674
                                                key.substring(0, key.length() - 16));
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   675
                                        found = true;
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   676
                                        break;
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   677
                                    }
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   678
                                }
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   679
                                if (!found) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   680
                                    unparsableSignatures.putIfAbsent(alias,
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   681
                                        String.format(
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   682
                                            rb.getString("history.unparsable"),
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   683
                                            name));
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   684
                                }
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   685
                            } else {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   686
                                sigNameMap.put(alias, name);
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   687
                                sigMap.put(alias, new PKCS7(is));
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   688
                            }
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   689
                        } catch (IOException ioe) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   690
                            unparsableSignatures.putIfAbsent(alias, String.format(
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   691
                                    rb.getString("history.unparsable"), name));
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   692
                        }
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   693
                    } else {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   694
                        while (is.read(buffer, 0, buffer.length) != -1) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   695
                            // we just read. this will throw a SecurityException
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   696
                            // if  a signature/digest check fails.
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   697
                        }
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   698
                    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   699
                }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   700
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   701
90ce3da70b43 Initial load
duke
parents:
diff changeset
   702
            Manifest man = jf.getManifest();
41582
246512d81eba 8165816: jarsigner -verify shows jar unsigned if it was signed with a weak algorithm
weijun
parents: 40177
diff changeset
   703
            boolean hasSignature = false;
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   704
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   705
            // The map to record display info, only used when -verbose provided
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   706
            //      key: signer info string
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   707
            //      value: the list of files with common key
7977
f47f211cd627 7008713: diamond conversion of kerberos5 and security tools
smarks
parents: 7525
diff changeset
   708
            Map<String,List<String>> output = new LinkedHashMap<>();
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   709
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   710
            if (man != null) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   711
                if (verbose != null) System.out.println();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   712
                Enumeration<JarEntry> e = entriesVec.elements();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   713
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   714
                String tab = rb.getString("6SPACE");
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   715
90ce3da70b43 Initial load
duke
parents:
diff changeset
   716
                while (e.hasMoreElements()) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   717
                    JarEntry je = e.nextElement();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   718
                    String name = je.getName();
41582
246512d81eba 8165816: jarsigner -verify shows jar unsigned if it was signed with a weak algorithm
weijun
parents: 40177
diff changeset
   719
246512d81eba 8165816: jarsigner -verify shows jar unsigned if it was signed with a weak algorithm
weijun
parents: 40177
diff changeset
   720
                    hasSignature = hasSignature
246512d81eba 8165816: jarsigner -verify shows jar unsigned if it was signed with a weak algorithm
weijun
parents: 40177
diff changeset
   721
                            || SignatureFileVerifier.isBlockOrSF(name);
246512d81eba 8165816: jarsigner -verify shows jar unsigned if it was signed with a weak algorithm
weijun
parents: 40177
diff changeset
   722
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   723
                    CodeSigner[] signers = je.getCodeSigners();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   724
                    boolean isSigned = (signers != null);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   725
                    anySigned |= isSigned;
90ce3da70b43 Initial load
duke
parents:
diff changeset
   726
                    hasUnsignedEntry |= !je.isDirectory() && !isSigned
90ce3da70b43 Initial load
duke
parents:
diff changeset
   727
                                        && !signatureRelated(name);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   728
40177
e2a7079bd50a 8163303: Remove identity scope information from jarsigner -verbose output
weijun
parents: 39633
diff changeset
   729
                    int inStoreWithAlias = inKeyStore(signers);
e2a7079bd50a 8163303: Remove identity scope information from jarsigner -verbose output
weijun
parents: 39633
diff changeset
   730
e2a7079bd50a 8163303: Remove identity scope information from jarsigner -verbose output
weijun
parents: 39633
diff changeset
   731
                    boolean inStore = (inStoreWithAlias & IN_KEYSTORE) != 0;
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   732
40177
e2a7079bd50a 8163303: Remove identity scope information from jarsigner -verbose output
weijun
parents: 39633
diff changeset
   733
                    notSignedByAlias |= (inStoreWithAlias & NOT_ALIAS) != 0;
7525
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
   734
                    if (keystore != null) {
40177
e2a7079bd50a 8163303: Remove identity scope information from jarsigner -verbose output
weijun
parents: 39633
diff changeset
   735
                        aliasNotInStore |= isSigned && !inStore;
7525
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
   736
                    }
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   737
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   738
                    // Only used when -verbose provided
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   739
                    StringBuffer sb = null;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   740
                    if (verbose != null) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   741
                        sb = new StringBuffer();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   742
                        boolean inManifest =
90ce3da70b43 Initial load
duke
parents:
diff changeset
   743
                            ((man.getAttributes(name) != null) ||
90ce3da70b43 Initial load
duke
parents:
diff changeset
   744
                             (man.getAttributes("./"+name) != null) ||
90ce3da70b43 Initial load
duke
parents:
diff changeset
   745
                             (man.getAttributes("/"+name) != null));
27957
24b4e6082f19 8055723: Replace concat String to append in StringBuilder parameters (dev)
weijun
parents: 27344
diff changeset
   746
                        sb.append(isSigned ? rb.getString("s") : rb.getString("SPACE"))
24b4e6082f19 8055723: Replace concat String to append in StringBuilder parameters (dev)
weijun
parents: 27344
diff changeset
   747
                                .append(inManifest ? rb.getString("m") : rb.getString("SPACE"))
24b4e6082f19 8055723: Replace concat String to append in StringBuilder parameters (dev)
weijun
parents: 27344
diff changeset
   748
                                .append(inStore ? rb.getString("k") : rb.getString("SPACE"))
40177
e2a7079bd50a 8163303: Remove identity scope information from jarsigner -verbose output
weijun
parents: 39633
diff changeset
   749
                                .append((inStoreWithAlias & NOT_ALIAS) != 0 ? 'X' : ' ')
27957
24b4e6082f19 8055723: Replace concat String to append in StringBuilder parameters (dev)
weijun
parents: 27344
diff changeset
   750
                                .append(rb.getString("SPACE"));
24b4e6082f19 8055723: Replace concat String to append in StringBuilder parameters (dev)
weijun
parents: 27344
diff changeset
   751
                        sb.append('|');
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   752
                    }
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   753
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   754
                    // When -certs provided, display info has extra empty
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   755
                    // lines at the beginning and end.
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   756
                    if (isSigned) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   757
                        if (showcerts) sb.append('\n');
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   758
                        for (CodeSigner signer: signers) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   759
                            // signerInfo() must be called even if -verbose
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   760
                            // not provided. The method updates various
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   761
                            // warning flags.
12046
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
   762
                            String si = signerInfo(signer, tab);
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   763
                            if (showcerts) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   764
                                sb.append(si);
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   765
                                sb.append('\n');
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   766
                            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   767
                        }
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   768
                    } else if (showcerts && !verbose.equals("all")) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   769
                        // Print no info for unsigned entries when -verbose:all,
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   770
                        // to be consistent with old behavior.
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   771
                        if (signatureRelated(name)) {
27957
24b4e6082f19 8055723: Replace concat String to append in StringBuilder parameters (dev)
weijun
parents: 27344
diff changeset
   772
                            sb.append('\n')
24b4e6082f19 8055723: Replace concat String to append in StringBuilder parameters (dev)
weijun
parents: 27344
diff changeset
   773
                                    .append(tab)
24b4e6082f19 8055723: Replace concat String to append in StringBuilder parameters (dev)
weijun
parents: 27344
diff changeset
   774
                                    .append(rb
24b4e6082f19 8055723: Replace concat String to append in StringBuilder parameters (dev)
weijun
parents: 27344
diff changeset
   775
                                            .getString(".Signature.related.entries."))
24b4e6082f19 8055723: Replace concat String to append in StringBuilder parameters (dev)
weijun
parents: 27344
diff changeset
   776
                                    .append("\n\n");
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   777
                        } else {
27957
24b4e6082f19 8055723: Replace concat String to append in StringBuilder parameters (dev)
weijun
parents: 27344
diff changeset
   778
                            sb.append('\n').append(tab)
24b4e6082f19 8055723: Replace concat String to append in StringBuilder parameters (dev)
weijun
parents: 27344
diff changeset
   779
                                    .append(rb.getString(".Unsigned.entries."))
24b4e6082f19 8055723: Replace concat String to append in StringBuilder parameters (dev)
weijun
parents: 27344
diff changeset
   780
                                    .append("\n\n");
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   781
                        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   782
                    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   783
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   784
                    if (verbose != null) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   785
                        String label = sb.toString();
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   786
                        if (signatureRelated(name)) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   787
                            // Entries inside META-INF and other unsigned
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   788
                            // entries are grouped separately.
7524
ec12e1e6fa20 7004035: signed jar with only META-INF/* inside is not verifiable
weijun
parents: 7179
diff changeset
   789
                            label = "-" + label;
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   790
                        }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   791
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   792
                        // The label finally contains 2 parts separated by '|':
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   793
                        // The legend displayed before the entry names, and
21278
ef8a3a2a72f2 8022746: List of spelling errors in API doc
malenkov
parents: 20754
diff changeset
   794
                        // the cert info (if -certs specified).
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   795
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   796
                        if (!output.containsKey(label)) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   797
                            output.put(label, new ArrayList<String>());
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   798
                        }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   799
24969
afa6934dd8e8 8041679: Replace uses of StringBuffer with StringBuilder within core library classes
psandoz
parents: 24868
diff changeset
   800
                        StringBuilder fb = new StringBuilder();
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   801
                        String s = Long.toString(je.getSize());
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   802
                        for (int i = 6 - s.length(); i > 0; --i) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   803
                            fb.append(' ');
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   804
                        }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   805
                        fb.append(s).append(' ').
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   806
                                append(new Date(je.getTime()).toString());
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   807
                        fb.append(' ').append(name);
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   808
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   809
                        output.get(label).add(fb.toString());
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   810
                    }
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   811
                }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   812
            }
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   813
            if (verbose != null) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   814
                for (Entry<String,List<String>> s: output.entrySet()) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   815
                    List<String> files = s.getValue();
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   816
                    String key = s.getKey();
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   817
                    if (key.charAt(0) == '-') { // the signature-related group
7524
ec12e1e6fa20 7004035: signed jar with only META-INF/* inside is not verifiable
weijun
parents: 7179
diff changeset
   818
                        key = key.substring(1);
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   819
                    }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   820
                    int pipe = key.indexOf('|');
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   821
                    if (verbose.equals("all")) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   822
                        for (String f: files) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   823
                            System.out.println(key.substring(0, pipe) + f);
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   824
                            System.out.printf(key.substring(pipe+1));
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   825
                        }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   826
                    } else {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   827
                        if (verbose.equals("grouped")) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   828
                            for (String f: files) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   829
                                System.out.println(key.substring(0, pipe) + f);
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   830
                            }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   831
                        } else if (verbose.equals("summary")) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   832
                            System.out.print(key.substring(0, pipe));
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   833
                            if (files.size() > 1) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   834
                                System.out.println(files.get(0) + " " +
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   835
                                        String.format(rb.getString(
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   836
                                        ".and.d.more."), files.size()-1));
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   837
                            } else {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   838
                                System.out.println(files.get(0));
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   839
                            }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   840
                        }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   841
                        System.out.printf(key.substring(pipe+1));
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   842
                    }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   843
                }
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   844
                System.out.println();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   845
                System.out.println(rb.getString(
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   846
                    ".s.signature.was.verified."));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   847
                System.out.println(rb.getString(
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   848
                    ".m.entry.is.listed.in.manifest"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   849
                System.out.println(rb.getString(
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   850
                    ".k.at.least.one.certificate.was.found.in.keystore"));
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   851
                if (ckaliases.size() > 0) {
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   852
                    System.out.println(rb.getString(
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   853
                        ".X.not.signed.by.specified.alias.es."));
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
   854
                }
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   855
            }
41590
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   856
            if (man == null) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   857
                System.out.println();
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   858
                System.out.println(rb.getString("no.manifest."));
41590
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   859
            }
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   860
34382
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
   861
            // If signer is a trusted cert or private entry in user's own
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
   862
            // keystore, it can be self-signed.
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
   863
            if (!aliasNotInStore) {
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
   864
                signerSelfSigned = false;
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
   865
            }
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
   866
41590
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   867
            // Even if the verbose option is not specified, all out strings
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   868
            // must be generated so seeWeak can be updated.
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   869
            if (!digestMap.isEmpty()
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   870
                    || !sigMap.isEmpty()
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   871
                    || !unparsableSignatures.isEmpty()) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   872
                if (verbose != null) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   873
                    System.out.println();
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   874
                }
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   875
                for (String s : sigMap.keySet()) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   876
                    if (!digestMap.containsKey(s)) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   877
                        unparsableSignatures.putIfAbsent(s, String.format(
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   878
                                rb.getString("history.nosf"), s));
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   879
                    }
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   880
                }
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   881
                for (String s : digestMap.keySet()) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   882
                    PKCS7 p7 = sigMap.get(s);
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   883
                    if (p7 != null) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   884
                        String history;
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   885
                        try {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   886
                            SignerInfo si = p7.getSignerInfos()[0];
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   887
                            X509Certificate signer = si.getCertificate(p7);
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   888
                            String digestAlg = digestMap.get(s);
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   889
                            String sigAlg = AlgorithmId.makeSigAlg(
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   890
                                    si.getDigestAlgorithmId().getName(),
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   891
                                    si.getDigestEncryptionAlgorithmId().getName());
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   892
                            PublicKey key = signer.getPublicKey();
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   893
                            PKCS7 tsToken = si.getTsToken();
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   894
                            if (tsToken != null) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   895
                                SignerInfo tsSi = tsToken.getSignerInfos()[0];
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   896
                                X509Certificate tsSigner = tsSi.getCertificate(tsToken);
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   897
                                byte[] encTsTokenInfo = tsToken.getContentInfo().getData();
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   898
                                TimestampToken tsTokenInfo = new TimestampToken(encTsTokenInfo);
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   899
                                PublicKey tsKey = tsSigner.getPublicKey();
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   900
                                String tsDigestAlg = tsTokenInfo.getHashAlgorithm().getName();
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   901
                                String tsSigAlg = AlgorithmId.makeSigAlg(
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   902
                                        tsSi.getDigestAlgorithmId().getName(),
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   903
                                        tsSi.getDigestEncryptionAlgorithmId().getName());
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   904
                                Calendar c = Calendar.getInstance(
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   905
                                        TimeZone.getTimeZone("UTC"),
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   906
                                        Locale.getDefault(Locale.Category.FORMAT));
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   907
                                c.setTime(tsTokenInfo.getDate());
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   908
                                history = String.format(
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   909
                                        rb.getString("history.with.ts"),
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   910
                                        signer.getSubjectX500Principal(),
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   911
                                        withWeak(digestAlg, DIGEST_PRIMITIVE_SET),
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   912
                                        withWeak(sigAlg, SIG_PRIMITIVE_SET),
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   913
                                        withWeak(key),
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   914
                                        c,
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   915
                                        tsSigner.getSubjectX500Principal(),
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   916
                                        withWeak(tsDigestAlg, DIGEST_PRIMITIVE_SET),
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   917
                                        withWeak(tsSigAlg, SIG_PRIMITIVE_SET),
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   918
                                        withWeak(tsKey));
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   919
                            } else {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   920
                                history = String.format(
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   921
                                        rb.getString("history.without.ts"),
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   922
                                        signer.getSubjectX500Principal(),
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   923
                                        withWeak(digestAlg, DIGEST_PRIMITIVE_SET),
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   924
                                        withWeak(sigAlg, SIG_PRIMITIVE_SET),
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   925
                                        withWeak(key));
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   926
                            }
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   927
                        } catch (Exception e) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   928
                            // The only usage of sigNameMap, remember the name
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   929
                            // of the block file if it's invalid.
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   930
                            history = String.format(
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   931
                                    rb.getString("history.unparsable"),
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   932
                                    sigNameMap.get(s));
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   933
                        }
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   934
                        if (verbose != null) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   935
                            System.out.println(history);
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   936
                        }
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   937
                    } else {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   938
                        unparsableSignatures.putIfAbsent(s, String.format(
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   939
                                rb.getString("history.nobk"), s));
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   940
                    }
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   941
                }
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   942
                if (verbose != null) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   943
                    for (String s : unparsableSignatures.keySet()) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   944
                        System.out.println(unparsableSignatures.get(s));
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   945
                    }
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   946
                }
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   947
            }
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   948
            System.out.println();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   949
            if (!anySigned) {
41590
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   950
                if (seeWeak) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   951
                    if (verbose != null) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   952
                        System.out.println(rb.getString("jar.treated.unsigned.see.weak.verbose"));
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   953
                        System.out.println("\n  " +
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   954
                                DisabledAlgorithmConstraints.PROPERTY_JAR_DISABLED_ALGS +
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   955
                                "=" + Security.getProperty(DisabledAlgorithmConstraints.PROPERTY_JAR_DISABLED_ALGS));
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   956
                    } else {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   957
                        System.out.println(rb.getString("jar.treated.unsigned.see.weak"));
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   958
                    }
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   959
                } else if (hasSignature) {
41582
246512d81eba 8165816: jarsigner -verify shows jar unsigned if it was signed with a weak algorithm
weijun
parents: 40177
diff changeset
   960
                    System.out.println(rb.getString("jar.treated.unsigned"));
246512d81eba 8165816: jarsigner -verify shows jar unsigned if it was signed with a weak algorithm
weijun
parents: 40177
diff changeset
   961
                } else {
246512d81eba 8165816: jarsigner -verify shows jar unsigned if it was signed with a weak algorithm
weijun
parents: 40177
diff changeset
   962
                    System.out.println(rb.getString("jar.is.unsigned"));
246512d81eba 8165816: jarsigner -verify shows jar unsigned if it was signed with a weak algorithm
weijun
parents: 40177
diff changeset
   963
                }
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   964
            } else {
22315
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   965
                boolean warningAppeared = false;
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   966
                boolean errorAppeared = false;
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   967
                if (badKeyUsage || badExtendedKeyUsage || badNetscapeCertType ||
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   968
                        notYetValidCert || chainNotValidated || hasExpiredCert ||
34382
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
   969
                        hasUnsignedEntry || signerSelfSigned || (weakAlg != 0) ||
22315
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   970
                        aliasNotInStore || notSignedByAlias) {
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   971
22315
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   972
                    if (strict) {
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   973
                        System.out.println(rb.getString("jar.verified.with.signer.errors."));
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   974
                        System.out.println();
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   975
                        System.out.println(rb.getString("Error."));
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   976
                        errorAppeared = true;
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   977
                    } else {
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   978
                        System.out.println(rb.getString("jar.verified."));
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   979
                        System.out.println();
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   980
                        System.out.println(rb.getString("Warning."));
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   981
                        warningAppeared = true;
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   982
                    }
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
   983
34382
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
   984
                    if (weakAlg != 0) {
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
   985
                        // In fact, jarsigner verification did not catch this
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
   986
                        // since it has not read the JarFile content itself.
41590
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   987
                        // Everything is done with JarFile API. The signing
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   988
                        // history (digestMap etc) will show these info and
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
   989
                        // print out proper warnings.
34382
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
   990
                    }
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
   991
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   992
                    if (badKeyUsage) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   993
                        System.out.println(
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   994
                            rb.getString("This.jar.contains.entries.whose.signer.certificate.s.KeyUsage.extension.doesn.t.allow.code.signing."));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
   995
                    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   996
90ce3da70b43 Initial load
duke
parents:
diff changeset
   997
                    if (badExtendedKeyUsage) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   998
                        System.out.println(
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
   999
                            rb.getString("This.jar.contains.entries.whose.signer.certificate.s.ExtendedKeyUsage.extension.doesn.t.allow.code.signing."));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1000
                    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1001
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1002
                    if (badNetscapeCertType) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1003
                        System.out.println(
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1004
                            rb.getString("This.jar.contains.entries.whose.signer.certificate.s.NetscapeCertType.extension.doesn.t.allow.code.signing."));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1005
                    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1006
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1007
                    if (hasUnsignedEntry) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1008
                        System.out.println(rb.getString(
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1009
                            "This.jar.contains.unsigned.entries.which.have.not.been.integrity.checked."));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1010
                    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1011
                    if (hasExpiredCert) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1012
                        System.out.println(rb.getString(
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1013
                            "This.jar.contains.entries.whose.signer.certificate.has.expired."));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1014
                    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1015
                    if (notYetValidCert) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1016
                        System.out.println(rb.getString(
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1017
                            "This.jar.contains.entries.whose.signer.certificate.is.not.yet.valid."));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1018
                    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1019
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1020
                    if (chainNotValidated) {
34382
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1021
                        System.out.println(String.format(
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1022
                                rb.getString("This.jar.contains.entries.whose.certificate.chain.is.not.validated.reason.1"),
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1023
                                chainNotValidatedReason.getLocalizedMessage()));
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1024
                    }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1025
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1026
                    if (notSignedByAlias) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1027
                        System.out.println(
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1028
                                rb.getString("This.jar.contains.signed.entries.which.is.not.signed.by.the.specified.alias.es."));
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1029
                    }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1030
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1031
                    if (aliasNotInStore) {
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1032
                        System.out.println(rb.getString("This.jar.contains.signed.entries.that.s.not.signed.by.alias.in.this.keystore."));
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1033
                    }
34382
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1034
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1035
                    if (signerSelfSigned) {
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1036
                        System.out.println(rb.getString(
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1037
                                "This.jar.contains.entries.whose.signer.certificate.is.self.signed."));
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1038
                    }
22315
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1039
                } else {
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1040
                    System.out.println(rb.getString("jar.verified."));
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1041
                }
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1042
                if (hasExpiringCert || noTimestamp) {
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1043
                    if (!warningAppeared) {
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1044
                        System.out.println();
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1045
                        System.out.println(rb.getString("Warning."));
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1046
                        warningAppeared = true;
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1047
                    }
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1048
                    if (hasExpiringCert) {
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1049
                        System.out.println(rb.getString(
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1050
                                "This.jar.contains.entries.whose.signer.certificate.will.expire.within.six.months."));
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1051
                    }
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1052
                    if (noTimestamp) {
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1053
                        System.out.println(
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1054
                                String.format(rb.getString("no.timestamp.verifying"), expireDate));
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1055
                    }
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1056
                }
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1057
                if (warningAppeared || errorAppeared) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1058
                    if (! (verbose != null && showcerts)) {
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1059
                        System.out.println();
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1060
                        System.out.println(rb.getString(
22315
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1061
                                "Re.run.with.the.verbose.and.certs.options.for.more.details."));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1062
                    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1063
                }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1064
            }
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1065
            return;
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1066
        } catch (Exception e) {
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1067
            System.out.println(rb.getString("jarsigner.") + e);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1068
            if (debug) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1069
                e.printStackTrace();
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1070
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1071
        } finally { // close the resource
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1072
            if (jf != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1073
                jf.close();
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1074
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1075
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1076
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1077
        System.exit(1);
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1078
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1079
41590
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1080
    private String withWeak(String alg, Set<CryptoPrimitive> primitiveSet) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1081
        if (DISABLED_CHECK.permits(primitiveSet, alg, null)) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1082
            return alg;
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1083
        } else {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1084
            seeWeak = true;
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1085
            return String.format(rb.getString("with.weak"), alg);
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1086
        }
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1087
    }
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1088
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1089
    private String withWeak(PublicKey key) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1090
        if (DISABLED_CHECK.permits(SIG_PRIMITIVE_SET, key)) {
46139
5196af754957 8185934: keytool shows "Signature algorithm: SHA1withECDSA, -1-bit key"
weijun
parents: 44597
diff changeset
  1091
            int kLen = KeyUtil.getKeySize(key);
5196af754957 8185934: keytool shows "Signature algorithm: SHA1withECDSA, -1-bit key"
weijun
parents: 44597
diff changeset
  1092
            if (kLen >= 0) {
5196af754957 8185934: keytool shows "Signature algorithm: SHA1withECDSA, -1-bit key"
weijun
parents: 44597
diff changeset
  1093
                return String.format(rb.getString("key.bit"), kLen);
5196af754957 8185934: keytool shows "Signature algorithm: SHA1withECDSA, -1-bit key"
weijun
parents: 44597
diff changeset
  1094
            } else {
5196af754957 8185934: keytool shows "Signature algorithm: SHA1withECDSA, -1-bit key"
weijun
parents: 44597
diff changeset
  1095
                return rb.getString("unknown.size");
5196af754957 8185934: keytool shows "Signature algorithm: SHA1withECDSA, -1-bit key"
weijun
parents: 44597
diff changeset
  1096
            }
41590
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1097
        } else {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1098
            seeWeak = true;
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1099
            return String.format(
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1100
                    rb.getString("key.bit.weak"), KeyUtil.getKeySize(key));
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1101
        }
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1102
    }
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1103
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1104
    private static MessageFormat validityTimeForm = null;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1105
    private static MessageFormat notYetTimeForm = null;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1106
    private static MessageFormat expiredTimeForm = null;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1107
    private static MessageFormat expiringTimeForm = null;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1108
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1109
    /*
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1110
     * Display some details about a certificate:
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1111
     *
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1112
     * [<tab>] <cert-type> [", " <subject-DN>] [" (" <keystore-entry-alias> ")"]
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1113
     * [<validity-period> | <expiry-warning>]
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1114
     *
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1115
     * Note: no newline character at the end
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1116
     */
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1117
    String printCert(String tab, Certificate c, boolean checkValidityPeriod,
12046
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1118
        Date timestamp, boolean checkUsage) {
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1119
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1120
        StringBuilder certStr = new StringBuilder();
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1121
        String space = rb.getString("SPACE");
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1122
        X509Certificate x509Cert = null;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1123
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1124
        if (c instanceof X509Certificate) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1125
            x509Cert = (X509Certificate) c;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1126
            certStr.append(tab).append(x509Cert.getType())
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1127
                .append(rb.getString("COMMA"))
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1128
                .append(x509Cert.getSubjectDN().getName());
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1129
        } else {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1130
            certStr.append(tab).append(c.getType());
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1131
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1132
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1133
        String alias = storeHash.get(c);
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1134
        if (alias != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1135
            certStr.append(space).append(alias);
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1136
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1137
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1138
        if (checkValidityPeriod && x509Cert != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1139
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1140
            certStr.append("\n").append(tab).append("[");
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1141
            Date notAfter = x509Cert.getNotAfter();
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1142
            try {
12046
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1143
                boolean printValidity = true;
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1144
                if (timestamp == null) {
22315
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1145
                    if (expireDate.getTime() == 0 || expireDate.after(notAfter)) {
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1146
                        expireDate = notAfter;
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1147
                    }
12046
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1148
                    x509Cert.checkValidity();
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1149
                    // test if cert will expire within six months
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1150
                    if (notAfter.getTime() < System.currentTimeMillis() + SIX_MONTHS) {
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1151
                        hasExpiringCert = true;
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1152
                        if (expiringTimeForm == null) {
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1153
                            expiringTimeForm = new MessageFormat(
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1154
                                rb.getString("certificate.will.expire.on"));
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1155
                        }
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1156
                        Object[] source = { notAfter };
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1157
                        certStr.append(expiringTimeForm.format(source));
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1158
                        printValidity = false;
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1159
                    }
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1160
                } else {
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1161
                    x509Cert.checkValidity(timestamp);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1162
                }
12046
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1163
                if (printValidity) {
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1164
                    if (validityTimeForm == null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1165
                        validityTimeForm = new MessageFormat(
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1166
                            rb.getString("certificate.is.valid.from"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1167
                    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1168
                    Object[] source = { x509Cert.getNotBefore(), notAfter };
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1169
                    certStr.append(validityTimeForm.format(source));
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1170
                }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1171
            } catch (CertificateExpiredException cee) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1172
                hasExpiredCert = true;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1173
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1174
                if (expiredTimeForm == null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1175
                    expiredTimeForm = new MessageFormat(
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1176
                        rb.getString("certificate.expired.on"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1177
                }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1178
                Object[] source = { notAfter };
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1179
                certStr.append(expiredTimeForm.format(source));
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1180
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1181
            } catch (CertificateNotYetValidException cnyve) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1182
                notYetValidCert = true;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1183
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1184
                if (notYetTimeForm == null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1185
                    notYetTimeForm = new MessageFormat(
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1186
                        rb.getString("certificate.is.not.valid.until"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1187
                }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1188
                Object[] source = { x509Cert.getNotBefore() };
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1189
                certStr.append(notYetTimeForm.format(source));
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1190
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1191
            certStr.append("]");
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1192
7525
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
  1193
            if (checkUsage) {
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
  1194
                boolean[] bad = new boolean[3];
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
  1195
                checkCertUsage(x509Cert, bad);
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
  1196
                if (bad[0] || bad[1] || bad[2]) {
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
  1197
                    String x = "";
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
  1198
                    if (bad[0]) {
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
  1199
                        x ="KeyUsage";
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
  1200
                    }
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
  1201
                    if (bad[1]) {
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
  1202
                        if (x.length() > 0) x = x + ", ";
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
  1203
                        x = x + "ExtendedKeyUsage";
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
  1204
                    }
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
  1205
                    if (bad[2]) {
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
  1206
                        if (x.length() > 0) x = x + ", ";
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
  1207
                        x = x + "NetscapeCertType";
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
  1208
                    }
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
  1209
                    certStr.append("\n").append(tab)
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1210
                        .append(MessageFormat.format(rb.getString(
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1211
                        ".{0}.extension.does.not.support.code.signing."), x));
7525
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
  1212
                }
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1213
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1214
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1215
        return certStr.toString();
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1216
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1217
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1218
    private static MessageFormat signTimeForm = null;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1219
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1220
    private String printTimestamp(String tab, Timestamp timestamp) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1221
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1222
        if (signTimeForm == null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1223
            signTimeForm =
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1224
                new MessageFormat(rb.getString("entry.was.signed.on"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1225
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1226
        Object[] source = { timestamp.getTimestamp() };
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1227
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1228
        return new StringBuilder().append(tab).append("[")
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1229
            .append(signTimeForm.format(source)).append("]").toString();
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1230
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1231
7977
f47f211cd627 7008713: diamond conversion of kerberos5 and security tools
smarks
parents: 7525
diff changeset
  1232
    private Map<CodeSigner,Integer> cacheForInKS = new IdentityHashMap<>();
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1233
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1234
    private int inKeyStoreForOneSigner(CodeSigner signer) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1235
        if (cacheForInKS.containsKey(signer)) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1236
            return cacheForInKS.get(signer);
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1237
        }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1238
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1239
        int result = 0;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1240
        List<? extends Certificate> certs = signer.getSignerCertPath().getCertificates();
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1241
        for (Certificate c : certs) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1242
            String alias = storeHash.get(c);
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1243
            if (alias != null) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1244
                if (alias.startsWith("(")) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1245
                    result |= IN_KEYSTORE;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1246
                }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1247
                if (ckaliases.contains(alias.substring(1, alias.length() - 1))) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1248
                    result |= SIGNED_BY_ALIAS;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1249
                }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1250
            } else {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1251
                if (store != null) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1252
                    try {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1253
                        alias = store.getCertificateAlias(c);
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1254
                    } catch (KeyStoreException kse) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1255
                        // never happens, because keystore has been loaded
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1256
                    }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1257
                    if (alias != null) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1258
                        storeHash.put(c, "(" + alias + ")");
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1259
                        result |= IN_KEYSTORE;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1260
                    }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1261
                }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1262
                if (ckaliases.contains(alias)) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1263
                    result |= SIGNED_BY_ALIAS;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1264
                }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1265
            }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1266
        }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1267
        cacheForInKS.put(signer, result);
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1268
        return result;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1269
    }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1270
7977
f47f211cd627 7008713: diamond conversion of kerberos5 and security tools
smarks
parents: 7525
diff changeset
  1271
    Hashtable<Certificate, String> storeHash = new Hashtable<>();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1272
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1273
    int inKeyStore(CodeSigner[] signers) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1274
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1275
        if (signers == null)
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1276
            return 0;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1277
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1278
        int output = 0;
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1279
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1280
        for (CodeSigner signer: signers) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1281
            int result = inKeyStoreForOneSigner(signer);
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1282
            output |= result;
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1283
        }
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1284
        if (ckaliases.size() > 0 && (output & SIGNED_BY_ALIAS) == 0) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1285
            output |= NOT_ALIAS;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1286
        }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1287
        return output;
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1288
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1289
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1290
    void signJar(String jarName, String alias)
34382
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1291
            throws Exception {
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1292
41590
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1293
        if (digestalg != null && !DISABLED_CHECK.permits(
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1294
                DIGEST_PRIMITIVE_SET, digestalg, null)) {
34382
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1295
            weakAlg |= 1;
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1296
        }
41590
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1297
        if (tSADigestAlg != null && !DISABLED_CHECK.permits(
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1298
                DIGEST_PRIMITIVE_SET, tSADigestAlg, null)) {
34382
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1299
            weakAlg |= 4;
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1300
        }
41590
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1301
        if (sigalg != null && !DISABLED_CHECK.permits(
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1302
                SIG_PRIMITIVE_SET , sigalg, null)) {
34382
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1303
            weakAlg |= 2;
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1304
        }
41590
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1305
        if (!DISABLED_CHECK.permits(
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1306
                SIG_PRIMITIVE_SET, privateKey)) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1307
            weakAlg |= 8;
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1308
        }
34382
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1309
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1310
        boolean aliasUsed = false;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1311
        X509Certificate tsaCert = null;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1312
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1313
        if (sigfile == null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1314
            sigfile = alias;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1315
            aliasUsed = true;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1316
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1317
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1318
        if (sigfile.length() > 8) {
4152
bc36a9f01ac6 6870812: enhance security tools to use ECC algorithms
weijun
parents: 3951
diff changeset
  1319
            sigfile = sigfile.substring(0, 8).toUpperCase(Locale.ENGLISH);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1320
        } else {
4152
bc36a9f01ac6 6870812: enhance security tools to use ECC algorithms
weijun
parents: 3951
diff changeset
  1321
            sigfile = sigfile.toUpperCase(Locale.ENGLISH);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1322
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1323
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1324
        StringBuilder tmpSigFile = new StringBuilder(sigfile.length());
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1325
        for (int j = 0; j < sigfile.length(); j++) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1326
            char c = sigfile.charAt(j);
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1327
            if (!
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1328
                    ((c>= 'A' && c<= 'Z') ||
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1329
                            (c>= '0' && c<= '9') ||
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1330
                            (c == '-') ||
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1331
                            (c == '_'))) {
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1332
                if (aliasUsed) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1333
                    // convert illegal characters from the alias to be _'s
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1334
                    c = '_';
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1335
                } else {
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1336
                    throw new
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1337
                            RuntimeException(rb.getString
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1338
                            ("signature.filename.must.consist.of.the.following.characters.A.Z.0.9.or."));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1339
                }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1340
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1341
            tmpSigFile.append(c);
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1342
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1343
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1344
        sigfile = tmpSigFile.toString();
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1345
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1346
        String tmpJarName;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1347
        if (signedjar == null) tmpJarName = jarName+".sig";
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1348
        else tmpJarName = signedjar;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1349
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1350
        File jarFile = new File(jarName);
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1351
        File signedJarFile = new File(tmpJarName);
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1352
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1353
        // Open the jar (zip) file
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1354
        try {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1355
            zipFile = new ZipFile(jarName);
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1356
        } catch (IOException ioe) {
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1357
            error(rb.getString("unable.to.open.jar.file.")+jarName, ioe);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1358
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1359
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1360
        FileOutputStream fos = null;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1361
        try {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1362
            fos = new FileOutputStream(signedJarFile);
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1363
        } catch (IOException ioe) {
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1364
            error(rb.getString("unable.to.create.")+tmpJarName, ioe);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1365
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1366
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1367
        CertPath cp = CertificateFactory.getInstance("X.509")
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1368
                .generateCertPath(Arrays.asList(certChain));
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1369
        JarSigner.Builder builder = new JarSigner.Builder(privateKey, cp);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1370
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1371
        if (verbose != null) {
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1372
            builder.eventHandler((action, file) -> {
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1373
                System.out.println(rb.getString("." + action + ".") + file);
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1374
            });
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1375
        }
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1376
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1377
        if (digestalg != null) {
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1378
            builder.digestAlgorithm(digestalg);
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1379
        }
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1380
        if (sigalg != null) {
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1381
            builder.signatureAlgorithm(sigalg);
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1382
        }
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1383
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1384
        URI tsaURI = null;
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1385
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1386
        if (tsaUrl != null) {
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1387
            tsaURI = new URI(tsaUrl);
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1388
        } else if (tsaAlias != null) {
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1389
            tsaCert = getTsaCert(tsaAlias);
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1390
            tsaURI = TimestampedSigner.getTimestampingURI(tsaCert);
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1391
        }
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1392
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1393
        if (tsaURI != null) {
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1394
            if (verbose != null) {
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1395
                System.out.println(
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1396
                        rb.getString("requesting.a.signature.timestamp"));
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1397
                if (tsaUrl != null) {
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1398
                    System.out.println(rb.getString("TSA.location.") + tsaUrl);
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1399
                } else if (tsaCert != null) {
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1400
                    System.out.println(rb.getString("TSA.certificate.") +
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1401
                            printCert("", tsaCert, false, null, false));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1402
                }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1403
            }
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1404
            builder.tsa(tsaURI);
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1405
            if (tSADigestAlg != null) {
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1406
                builder.setProperty("tsaDigestAlg", tSADigestAlg);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1407
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1408
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1409
            if (tSAPolicyID != null) {
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1410
                builder.setProperty("tsaPolicyId", tSAPolicyID);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1411
            }
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1412
        } else {
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1413
            noTimestamp = true;
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1414
        }
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1415
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1416
        if (altSignerClass != null) {
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1417
            builder.setProperty("altSigner", altSignerClass);
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1418
            if (verbose != null) {
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1419
                System.out.println(
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1420
                        rb.getString("using.an.alternative.signing.mechanism"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1421
            }
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1422
        }
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1423
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1424
        if (altSignerClasspath != null) {
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1425
            builder.setProperty("altSignerPath", altSignerClasspath);
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1426
        }
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1427
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1428
        builder.signerName(sigfile);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1429
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1430
        builder.setProperty("sectionsOnly", Boolean.toString(!signManifest));
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1431
        builder.setProperty("internalSF", Boolean.toString(!externalSF));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1432
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1433
        try {
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1434
            builder.build().sign(zipFile, fos);
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1435
        } catch (JarSignerException e) {
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1436
            Throwable cause = e.getCause();
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1437
            if (cause != null && cause instanceof SocketTimeoutException) {
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1438
                // Provide a helpful message when TSA is beyond a firewall
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1439
                error(rb.getString("unable.to.sign.jar.") +
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1440
                        rb.getString("no.response.from.the.Timestamping.Authority.") +
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1441
                        "\n  -J-Dhttp.proxyHost=<hostname>" +
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1442
                        "\n  -J-Dhttp.proxyPort=<portnumber>\n" +
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1443
                        rb.getString("or") +
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1444
                        "\n  -J-Dhttps.proxyHost=<hostname> " +
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1445
                        "\n  -J-Dhttps.proxyPort=<portnumber> ", e);
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1446
            } else {
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1447
                error(rb.getString("unable.to.sign.jar.")+e.getCause(), e.getCause());
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1448
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1449
        } finally {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1450
            // close the resouces
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1451
            if (zipFile != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1452
                zipFile.close();
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1453
                zipFile = null;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1454
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1455
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1456
            if (fos != null) {
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1457
                fos.close();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1458
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1459
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1460
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1461
        // no IOException thrown in the follow try clause, so disable
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1462
        // the try clause.
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1463
        // try {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1464
            if (signedjar == null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1465
                // attempt an atomic rename. If that fails,
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1466
                // rename the original jar file, then the signed
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1467
                // one, then delete the original.
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1468
                if (!signedJarFile.renameTo(jarFile)) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1469
                    File origJar = new File(jarName+".orig");
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1470
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1471
                    if (jarFile.renameTo(origJar)) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1472
                        if (signedJarFile.renameTo(jarFile)) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1473
                            origJar.delete();
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1474
                        } else {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1475
                            MessageFormat form = new MessageFormat(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1476
                        ("attempt.to.rename.signedJarFile.to.jarFile.failed"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1477
                            Object[] source = {signedJarFile, jarFile};
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1478
                            error(form.format(source));
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1479
                        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1480
                    } else {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1481
                        MessageFormat form = new MessageFormat(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1482
                            ("attempt.to.rename.jarFile.to.origJar.failed"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1483
                        Object[] source = {jarFile, origJar};
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1484
                        error(form.format(source));
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1485
                    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1486
                }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1487
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1488
22315
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1489
            boolean warningAppeared = false;
34382
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1490
            if (weakAlg != 0 || badKeyUsage || badExtendedKeyUsage || badNetscapeCertType ||
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1491
                    notYetValidCert || chainNotValidated || hasExpiredCert || signerSelfSigned) {
22315
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1492
                if (strict) {
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1493
                    System.out.println(rb.getString("jar.signed.with.signer.errors."));
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1494
                    System.out.println();
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1495
                    System.out.println(rb.getString("Error."));
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1496
                } else {
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1497
                    System.out.println(rb.getString("jar.signed."));
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1498
                    System.out.println();
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1499
                    System.out.println(rb.getString("Warning."));
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1500
                    warningAppeared = true;
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1501
                }
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1502
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1503
                if (badKeyUsage) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1504
                    System.out.println(
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1505
                        rb.getString("The.signer.certificate.s.KeyUsage.extension.doesn.t.allow.code.signing."));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1506
                }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1507
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1508
                if (badExtendedKeyUsage) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1509
                    System.out.println(
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1510
                        rb.getString("The.signer.certificate.s.ExtendedKeyUsage.extension.doesn.t.allow.code.signing."));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1511
                }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1512
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1513
                if (badNetscapeCertType) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1514
                    System.out.println(
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1515
                        rb.getString("The.signer.certificate.s.NetscapeCertType.extension.doesn.t.allow.code.signing."));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1516
                }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1517
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1518
                if (hasExpiredCert) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1519
                    System.out.println(
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1520
                        rb.getString("The.signer.certificate.has.expired."));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1521
                } else if (notYetValidCert) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1522
                    System.out.println(
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1523
                        rb.getString("The.signer.certificate.is.not.yet.valid."));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1524
                }
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1525
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1526
                if (chainNotValidated) {
34382
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1527
                    System.out.println(String.format(
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1528
                            rb.getString("The.signer.s.certificate.chain.is.not.validated.reason.1"),
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1529
                            chainNotValidatedReason.getLocalizedMessage()));
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1530
                }
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1531
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1532
                if (signerSelfSigned) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1533
                    System.out.println(
34382
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1534
                            rb.getString("The.signer.s.certificate.is.self.signed."));
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1535
                }
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1536
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1537
                if ((weakAlg & 1) == 1) {
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1538
                    System.out.println(String.format(
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1539
                            rb.getString("The.1.algorithm.specified.for.the.2.option.is.considered.a.security.risk."),
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1540
                            digestalg, "-digestalg"));
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1541
                }
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1542
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1543
                if ((weakAlg & 2) == 2) {
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1544
                    System.out.println(String.format(
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1545
                            rb.getString("The.1.algorithm.specified.for.the.2.option.is.considered.a.security.risk."),
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1546
                            sigalg, "-sigalg"));
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1547
                }
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1548
                if ((weakAlg & 4) == 4) {
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1549
                    System.out.println(String.format(
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1550
                            rb.getString("The.1.algorithm.specified.for.the.2.option.is.considered.a.security.risk."),
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1551
                            tSADigestAlg, "-tsadigestalg"));
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1552
                }
41590
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1553
                if ((weakAlg & 8) == 8) {
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1554
                    System.out.println(String.format(
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1555
                            rb.getString("The.1.signing.key.has.a.keysize.of.2.which.is.considered.a.security.risk."),
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1556
                            privateKey.getAlgorithm(), KeyUtil.getKeySize(privateKey)));
1c5b1891b8e0 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar
weijun
parents: 41582
diff changeset
  1557
                }
22315
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1558
            } else {
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1559
                System.out.println(rb.getString("jar.signed."));
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1560
            }
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1561
            if (hasExpiringCert || noTimestamp) {
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1562
                if (!warningAppeared) {
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1563
                    System.out.println();
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1564
                    System.out.println(rb.getString("Warning."));
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1565
                }
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1566
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1567
                if (hasExpiringCert) {
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1568
                    System.out.println(
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1569
                            rb.getString("The.signer.certificate.will.expire.within.six.months."));
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1570
                }
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1571
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1572
                if (noTimestamp) {
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1573
                    System.out.println(
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1574
                            String.format(rb.getString("no.timestamp.signing"), expireDate));
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1575
                }
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1576
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1577
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1578
        // no IOException thrown in the above try clause, so disable
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1579
        // the catch clause.
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1580
        // } catch(IOException ioe) {
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1581
        //     error(rb.getString("unable.to.sign.jar.")+ioe, ioe);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1582
        // }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1583
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1584
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1585
    /**
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1586
     * signature-related files include:
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1587
     * . META-INF/MANIFEST.MF
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1588
     * . META-INF/SIG-*
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1589
     * . META-INF/*.SF
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1590
     * . META-INF/*.DSA
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1591
     * . META-INF/*.RSA
4152
bc36a9f01ac6 6870812: enhance security tools to use ECC algorithms
weijun
parents: 3951
diff changeset
  1592
     * . META-INF/*.EC
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1593
     */
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1594
    private boolean signatureRelated(String name) {
23912
9eab25093a89 8026067: Enhance signed jar verification
weijun
parents: 23010
diff changeset
  1595
        return SignatureFileVerifier.isSigningRelated(name);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1596
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1597
7977
f47f211cd627 7008713: diamond conversion of kerberos5 and security tools
smarks
parents: 7525
diff changeset
  1598
    Map<CodeSigner,String> cacheForSignerInfo = new IdentityHashMap<>();
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1599
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1600
    /**
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1601
     * Returns a string of singer info, with a newline at the end
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1602
     */
12046
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1603
    private String signerInfo(CodeSigner signer, String tab) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1604
        if (cacheForSignerInfo.containsKey(signer)) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1605
            return cacheForSignerInfo.get(signer);
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1606
        }
24969
afa6934dd8e8 8041679: Replace uses of StringBuffer with StringBuilder within core library classes
psandoz
parents: 24868
diff changeset
  1607
        StringBuilder sb = new StringBuilder();
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1608
        List<? extends Certificate> certs = signer.getSignerCertPath().getCertificates();
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1609
        // display the signature timestamp, if present
12046
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1610
        Date timestamp;
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1611
        Timestamp ts = signer.getTimestamp();
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1612
        if (ts != null) {
24969
afa6934dd8e8 8041679: Replace uses of StringBuffer with StringBuilder within core library classes
psandoz
parents: 24868
diff changeset
  1613
            sb.append(printTimestamp(tab, ts));
afa6934dd8e8 8041679: Replace uses of StringBuffer with StringBuilder within core library classes
psandoz
parents: 24868
diff changeset
  1614
            sb.append('\n');
12046
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1615
            timestamp = ts.getTimestamp();
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1616
        } else {
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1617
            timestamp = null;
22315
529f1cc322fc 8024302: Clarify jar verifications
weijun
parents: 19189
diff changeset
  1618
            noTimestamp = true;
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1619
        }
24969
afa6934dd8e8 8041679: Replace uses of StringBuffer with StringBuilder within core library classes
psandoz
parents: 24868
diff changeset
  1620
        // display the certificate(sb). The first one is end-entity cert and
7525
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
  1621
        // its KeyUsage should be checked.
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
  1622
        boolean first = true;
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1623
        for (Certificate c : certs) {
24969
afa6934dd8e8 8041679: Replace uses of StringBuffer with StringBuilder within core library classes
psandoz
parents: 24868
diff changeset
  1624
            sb.append(printCert(tab, c, true, timestamp, first));
afa6934dd8e8 8041679: Replace uses of StringBuffer with StringBuilder within core library classes
psandoz
parents: 24868
diff changeset
  1625
            sb.append('\n');
7525
16d2b5e6517a 7004168: jarsigner -verify checks for KeyUsage codesigning ext on all certs instead of just signing cert
weijun
parents: 7524
diff changeset
  1626
            first = false;
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1627
        }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1628
        try {
24625
22fb8a68756f 8036709: Java 7 jarsigner displays warning about cert policy tree
weijun
parents: 24034
diff changeset
  1629
            validateCertChain(certs);
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1630
        } catch (Exception e) {
43183
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1631
            chainNotValidated = true;
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1632
            chainNotValidatedReason = e;
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1633
            sb.append(tab).append(rb.getString(".CertPath.not.validated."))
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1634
                    .append(e.getLocalizedMessage()).append("]\n"); // TODO
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1635
        }
34382
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1636
        if (certs.size() == 1
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1637
                && KeyStoreUtil.isSelfSigned((X509Certificate)certs.get(0))) {
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1638
            signerSelfSigned = true;
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1639
        }
24969
afa6934dd8e8 8041679: Replace uses of StringBuffer with StringBuilder within core library classes
psandoz
parents: 24868
diff changeset
  1640
        String result = sb.toString();
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1641
        cacheForSignerInfo.put(signer, result);
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1642
        return result;
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1643
    }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1644
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1645
    void loadKeyStore(String keyStoreName, boolean prompt) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1646
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1647
        if (!nullStream && keyStoreName == null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1648
            keyStoreName = System.getProperty("user.home") + File.separator
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1649
                + ".keystore";
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1650
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1651
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1652
        try {
10427
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1653
            Set<TrustAnchor> tas = new HashSet<>();
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1654
            try {
14182
3041082abb40 7194449: String resources for Key Tool and Policy Tool should be in their respective packages
sflores
parents: 12046
diff changeset
  1655
                KeyStore caks = KeyStoreUtil.getCacertsKeyStore();
10427
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1656
                if (caks != null) {
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1657
                    Enumeration<String> aliases = caks.aliases();
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1658
                    while (aliases.hasMoreElements()) {
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1659
                        String a = aliases.nextElement();
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1660
                        try {
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1661
                            tas.add(new TrustAnchor((X509Certificate)caks.getCertificate(a), null));
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1662
                        } catch (Exception e2) {
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1663
                            // ignore, when a SecretkeyEntry does not include a cert
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1664
                        }
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1665
                    }
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1666
                }
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1667
            } catch (Exception e) {
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1668
                // Ignore, if cacerts cannot be loaded
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1669
            }
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1670
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1671
            if (providerName == null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1672
                store = KeyStore.getInstance(storetype);
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1673
            } else {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1674
                store = KeyStore.getInstance(storetype, providerName);
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1675
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1676
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1677
            // Get pass phrase
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1678
            // XXX need to disable echo; on UNIX, call getpass(char *prompt)Z
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1679
            // and on NT call ??
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1680
            if (token && storepass == null && !protectedPath
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1681
                    && !KeyStoreUtil.isWindowsKeyStore(storetype)) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1682
                storepass = getPass
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1683
                        (rb.getString("Enter.Passphrase.for.keystore."));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1684
            } else if (!token && storepass == null && prompt) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1685
                storepass = getPass
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1686
                        (rb.getString("Enter.Passphrase.for.keystore."));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1687
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1688
10427
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1689
            try {
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1690
                if (nullStream) {
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1691
                    store.load(null, storepass);
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1692
                } else {
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1693
                    keyStoreName = keyStoreName.replace(File.separatorChar, '/');
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1694
                    URL url = null;
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1695
                    try {
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1696
                        url = new URL(keyStoreName);
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1697
                    } catch (java.net.MalformedURLException e) {
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1698
                        // try as file
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1699
                        url = new File(keyStoreName).toURI().toURL();
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1700
                    }
10427
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1701
                    InputStream is = null;
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1702
                    try {
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1703
                        is = url.openStream();
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1704
                        store.load(is, storepass);
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1705
                    } finally {
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1706
                        if (is != null) {
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1707
                            is.close();
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1708
                        }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1709
                    }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1710
                }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1711
                Enumeration<String> aliases = store.aliases();
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1712
                while (aliases.hasMoreElements()) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1713
                    String a = aliases.nextElement();
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1714
                    try {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1715
                        X509Certificate c = (X509Certificate)store.getCertificate(a);
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1716
                        // Only add TrustedCertificateEntry and self-signed
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1717
                        // PrivateKeyEntry
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1718
                        if (store.isCertificateEntry(a) ||
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1719
                                c.getSubjectDN().equals(c.getIssuerDN())) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1720
                            tas.add(new TrustAnchor(c, null));
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1721
                        }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1722
                    } catch (Exception e2) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1723
                        // ignore, when a SecretkeyEntry does not include a cert
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1724
                    }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1725
                }
10427
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1726
            } finally {
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1727
                try {
43183
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1728
                    pkixParameters = new PKIXBuilderParameters(tas, null);
10427
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1729
                    pkixParameters.setRevocationEnabled(false);
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1730
                } catch (InvalidAlgorithmParameterException ex) {
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1731
                    // Only if tas is empty
c255e1803e4d 7081783: jarsigner error when no $HOME/.keystore
weijun
parents: 10336
diff changeset
  1732
                }
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1733
            }
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1734
        } catch (IOException ioe) {
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1735
            throw new RuntimeException(rb.getString("keystore.load.") +
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1736
                                        ioe.getMessage());
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1737
        } catch (java.security.cert.CertificateException ce) {
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1738
            throw new RuntimeException(rb.getString("certificate.exception.") +
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1739
                                        ce.getMessage());
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1740
        } catch (NoSuchProviderException pe) {
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1741
            throw new RuntimeException(rb.getString("keystore.load.") +
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1742
                                        pe.getMessage());
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1743
        } catch (NoSuchAlgorithmException nsae) {
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1744
            throw new RuntimeException(rb.getString("keystore.load.") +
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1745
                                        nsae.getMessage());
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1746
        } catch (KeyStoreException kse) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1747
            throw new RuntimeException
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1748
                (rb.getString("unable.to.instantiate.keystore.class.") +
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1749
                kse.getMessage());
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1750
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1751
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1752
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1753
    X509Certificate getTsaCert(String alias) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1754
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1755
        java.security.cert.Certificate cs = null;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1756
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1757
        try {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1758
            cs = store.getCertificate(alias);
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1759
        } catch (KeyStoreException kse) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1760
            // this never happens, because keystore has been loaded
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1761
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1762
        if (cs == null || (!(cs instanceof X509Certificate))) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1763
            MessageFormat form = new MessageFormat(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1764
                ("Certificate.not.found.for.alias.alias.must.reference.a.valid.KeyStore.entry.containing.an.X.509.public.key.certificate.for.the"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1765
            Object[] source = {alias, alias};
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1766
            error(form.format(source));
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1767
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1768
        return (X509Certificate) cs;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1769
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1770
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1771
    /**
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1772
     * Check if userCert is designed to be a code signer
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1773
     * @param userCert the certificate to be examined
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1774
     * @param bad 3 booleans to show if the KeyUsage, ExtendedKeyUsage,
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1775
     *            NetscapeCertType has codeSigning flag turned on.
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1776
     *            If null, the class field badKeyUsage, badExtendedKeyUsage,
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1777
     *            badNetscapeCertType will be set.
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1778
     */
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1779
    void checkCertUsage(X509Certificate userCert, boolean[] bad) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1780
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1781
        // Can act as a signer?
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1782
        // 1. if KeyUsage, then [0:digitalSignature] or
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1783
        //    [1:nonRepudiation] should be true
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1784
        // 2. if ExtendedKeyUsage, then should contains ANY or CODE_SIGNING
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1785
        // 3. if NetscapeCertType, then should contains OBJECT_SIGNING
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1786
        // 1,2,3 must be true
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1787
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1788
        if (bad != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1789
            bad[0] = bad[1] = bad[2] = false;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1790
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1791
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1792
        boolean[] keyUsage = userCert.getKeyUsage();
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1793
        if (keyUsage != null) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1794
            keyUsage = Arrays.copyOf(keyUsage, 9);
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1795
            if (!keyUsage[0] && !keyUsage[1]) {
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1796
                if (bad != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1797
                    bad[0] = true;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1798
                    badKeyUsage = true;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1799
                }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1800
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1801
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1802
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1803
        try {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1804
            List<String> xKeyUsage = userCert.getExtendedKeyUsage();
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1805
            if (xKeyUsage != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1806
                if (!xKeyUsage.contains("2.5.29.37.0") // anyExtendedKeyUsage
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1807
                        && !xKeyUsage.contains("1.3.6.1.5.5.7.3.3")) {  // codeSigning
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1808
                    if (bad != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1809
                        bad[1] = true;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1810
                        badExtendedKeyUsage = true;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1811
                    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1812
                }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1813
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1814
        } catch (java.security.cert.CertificateParsingException e) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1815
            // shouldn't happen
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1816
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1817
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1818
        try {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1819
            // OID_NETSCAPE_CERT_TYPE
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1820
            byte[] netscapeEx = userCert.getExtensionValue
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1821
                    ("2.16.840.1.113730.1.1");
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1822
            if (netscapeEx != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1823
                DerInputStream in = new DerInputStream(netscapeEx);
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1824
                byte[] encoded = in.getOctetString();
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1825
                encoded = new DerValue(encoded).getUnalignedBitString()
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1826
                        .toByteArray();
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1827
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1828
                NetscapeCertTypeExtension extn =
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1829
                        new NetscapeCertTypeExtension(encoded);
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1830
10336
0bb1999251f8 7064075: Security libraries don't build with javac -Xlint:all,-deprecation -Werror
jjg
parents: 9011
diff changeset
  1831
                Boolean val = extn.get(NetscapeCertTypeExtension.OBJECT_SIGNING);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1832
                if (!val) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1833
                    if (bad != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1834
                        bad[2] = true;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1835
                        badNetscapeCertType = true;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1836
                    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1837
                }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1838
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1839
        } catch (IOException e) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1840
            //
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1841
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1842
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1843
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1844
    void getAliasInfo(String alias) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1845
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1846
        Key key = null;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1847
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1848
        try {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1849
            java.security.cert.Certificate[] cs = null;
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1850
            if (altCertChain != null) {
21642
5efc900f8ecd 8027991: InputStream should be closed in sun.security.tools.jarsigner.Main
weijun
parents: 21278
diff changeset
  1851
                try (FileInputStream fis = new FileInputStream(altCertChain)) {
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1852
                    cs = CertificateFactory.getInstance("X.509").
21642
5efc900f8ecd 8027991: InputStream should be closed in sun.security.tools.jarsigner.Main
weijun
parents: 21278
diff changeset
  1853
                            generateCertificates(fis).
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1854
                            toArray(new Certificate[0]);
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1855
                } catch (FileNotFoundException ex) {
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1856
                    error(rb.getString("File.specified.by.certchain.does.not.exist"));
21642
5efc900f8ecd 8027991: InputStream should be closed in sun.security.tools.jarsigner.Main
weijun
parents: 21278
diff changeset
  1857
                } catch (CertificateException | IOException ex) {
5efc900f8ecd 8027991: InputStream should be closed in sun.security.tools.jarsigner.Main
weijun
parents: 21278
diff changeset
  1858
                    error(rb.getString("Cannot.restore.certchain.from.file.specified"));
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1859
                }
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1860
            } else {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1861
                try {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1862
                    cs = store.getCertificateChain(alias);
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1863
                } catch (KeyStoreException kse) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1864
                    // this never happens, because keystore has been loaded
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1865
                }
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1866
            }
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1867
            if (cs == null || cs.length == 0) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1868
                if (altCertChain != null) {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1869
                    error(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1870
                            ("Certificate.chain.not.found.in.the.file.specified."));
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1871
                } else {
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1872
                    MessageFormat form = new MessageFormat(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1873
                        ("Certificate.chain.not.found.for.alias.alias.must.reference.a.valid.KeyStore.key.entry.containing.a.private.key.and"));
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1874
                    Object[] source = {alias, alias};
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1875
                    error(form.format(source));
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1876
                }
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1877
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1878
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1879
            certChain = new X509Certificate[cs.length];
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1880
            for (int i=0; i<cs.length; i++) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1881
                if (!(cs[i] instanceof X509Certificate)) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1882
                    error(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1883
                        ("found.non.X.509.certificate.in.signer.s.chain"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1884
                }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1885
                certChain[i] = (X509Certificate)cs[i];
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1886
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1887
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1888
            // We don't meant to print anything, the next call
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1889
            // checks validity and keyUsage etc
12046
378aa3362868 7149012: jarsigner needs not warn about cert expiration if the jar has a TSA timestamp
weijun
parents: 10788
diff changeset
  1890
            printCert("", certChain[0], true, null, true);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1891
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1892
            try {
24625
22fb8a68756f 8036709: Java 7 jarsigner displays warning about cert policy tree
weijun
parents: 24034
diff changeset
  1893
                validateCertChain(Arrays.asList(certChain));
2432
dc17f417ef85 6802846: jarsigner needs enhanced cert validation(options)
weijun
parents: 2
diff changeset
  1894
            } catch (Exception e) {
43183
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1895
                chainNotValidated = true;
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1896
                chainNotValidatedReason = e;
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1897
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1898
34382
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1899
            if (KeyStoreUtil.isSelfSigned(certChain[0])) {
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1900
                signerSelfSigned = true;
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1901
            }
5d11306d6969 8130132: jarsigner should emit warning if weak algorithms or keysizes are used
weijun
parents: 33872
diff changeset
  1902
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1903
            try {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1904
                if (!token && keypass == null)
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1905
                    key = store.getKey(alias, storepass);
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1906
                else
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1907
                    key = store.getKey(alias, keypass);
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1908
            } catch (UnrecoverableKeyException e) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1909
                if (token) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1910
                    throw e;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1911
                } else if (keypass == null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1912
                    // Did not work out, so prompt user for key password
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1913
                    MessageFormat form = new MessageFormat(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1914
                        ("Enter.key.password.for.alias."));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1915
                    Object[] source = {alias};
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1916
                    keypass = getPass(form.format(source));
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1917
                    key = store.getKey(alias, keypass);
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1918
                }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1919
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1920
        } catch (NoSuchAlgorithmException e) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1921
            error(e.getMessage());
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1922
        } catch (UnrecoverableKeyException e) {
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1923
            error(rb.getString("unable.to.recover.key.from.keystore"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1924
        } catch (KeyStoreException kse) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1925
            // this never happens, because keystore has been loaded
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1926
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1927
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1928
        if (!(key instanceof PrivateKey)) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1929
            MessageFormat form = new MessageFormat(rb.getString
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1930
                ("key.associated.with.alias.not.a.private.key"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1931
            Object[] source = {alias};
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1932
            error(form.format(source));
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1933
        } else {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1934
            privateKey = (PrivateKey)key;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1935
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1936
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1937
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1938
    void error(String message) {
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1939
        System.out.println(rb.getString("jarsigner.")+message);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1940
        System.exit(1);
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1941
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1942
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1943
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1944
    void error(String message, Throwable e) {
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1945
        System.out.println(rb.getString("jarsigner.")+message);
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1946
        if (debug) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1947
            e.printStackTrace();
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1948
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1949
        System.exit(1);
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1950
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1951
24625
22fb8a68756f 8036709: Java 7 jarsigner displays warning about cert policy tree
weijun
parents: 24034
diff changeset
  1952
    void validateCertChain(List<? extends Certificate> certs) throws Exception {
43183
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1953
        try {
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1954
            Validator.getInstance(Validator.TYPE_PKIX,
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1955
                    Validator.VAR_CODE_SIGNING,
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1956
                    pkixParameters)
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1957
                    .validate(certs.toArray(new X509Certificate[certs.size()]));
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1958
        } catch (Exception e) {
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1959
            if (debug) {
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1960
                e.printStackTrace();
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1961
            }
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1962
            if (e instanceof ValidatorException) {
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1963
                // Throw cause if it's CertPathValidatorException,
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1964
                if (e.getCause() != null &&
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1965
                        e.getCause() instanceof CertPathValidatorException) {
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1966
                    e = (Exception) e.getCause();
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1967
                    Throwable t = e.getCause();
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1968
                    if ((t instanceof CertificateExpiredException &&
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1969
                                hasExpiredCert) ||
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1970
                            (t instanceof CertificateNotYetValidException &&
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1971
                                    notYetValidCert)) {
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1972
                        // we already have hasExpiredCert and notYetValidCert
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1973
                        return;
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1974
                    }
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1975
                }
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1976
                if (e instanceof ValidatorException) {
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1977
                    ValidatorException ve = (ValidatorException)e;
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1978
                    if (ve.getErrorType() == ValidatorException.T_EE_EXTENSIONS &&
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1979
                            (badKeyUsage || badExtendedKeyUsage || badNetscapeCertType)) {
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1980
                        // We already have badKeyUsage, badExtendedKeyUsage
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1981
                        // and badNetscapeCertType
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1982
                        return;
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1983
                    }
24625
22fb8a68756f 8036709: Java 7 jarsigner displays warning about cert policy tree
weijun
parents: 24034
diff changeset
  1984
                }
22fb8a68756f 8036709: Java 7 jarsigner displays warning about cert policy tree
weijun
parents: 24034
diff changeset
  1985
            }
43183
b50e0f90d284 8172529: Use PKIXValidator in jarsigner
weijun
parents: 41590
diff changeset
  1986
            throw e;
24625
22fb8a68756f 8036709: Java 7 jarsigner displays warning about cert policy tree
weijun
parents: 24034
diff changeset
  1987
        }
22fb8a68756f 8036709: Java 7 jarsigner displays warning about cert policy tree
weijun
parents: 24034
diff changeset
  1988
    }
22fb8a68756f 8036709: Java 7 jarsigner displays warning about cert policy tree
weijun
parents: 24034
diff changeset
  1989
33872
94e3836950ec 8056174: New APIs for jar signing
weijun
parents: 31270
diff changeset
  1990
    char[] getPass(String prompt) {
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1991
        System.err.print(prompt);
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1992
        System.err.flush();
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1993
        try {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1994
            char[] pass = Password.readPassword(System.in);
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1995
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1996
            if (pass == null) {
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  1997
                error(rb.getString("you.must.enter.key.password"));
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1998
            } else {
90ce3da70b43 Initial load
duke
parents:
diff changeset
  1999
                return pass;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  2000
            }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  2001
        } catch (IOException ioe) {
7179
4afb81e50183 6987827: security/util/Resources.java needs improvement
weijun
parents: 5627
diff changeset
  2002
            error(rb.getString("unable.to.read.password.")+ioe.getMessage());
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
  2003
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  2004
        // this shouldn't happen
90ce3da70b43 Initial load
duke
parents:
diff changeset
  2005
        return null;
90ce3da70b43 Initial load
duke
parents:
diff changeset
  2006
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
  2007
}