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

Side by Side Diff: crypto/ec_signature_creator_openssl.cc

Issue 26911006: crypto: Implement ECSignatureCreatorImpl for OpenSSL (Closed) Base URL: https://codereview.chromium.org/27195002/
Patch Set: Formatting Created 7 years, 2 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 unified diff | Download patch
« no previous file with comments | « no previous file | crypto/ec_signature_creator_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "crypto/ec_signature_creator_impl.h" 5 #include "crypto/ec_signature_creator_impl.h"
6 6
7 #include <openssl/bn.h>
8 #include <openssl/ec.h>
9 #include <openssl/ecdsa.h>
10 #include <openssl/evp.h>
11 #include <openssl/sha.h>
12
7 #include "base/logging.h" 13 #include "base/logging.h"
14 #include "crypto/ec_private_key.h"
15 #include "crypto/openssl_util.h"
8 16
9 namespace crypto { 17 namespace crypto {
10 18
11 ECSignatureCreatorImpl::ECSignatureCreatorImpl(ECPrivateKey* key) 19 ECSignatureCreatorImpl::ECSignatureCreatorImpl(ECPrivateKey* key)
12 : key_(key) { 20 : key_(key), signature_len_(0) {
13 NOTIMPLEMENTED(); 21 EnsureOpenSSLInit();
14 } 22 }
15 23
16 ECSignatureCreatorImpl::~ECSignatureCreatorImpl() {} 24 ECSignatureCreatorImpl::~ECSignatureCreatorImpl() {}
17 25
18 bool ECSignatureCreatorImpl::Sign(const uint8* data, 26 bool ECSignatureCreatorImpl::Sign(const uint8* data,
19 int data_len, 27 int data_len,
20 std::vector<uint8>* signature) { 28 std::vector<uint8>* signature) {
21 NOTIMPLEMENTED(); 29 OpenSSLErrStackTracer err_tracer(FROM_HERE);
22 return false; 30 ScopedOpenSSL<EVP_MD_CTX, EVP_MD_CTX_destroy> ctx(EVP_MD_CTX_create());
31 size_t sig_len = 0;
32 if (!ctx.get() ||
33 !EVP_DigestSignInit(ctx.get(), NULL, EVP_sha256(), NULL, key_->key()) ||
34 !EVP_DigestSignUpdate(ctx.get(), data, data_len) ||
35 !EVP_DigestSignFinal(ctx.get(), NULL, &sig_len))
36 return false;
37
38 signature->resize(sig_len);
39 if (!EVP_DigestSignFinal(ctx.get(), &(*signature)[0], &sig_len))
agl 2013/10/15 16:24:08 &signature->front() might be easier on the eyes? U
digit1 2013/10/15 19:32:27 Done.
40 return false;
41
42 return true;
23 } 43 }
24 44
25 bool ECSignatureCreatorImpl::DecodeSignature(const std::vector<uint8>& der_sig, 45 bool ECSignatureCreatorImpl::DecodeSignature(const std::vector<uint8>& der_sig,
26 std::vector<uint8>* out_raw_sig) { 46 std::vector<uint8>* out_raw_sig) {
27 NOTIMPLEMENTED(); 47 OpenSSLErrStackTracer err_tracer(FROM_HERE);
28 return false; 48 // Create ECDSA_SIG object from DER-encoded data.
49 const unsigned char* der_data =
50 reinterpret_cast<const unsigned char*>(der_sig.front());
51 ScopedOpenSSL<ECDSA_SIG, ECDSA_SIG_free> ecdsa_sig(
52 d2i_ECDSA_SIG(NULL, &der_data, static_cast<long>(der_sig.size())));
53 if (!ecdsa_sig.get())
54 return false;
55
56 // The result is made of two 256-bit vectors.
57 const size_t kMaxBitsPerBN = 256;
58 const size_t kMaxBytesPerBN = (kMaxBitsPerBN + 7) / 8;
59 std::vector<uint8> result;
60 result.resize(2 * kMaxBytesPerBN);
61 memset(&result[0], 0, result.size());
62
63 // NOTE: Can't really check for equality here since sometimes the value
agl 2013/10/15 16:24:08 Move this comment down four lines?
digit1 2013/10/15 19:32:27 Done.
64 // returned by BN_num_bytes() will be slightly smaller than kMaxBytesPerBN.
65 BIGNUM* r = ecdsa_sig.get()->r;
66 BIGNUM* s = ecdsa_sig.get()->s;
67 int r_bytes = BN_num_bytes(r);
68 int s_bytes = BN_num_bytes(s);
69 if (r_bytes > static_cast<int>(kMaxBytesPerBN) ||
70 s_bytes > static_cast<int>(kMaxBytesPerBN)) {
71 DLOG(ERROR) << "Invalid key sizes r(" << r_bytes << ") s(" << s_bytes
72 << ")";
73 return false;
74 }
75 BN_bn2bin(ecdsa_sig.get()->r, &result[kMaxBytesPerBN - r_bytes]);
76 BN_bn2bin(ecdsa_sig.get()->s, &result[2 * kMaxBytesPerBN - s_bytes]);
77 out_raw_sig->swap(result);
78 return true;
29 } 79 }
30 80
31 } // namespace crypto 81 } // namespace crypto
OLDNEW
« no previous file with comments | « no previous file | crypto/ec_signature_creator_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698