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

Side by Side Diff: crypto/signature_creator_win.cc

Issue 9005015: Coverity: Initialize member variables. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Build fix. Created 9 years 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 | Annotate | Revision Log
« no previous file with comments | « crypto/p224_spake.cc ('k') | media/base/pipeline_impl.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) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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/signature_creator.h" 5 #include "crypto/signature_creator.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "base/memory/scoped_ptr.h" 8 #include "base/memory/scoped_ptr.h"
9 9
10 namespace crypto { 10 namespace crypto {
11 11
12 // static 12 // static
13 SignatureCreator* SignatureCreator::Create(RSAPrivateKey* key) { 13 SignatureCreator* SignatureCreator::Create(RSAPrivateKey* key) {
14 scoped_ptr<SignatureCreator> result(new SignatureCreator); 14 scoped_ptr<SignatureCreator> result(new SignatureCreator);
15 result->key_ = key; 15 result->key_ = key;
16 16
17 if (!CryptCreateHash(key->provider(), CALG_SHA1, 0, 0, 17 if (!CryptCreateHash(key->provider(), CALG_SHA1, 0, 0,
18 result->hash_object_.receive())) { 18 result->hash_object_.receive())) {
19 NOTREACHED(); 19 NOTREACHED();
20 return NULL; 20 return NULL;
21 } 21 }
22 22
23 return result.release(); 23 return result.release();
24 } 24 }
25 25
26 SignatureCreator::SignatureCreator() : hash_object_(0) {} 26 SignatureCreator::SignatureCreator() : key_(NULL), hash_object_(0) {}
27 27
28 SignatureCreator::~SignatureCreator() {} 28 SignatureCreator::~SignatureCreator() {}
29 29
30 bool SignatureCreator::Update(const uint8* data_part, int data_part_len) { 30 bool SignatureCreator::Update(const uint8* data_part, int data_part_len) {
31 if (!CryptHashData(hash_object_, data_part, data_part_len, 0)) { 31 if (!CryptHashData(hash_object_, data_part, data_part_len, 0)) {
32 NOTREACHED(); 32 NOTREACHED();
33 return false; 33 return false;
34 } 34 }
35 35
36 return true; 36 return true;
(...skipping 14 matching lines...) Expand all
51 } 51 }
52 52
53 temp.resize(signature_length); 53 temp.resize(signature_length);
54 for (size_t i = temp.size(); i > 0; --i) 54 for (size_t i = temp.size(); i > 0; --i)
55 signature->push_back(temp[i - 1]); 55 signature->push_back(temp[i - 1]);
56 56
57 return true; 57 return true;
58 } 58 }
59 59
60 } // namespace crypto 60 } // namespace crypto
OLDNEW
« no previous file with comments | « crypto/p224_spake.cc ('k') | media/base/pipeline_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698