Bug 1339010 - Ensure pinning and CT telemetry info has been initialized. r=jcj, a=gchang
authorDavid Keeler <dkeeler@mozilla.com>
Wed, 15 Feb 2017 11:02:43 -0800
changeset 376276 4c83176013d37417bf9724bc970295b80ead7776
parent 376275 d3470b4a561d9cf753ee4234db7a4babc4d06e14
child 376277 a88c1bc559da79f86c4a55be3b3a01ec1fc44ec6
push id6996
push userjlorenzo@mozilla.com
push dateMon, 06 Mar 2017 20:48:21 +0000
treeherdermozilla-beta@d89512dab048 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjcj, gchang
bugs1339010
milestone53.0a2
Bug 1339010 - Ensure pinning and CT telemetry info has been initialized. r=jcj, a=gchang
security/certverifier/CertVerifier.h
--- a/security/certverifier/CertVerifier.h
+++ b/security/certverifier/CertVerifier.h
@@ -57,30 +57,34 @@ enum class SHA1ModeResult {
   Failed = 5,
 };
 
 enum class NetscapeStepUpPolicy : uint32_t;
 
 class PinningTelemetryInfo
 {
 public:
+  PinningTelemetryInfo() { Reset(); }
+
   // Should we accumulate pinning telemetry for the result?
   bool accumulateResult;
   Telemetry::ID certPinningResultHistogram;
   int32_t certPinningResultBucket;
   // Should we accumulate telemetry for the root?
   bool accumulateForRoot;
   int32_t rootBucket;
 
   void Reset() { accumulateForRoot = false; accumulateResult = false; }
 };
 
 class CertificateTransparencyInfo
 {
 public:
+  CertificateTransparencyInfo() { Reset(); }
+
   // Was CT enabled?
   bool enabled;
   // Did we receive and process any binary SCT data from the supported sources?
   bool processedSCTs;
   // Verification result of the processed SCTs.
   mozilla::ct::CTVerifyResult verifyResult;
 
   void Reset() { enabled = false; processedSCTs = false; verifyResult.Reset(); }