rpms/crypto-utils
3
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Fix password argument usage and require module access password on fips mode

Browse Source 
per code review, move password struct setup earlier, and use
    SECU_PrintError instead of printf whenever possible
This commit is contained in:
Elio Maldonado 2008-10-26 23:50:45 +00:00
parent 1746d724ae
commit 6367e39f7c
1 changed files with 121 additions and 70 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

177
keyutil.c
View File

@ -214,7 +214,7 @@ Usage(char *progName)
fprintf(stderr, "{-g|--gsize} key_size size in bitsof the rsa key to generate"); fprintf(stderr, "{-g|--gsize} key_size size in bitsof the rsa key to generate");
fprintf(stderr, "{-v|--validity} months cert validity in months"); fprintf(stderr, "{-v|--validity} months cert validity in months");
fprintf(stderr, "{-z|--znoisefile} noisefile seed file for use in key gneration"); fprintf(stderr, "{-z|--znoisefile} noisefile seed file for use in key gneration");
fprintf(stderr, "{-f|--filepwdnss} pwdfile file with the key encryption_password"); fprintf(stderr, "{-e|--filepwdnss} keypwdfile file with the key encryption_password");
fprintf(stderr, "{-f|--filepwdnss} modpwdfile file with the module access_password"); fprintf(stderr, "{-f|--filepwdnss} modpwdfile file with the module access_password");
fprintf(stderr, "{-d|--digest} digest-algorithm digest algorithm"); fprintf(stderr, "{-d|--digest} digest-algorithm digest algorithm");
fprintf(stderr, "{-i|--input} inputkey-file file with key with which to encrypt or to sign a request"); fprintf(stderr, "{-i|--input} inputkey-file file with key with which to encrypt or to sign a request");
@ -227,6 +227,61 @@ Usage(char *progName)
exit(1); exit(1);
} }
#if(0)
/*
* Authenticates to any token that may require it.
* It also checks that the NSS database ahs been initialized.
* This function is modeled after the one in libcurl.
*/
static SECStatus nss_Init_Tokens(secuPWData *pwdata)
{
PK11SlotList *slotList;
PK11SlotListElement *listEntry;
SECStatus ret, status = SECSuccess;
PK11_SetPasswordFunc(SECU_GetModulePassword);
/* List all currently available tokens and traverse
* the list authenticating to them
*/
slotList = PK11_GetAllTokens(CKM_INVALID_MECHANISM, PR_FALSE, PR_TRUE, NULL);
for (listEntry = PK11_GetFirstSafe(slotList);
listEntry; listEntry = listEntry->next) {
PK11SlotInfo *slot = listEntry->slot;
if (PK11_NeedLogin(slot) && PK11_NeedUserInit(slot)) {
if (slot == PK11_GetInternalKeySlot()) {
SECU_PrintError(progName,
"The NSS database has not been initialized\n");
} else {
SECU_PrintError(progName,
"The token %s has not been initialized",
PK11_GetTokenName(slot));
}
PK11_FreeSlot(slot);
continue;
}
ret = PK11_Authenticate(slot, PR_TRUE, &pwdata);
if (SECSuccess != ret) {
if (PR_GetError() == SEC_ERROR_BAD_PASSWORD) {
SECU_PrintError(progName,
"%s: The password for token '%s' is incorrect\n",
PK11_GetTokenName(slot));
}
status = SECFailure;
break;
}
PK11_FreeSlot(slot);
}
return status;
}
#endif
/* /*
* Loads the cert from the specified file into the module at * Loads the cert from the specified file into the module at
* the specified slot. * the specified slot.
@ -271,9 +326,8 @@ static SECStatus loadCert(
genericObjCert = PK11_CreateGenericObject(slot, theCertTemplate, 4, PR_FALSE /* isPerm */); genericObjCert = PK11_CreateGenericObject(slot, theCertTemplate, 4, PR_FALSE /* isPerm */);
if (!genericObjCert) { if (!genericObjCert) {
rv = PR_GetError(); rv = PR_GetError();
PR_fprintf(PR_STDERR, SECU_PrintError(progName,
"%s: unable to Create object for cert, (%s)\n", "Unable to create object for cert, (%s)", SECU_Strerror(rv));
progName, SECU_Strerror(rv));
break; break;
} }
if (!cacert) { if (!cacert) {
@ -282,9 +336,8 @@ static SECStatus loadCert(
*/ */
cert = PK11_FindCertFromNickname((char *)nickname, NULL); cert = PK11_FindCertFromNickname((char *)nickname, NULL);
if (!cert) { if (!cert) {
PR_fprintf(PR_STDERR, SECU_PrintError(progName,
"%s: Can't find cert named (%s), bailing out\n", "%s: Can't find cert named (%s), bailing out\n", nickname);
progName, nickname);
rv = 255; rv = 255;
break; break;
} else { } else {
@ -338,9 +391,8 @@ static SECStatus loadKey(
if (!object) { if (!object) {
rv = SEC_ERROR_BAD_KEY; rv = SEC_ERROR_BAD_KEY;
PR_SetError(rv, 0); PR_SetError(rv, 0);
PR_fprintf(PR_STDERR, SECU_PrintError(progName,
"%s: unable to create key object (%s)\n", "Unable to create key object (%s)\n", SECU_Strerror(rv));
progName, SECU_Strerror(rv));
break; break;
} }
@ -349,9 +401,10 @@ static SECStatus loadKey(
isPresent = PK11_IsPresent(slot); isPresent = PK11_IsPresent(slot);
assert(isPresent); assert(isPresent);
rv = PK11_Authenticate(slot, PR_TRUE, pwdata->data); rv = PK11_Authenticate(slot, PR_TRUE, pwdata);
if (rv != SECSuccess) { if (rv != SECSuccess) {
PR_fprintf(PR_STDERR, "Can't authenticate\n"); SECU_PrintError(progName,
"Can't authenticate\n", SECU_Strerror(rv));
break; break;
} }
@ -361,12 +414,11 @@ static SECStatus loadKey(
/* Can we find the key? */ /* Can we find the key? */
privkey = PK11_FindPrivateKeyFromCert(slot, cert, pwdata->data); privkey = PK11_FindPrivateKeyFromCert(slot, cert, pwdata);
if (!privkey) { if (!privkey) {
rv = PR_GetError(); rv = PR_GetError();
PR_fprintf(PR_STDERR, SECU_PrintError(progName,
"%s: unable to find the key for cert, (%s)\n", "Unable to find the key for cert, (%s)\n", SECU_Strerror(rv));
progName, SECU_Strerror(rv));
GEN_BREAK(SECFailure); GEN_BREAK(SECFailure);
} }
rv = SECSuccess; rv = SECSuccess;
@ -421,7 +473,7 @@ loadCertAndKey(
* *
* @param nickname the certificate nickname * @param nickname the certificate nickname
* @param slot the slot where keys it was loaded * @param slot the slot where keys it was loaded
* @param pwdat password to authenication into slot * @param pwdat module authentication password
* @param privkey private key out * @param privkey private key out
* @param pubkey public key out * @param pubkey public key out
* @param subject subject out * @param subject subject out
@ -429,7 +481,7 @@ loadCertAndKey(
static SECStatus extractRSAKeysAndSubject( static SECStatus extractRSAKeysAndSubject(
const char *nickname, const char *nickname,
PK11SlotInfo *slot, PK11SlotInfo *slot,
void *pwdata, secuPWData *pwdata,
SECKEYPrivateKey **privkey, SECKEYPrivateKey **privkey,
SECKEYPublicKey **pubkey, SECKEYPublicKey **pubkey,
CERTName **subject) CERTName **subject)
@ -445,23 +497,23 @@ static SECStatus extractRSAKeysAndSubject(
*pubkey = CERT_ExtractPublicKey(cert); *pubkey = CERT_ExtractPublicKey(cert);
if (!*pubkey) { if (!*pubkey) {
PR_fprintf(PR_STDERR, SECU_PrintError(progName,
"%s: Could not get public key from cert, (%s)\n", "Could not get public key from cert, (%s)\n",
progName, SECU_Strerror(PR_GetError())); SECU_Strerror(PR_GetError()));
GEN_BREAK(SECFailure); GEN_BREAK(SECFailure);
} }
*privkey = PK11_FindKeyByDERCert(slot, cert, &pwdata); *privkey = PK11_FindKeyByDERCert(slot, cert, pwdata);
if (!*privkey) { if (!*privkey) {
rv = PR_GetError(); rv = PR_GetError();
PR_fprintf(PR_STDERR, SECU_PrintError(progName,
"%s: unable to find the key with PK11_FindKeyByDERCert, (%s)\n", "Unable to find the key with PK11_FindKeyByDERCert, (%s)\n",
progName, SECU_Strerror(rv)); SECU_Strerror(rv));
*privkey= PK11_FindKeyByAnyCert(cert, &pwdata); *privkey= PK11_FindKeyByAnyCert(cert, &pwdata);
rv = PR_GetError(); rv = PR_GetError();
PR_fprintf(PR_STDERR, SECU_PrintError(progName,
"%s: unable to find the key with PK11_FindKeyByAnyCert, (%s)\n", "Unable to find the key with PK11_FindKeyByAnyCert, (%s)\n",
progName, SECU_Strerror(rv)); SECU_Strerror(rv));
GEN_BREAK(SECFailure); GEN_BREAK(SECFailure);
} }
@ -469,8 +521,8 @@ static SECStatus extractRSAKeysAndSubject(
*subject = CERT_AsciiToName(cert->subjectName); *subject = CERT_AsciiToName(cert->subjectName);
if (!*subject) { if (!*subject) {
PR_fprintf(PR_STDERR, SECU_PrintError(progName,
"%s -s: improperly formatted name: \"%s\"\n", "Improperly formatted name: \"%s\"\n",
progName, cert->subjectName); progName, cert->subjectName);
GEN_BREAK(SECFailure); GEN_BREAK(SECFailure);
} }
@ -755,13 +807,13 @@ SignCert(CERTCertDBHandle *handle, CERTCertificate *cert, PRBool selfsign,
algID = SEC_GetSignatureAlgorithmOidTag(privKey->keyType, hashAlgTag); algID = SEC_GetSignatureAlgorithmOidTag(privKey->keyType, hashAlgTag);
if (algID == SEC_OID_UNKNOWN) { if (algID == SEC_OID_UNKNOWN) {
fprintf(stderr, "Unknown key or hash type for issuer."); SECU_PrintError(progName, "Unknown key or hash type for issuer.");
goto done; goto done;
} }
rv = SECOID_SetAlgorithmID(arena, &cert->signature, algID, 0); rv = SECOID_SetAlgorithmID(arena, &cert->signature, algID, 0);
if (rv != SECSuccess) { if (rv != SECSuccess) {
fprintf(stderr, "Could not set signature algorithm id."); SECU_PrintError(progName, "Could not set signature algorithm id.");
goto done; goto done;
} }
@ -774,13 +826,13 @@ SignCert(CERTCertDBHandle *handle, CERTCertificate *cert, PRBool selfsign,
dummy = SEC_ASN1EncodeItem (arena, &der, cert, dummy = SEC_ASN1EncodeItem (arena, &der, cert,
SEC_ASN1_GET(CERT_CertificateTemplate)); SEC_ASN1_GET(CERT_CertificateTemplate));
if (!dummy) { if (!dummy) {
fprintf (stderr, "Could not encode certificate.\n"); SECU_PrintError(progName, "Could not encode certificate.\n");
goto done; goto done;
} }
result = (SECItem *) PORT_ArenaZAlloc (arena, sizeof (SECItem)); result = (SECItem *) PORT_ArenaZAlloc (arena, sizeof (SECItem));
if (result == NULL) { if (result == NULL) {
fprintf (stderr, "Could not allocate item for certificate data.\n"); SECU_PrintError(progName, "Could not allocate item for certificate data.\n");
goto done; goto done;
} }
@ -898,8 +950,7 @@ CreateCert(
*outCert = subjectCert; *outCert = subjectCert;
} else { } else {
PRErrorCode perr = PR_GetError(); PRErrorCode perr = PR_GetError();
fprintf(stderr, "%s: unable to create cert, (%s)\n", SECU_PrintError(progName, "Unable to create cert, (%s)\n", SECU_Strerror(perr));
progName, SECU_Strerror(perr));
if (subjectCert) if (subjectCert)
CERT_DestroyCertificate (subjectCert); CERT_DestroyCertificate (subjectCert);
} }
@ -1043,7 +1094,7 @@ CERTUTIL_FileForRNG(const char *noise)
fd = PR_Open(noise,PR_RDONLY,0); fd = PR_Open(noise,PR_RDONLY,0);
if (!fd) { if (!fd) {
fprintf(stderr, "%s: failed to open noise file %s\n", progName, noise); SECU_PrintError(progName, "Failed to open noise file %s\n", noise);
return SECFailure; return SECFailure;
} }
@ -1241,9 +1292,7 @@ KeyOut(const char *keyoutfile,
keyOutFile = PR_Open(keyoutfile, PR_RDWR | PR_CREATE_FILE | PR_TRUNCATE, 00660); keyOutFile = PR_Open(keyoutfile, PR_RDWR | PR_CREATE_FILE | PR_TRUNCATE, 00660);
if (!keyOutFile) { if (!keyOutFile) {
PR_fprintf(PR_STDERR, SECU_PrintError(progName, "Unable to open \"%s\" for writing\n", keyoutfile);
"%s -o: unable to open \"%s\" for writing\n",
progName, keyoutfile);
GEN_BREAK(255); GEN_BREAK(255);
} }
@ -1251,8 +1300,7 @@ KeyOut(const char *keyoutfile,
algTag, &pwitem, privkey, 1000, pwdata); algTag, &pwitem, privkey, 1000, pwdata);
if (!epki) { if (!epki) {
rv = PORT_GetError(); rv = PORT_GetError();
SECU_PrintError(progName, SECU_PrintError(progName, "Can't export private key info (%d)\n", rv);
"Can't export private key info (%d)\n", rv);
GEN_BREAK(rv); GEN_BREAK(rv);
} }
@ -1263,9 +1311,9 @@ KeyOut(const char *keyoutfile,
/* NULL dest to let it allocate memory for us */ /* NULL dest to let it allocate memory for us */
derEPKI = SEC_ASN1EncodeItem(arenaForEPKI, NULL, epki, derEPKI = SEC_ASN1EncodeItem(arenaForEPKI, NULL, epki,
SECKEY_EncryptedPrivateKeyInfoTemplate); SECKEY_EncryptedPrivateKeyInfoTemplate);
if (rv != SECSuccess) { if (!derEPKI) {
PR_fprintf(PR_STDERR, "%s ASN1 Encode failed (%dl)\n", SECU_PrintError(progName, "%s ASN1 Encode failed (%dl)\n",
progName, rv); SECU_Strerror(PR_GetError()));
GEN_BREAK(rv); GEN_BREAK(rv);
} }
@ -1394,6 +1442,11 @@ static int keyutil_main(
PRBool doCert = certfile != NULL; PRBool doCert = certfile != NULL;
int rv; int rv;
if (access_pwd_file) {
pwdata.source = PW_FROMFILE;
pwdata.data = (char *)access_pwd_file;
}
if (cert_to_renew && input_key_file) { if (cert_to_renew && input_key_file) {
/* /*
* This certificate request is for a renewal, * This certificate request is for a renewal,
@ -1426,7 +1479,7 @@ static int keyutil_main(
&pwdata); &pwdata);
if (rv != SECSuccess) { if (rv != SECSuccess) {
printf("%s: Can't load the key or cert, bailing out\n", progName); SECU_PrintError(progName, "Can't load the key or cert, bailing out\n");
goto shutdown; goto shutdown;
} }
@ -1446,7 +1499,7 @@ static int keyutil_main(
} else { } else {
/* /*
* This a certificate request for a bran-new cert, * This is a certificate signing request for a new cert,
* will generate a key pair * will generate a key pair
*/ */
slot = PK11_GetInternalKeySlot(); /* PK11_GetInternalSlot() ? */ slot = PK11_GetInternalKeySlot(); /* PK11_GetInternalSlot() ? */
@ -1455,16 +1508,15 @@ static int keyutil_main(
keysize, 65537L, (char *)noisefile, &pubkey, &pwdata); keysize, 65537L, (char *)noisefile, &pubkey, &pwdata);
if (!privkey) { if (!privkey) {
PR_fprintf(PR_STDERR, SECU_PrintError(progName,
"%s Keypair generation failed: \"%d\"\n", "Keypair generation failed: \"%d\"\n", PORT_GetError());
progName, PORT_GetError());
rv = 255; rv = 255;
goto shutdown; goto shutdown;
} }
subject = CERT_AsciiToName((char *)subjectstr); subject = CERT_AsciiToName((char *)subjectstr);
if (!subject) { if (!subject) {
PR_fprintf(PR_STDERR, "%s -s: improperly formatted name: \"%s\"\n", SECU_PrintError(progName,
progName, subjectstr); "Improperly formatted name: \"%s\"\n", subjectstr);
rv = 255; rv = 255;
goto shutdown; goto shutdown;
} }
@ -1473,17 +1525,12 @@ static int keyutil_main(
outFile = PR_Open(certreqfile, PR_RDWR | PR_CREATE_FILE | PR_TRUNCATE, 00660); outFile = PR_Open(certreqfile, PR_RDWR | PR_CREATE_FILE | PR_TRUNCATE, 00660);
if (!outFile) { if (!outFile) {
PR_fprintf(PR_STDERR, SECU_PrintError(progName,
"%s -o: unable to open \"%s\" for writing (%ld, %ld)\n", "%s -o: unable to open \"%s\" for writing (%ld, %ld)\n",
progName, certreqfile, certreqfile, PR_GetError(), PR_GetOSError());
PR_GetError(), PR_GetOSError());
return 255; return 255;
} }
printf("Opened %s for writing\n", certreqfile); printf("Opened %s for writing\n", certreqfile);
if (access_pwd_file) {
pwdata.source = PW_FROMFILE;
pwdata.data = (char *)access_pwd_file;
}
/* /*
* Certificate request * Certificate request
@ -1529,7 +1576,7 @@ static int keyutil_main(
inFile = PR_Open(certreqfile, PR_RDONLY, 0); inFile = PR_Open(certreqfile, PR_RDONLY, 0);
assert(inFile); assert(inFile);
if (!inFile) { if (!inFile) {
PR_fprintf(PR_STDERR, "Failed to open file \"%s\" (%ld, %ld) for reading.\n", SECU_PrintError(progName, "Failed to open file \"%s\" (%ld, %ld) for reading.\n",
certreqfile, PR_GetError(), PR_GetOSError()); certreqfile, PR_GetError(), PR_GetOSError());
rv = SECFailure; rv = SECFailure;
goto shutdown; goto shutdown;
@ -1538,7 +1585,7 @@ static int keyutil_main(
outFile = PR_Open(certfile, PR_RDWR | PR_CREATE_FILE | PR_TRUNCATE, 00660); outFile = PR_Open(certfile, PR_RDWR | PR_CREATE_FILE | PR_TRUNCATE, 00660);
if (!outFile) { if (!outFile) {
PR_fprintf(PR_STDERR, "Failed to open file \"%s\" (%ld, %ld).\n", SECU_PrintError(progName, "Failed to open file \"%s\" (%ld, %ld).\n",
certfile, PR_GetError(), PR_GetOSError()); certfile, PR_GetError(), PR_GetOSError());
rv = SECFailure; rv = SECFailure;
goto shutdown; goto shutdown;
@ -1558,7 +1605,7 @@ static int keyutil_main(
ASCIIForIO,SelfSign,certutil_extns, thecert ASCIIForIO,SelfSign,certutil_extns, thecert
*/ */
if (rv) { if (rv) {
PR_fprintf(PR_STDERR, "Failed to create certificate \"%s\" (%ld).\n", SECU_PrintError(progName, "Failed to create certificate \"%s\" (%ld).\n",
outFile, PR_GetError()); outFile, PR_GetError());
rv = SECFailure; rv = SECFailure;
goto shutdown; goto shutdown;
@ -1617,7 +1664,7 @@ shutdown:
return rv == SECSuccess ? 0 : 255; return rv == SECSuccess ? 0 : 255;
} }
/* $Id: keyutil.c,v 1.8 2008/10/19 17:50:08 emaldonado Exp $ */ /* $Id: keyutil.c,v 1.9 2008/10/20 20:45:04 emaldonado Exp $ */
/* Key generation, encryption, and certificate utility code, based on /* Key generation, encryption, and certificate utility code, based on
* code from NSS's security utilities and the certutil application. * code from NSS's security utilities and the certutil application.
@ -1750,6 +1797,10 @@ int main(int argc, char **argv)
PR_Cleanup(); PR_Cleanup();
return EXIT_FAILURE; return EXIT_FAILURE;
} }
if (PK11_IsFIPS() && !access_pwd_file) {
printf("Default module in FIPS mode requires password\n");
return EXIT_FAILURE;
}
} }
initialized = PR_TRUE; initialized = PR_TRUE;