Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(148)

Unified Diff: crypto/signature_creator_nss.cc

Issue 1305183005: Add a CreatePSS method to SignatureCreator to permit the generation of PSS signatures. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: crypto/signature_creator_nss.cc
diff --git a/crypto/signature_creator_nss.cc b/crypto/signature_creator_nss.cc
index da03312881dbb9a5009c08e78023b5bd897183c8..c5a05c117de190d94e7506f0a6f51f6d456dbb87 100644
--- a/crypto/signature_creator_nss.cc
+++ b/crypto/signature_creator_nss.cc
@@ -46,9 +46,25 @@ SignatureCreator::~SignatureCreator() {
}
}
-// static
SignatureCreator* SignatureCreator::Create(RSAPrivateKey* key,
- HashAlgorithm hash_alg) {
+ HashAlgorithm hash_algm) {
davidben 2015/08/21 21:52:22 hash_alg, to match the others?
+ return SignatureCreator::CreateImpl(key, hash_algm, false);
+}
+
+SignatureCreator* SignatureCreator::CreatePSS(RSAPrivateKey* key,
+ HashAlgorithm hash_algm) {
+ return SignatureCreator::CreateImpl(key, hash_algm, true);
+}
+
+// static
+SignatureCreator* SignatureCreator::CreateImpl(RSAPrivateKey* key,
+ HashAlgorithm hash_alg,
+ bool use_pss) {
+ if (use_pss) {
+ // TODO(rch): make this work :>
+ NOTREACHED();
+ return NULL;
davidben 2015/08/21 21:52:22 Pfft. Judging by crypto/signature_verifier_nss.cc,
+ }
scoped_ptr<SignatureCreator> result(new SignatureCreator);
result->sign_context_ = SGN_NewContext(ToNSSSigOid(hash_alg), key->key());
if (!result->sign_context_) {

Powered by Google App Engine
This is Rietveld 408576698