bug 1339010 - ensure pinning and CT telemetry info has been initialized r?jcj draft
authorDavid Keeler <dkeeler@mozilla.com>
Mon, 13 Feb 2017 16:47:43 -0800
changeset 483197 3879f91fe725fdb2d48c4c01f2c878a3964e65ba
parent 482882 e1a4314f8e6eae8bbc06394c14132a9c5011371b
child 545597 9d1b63da61f0c5aff7d93f9acefcc4ebd9e65c9f
push id45265
push userdkeeler@mozilla.com
push dateTue, 14 Feb 2017 00:56:36 +0000
reviewersjcj
bugs1339010
milestone54.0a1
bug 1339010 - ensure pinning and CT telemetry info has been initialized r?jcj MozReview-Commit-ID: F7pOqCK145n
security/certverifier/CertVerifier.h
--- a/security/certverifier/CertVerifier.h
+++ b/security/certverifier/CertVerifier.h
@@ -59,30 +59,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;
   // Verification result of the processed SCTs.
   mozilla::ct::CTVerifyResult verifyResult;
   // Connection compliance to the CT Policy.
   mozilla::ct::CTPolicyCompliance policyCompliance;
 
   void Reset();