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

Unified Diff: crypto/p224_spake.cc

Issue 9005015: Coverity: Initialize member variables. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: 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 side-by-side diff with in-line comments
Download patch
Index: crypto/p224_spake.cc
diff --git a/crypto/p224_spake.cc b/crypto/p224_spake.cc
index ec0c416f48f759b822ea58788a9dc4a3541f9997..af3c2b8731e024adcb607de44987bda953f38bac 100644
--- a/crypto/p224_spake.cc
+++ b/crypto/p224_spake.cc
@@ -99,6 +99,9 @@ P224EncryptedKeyExchange::P224EncryptedKeyExchange(
PeerType peer_type, const base::StringPiece& password)
: state_(kStateInitial),
is_server_(peer_type == kPeerTypeServer) {
+ memset(&x_, 0, sizeof(x_));
+ memset(&expected_authenticator_, 0, sizeof(expected_authenticator_));
+
// x_ is a random scalar.
base::RandBytes(x_, sizeof(x_));

Powered by Google App Engine
This is Rietveld 408576698