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

Unified Diff: components/certificate_transparency/single_tree_tracker.cc

Issue 2017563002: Add Certificate Transparency logs auditing (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Marking comparator const Created 4 years, 6 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 side-by-side diff with in-line comments
Download patch
Index: components/certificate_transparency/single_tree_tracker.cc
diff --git a/components/certificate_transparency/single_tree_tracker.cc b/components/certificate_transparency/single_tree_tracker.cc
index 7946208ca753303a7589260e7a60fd34a58bb5e0..f7728c3fb884040c641245bf300af690ea72fcce 100644
--- a/components/certificate_transparency/single_tree_tracker.cc
+++ b/components/certificate_transparency/single_tree_tracker.cc
@@ -4,12 +4,16 @@
#include "components/certificate_transparency/single_tree_tracker.h"
+#include <algorithm>
+#include <iterator>
#include <utility>
#include "net/cert/ct_log_verifier.h"
#include "net/cert/signed_certificate_timestamp.h"
#include "net/cert/x509_certificate.h"
+using net::ct::LogEntry;
+using net::ct::MerkleTreeLeaf;
using net::ct::SignedTreeHead;
namespace certificate_transparency {
@@ -25,26 +29,31 @@ void SingleTreeTracker::OnSCTVerified(
const net::ct::SignedCertificateTimestamp* sct) {
DCHECK_EQ(ct_log_->key_id(), sct->log_id);
+ MerkleTreeLeaf leaf;
+ if (!GetMerkleTreeLeaf(cert, sct, &leaf))
+ return;
+
// SCT was previously observed, so its status should not be changed.
- if (entries_status_.find(sct->timestamp) != entries_status_.end())
+ if (LeafAlreadyEncountered(leaf))
return;
+ SCTInclusionStatus state = SCT_NOT_OBSERVED;
+ const base::TimeDelta kMaximumMergeDelay = base::TimeDelta::FromHours(24);
// If there isn't a valid STH or the STH is not fresh enough to check
// inclusion against, store the SCT for future checking and return.
if (verified_sth_.timestamp.is_null() ||
- (verified_sth_.timestamp <
- (sct->timestamp + base::TimeDelta::FromHours(24)))) {
+ (verified_sth_.timestamp < (sct->timestamp + kMaximumMergeDelay))) {
// TODO(eranm): UMA - how often SCTs have to wait for a newer STH for
// inclusion check.
- entries_status_.insert(
- std::make_pair(sct->timestamp, SCT_PENDING_NEWER_STH));
- return;
+ state = SCT_PENDING_NEWER_STH;
+ } else {
+ // TODO(eranm): UMA - how often inclusion can be checked immediately.
+ state = SCT_PENDING_INCLUSION_CHECK;
}
- // TODO(eranm): Check inclusion here.
- // TODO(eranm): UMA - how often inclusion can be checked immediately.
- entries_status_.insert(
- std::make_pair(sct->timestamp, SCT_PENDING_INCLUSION_CHECK));
+ // TODO(eranm): Check inclusion here, see https://crbug.com/624894
+ observed_entries_.insert(
+ std::make_pair(std::move(leaf), LeafState(state, base::Time::Now())));
}
void SingleTreeTracker::NewSTHObserved(const SignedTreeHead& sth) {
@@ -74,27 +83,104 @@ void SingleTreeTracker::NewSTHObserved(const SignedTreeHead& sth) {
}
// Find out which SCTs can now be checked for inclusion.
- // TODO(eranm): Keep two maps of MerkleTreeLeaf instances, one for leaves
- // pending inclusion checks and one for leaves pending a new STH.
- // The comparison function between MerkleTreeLeaf instances should use the
- // timestamp to determine sorting order, so that bulk moving from one
- // map to the other can happen.
- auto entry = entries_status_.begin();
- while (entry != entries_status_.end() &&
- entry->first < verified_sth_.timestamp) {
- entry->second = SCT_PENDING_INCLUSION_CHECK;
- ++entry;
- // TODO(eranm): Check inclusion here.
+ auto first_entry_not_included = std::find_if(
+ observed_entries_.begin(), observed_entries_.end(),
+ [&sth](const std::pair<const MerkleTreeLeaf&, const LeafState&> entry) {
+ return entry.first.timestamp > sth.timestamp;
+ });
+ for (auto it = observed_entries_.begin(); it != first_entry_not_included;
+ ++it) {
+ it->second.SetState(SCT_PENDING_INCLUSION_CHECK);
}
+
+ // TODO(eranm): Check inclusion here of entries that can now be checked.
+ // See https://crbug.com/624894
}
SingleTreeTracker::SCTInclusionStatus
SingleTreeTracker::GetLogEntryInclusionStatus(
net::X509Certificate* cert,
const net::ct::SignedCertificateTimestamp* sct) {
- auto it = entries_status_.find(sct->timestamp);
+ MerkleTreeLeaf leaf;
+ if (!GetMerkleTreeLeaf(cert, sct, &leaf))
+ return SCT_NOT_OBSERVED;
+
+ auto entry_iterator = observed_entries_.find(leaf);
+ if (entry_iterator == observed_entries_.end())
+ return SCT_NOT_OBSERVED;
+
+ return entry_iterator->second.State();
+}
+
+SingleTreeTracker::LeafState::LeafState(SCTInclusionStatus status,
+ const base::Time& observed_at)
+ : observed_at_(observed_at), state_(status) {}
+
+SingleTreeTracker::LeafState::LeafState(const LeafState& other) = default;
+
+SingleTreeTracker::LeafState::LeafState(LeafState&&) = default;
+
+SingleTreeTracker::LeafState::~LeafState() = default;
+
+void SingleTreeTracker::LeafState::SetLeafIndex(uint64_t index) {
+ // The leaf index may be set only once. Doing so more than once indicates
+ // a programming error where the index is fetched more than once for a given
+ // leaf.
+ DCHECK(!leaf_index_);
+ leaf_index_ = index;
+}
+
+bool SingleTreeTracker::LeafState::HasLeafIndex() const {
+ return bool(leaf_index_);
+}
+
+uint64_t SingleTreeTracker::LeafState::GetLeafIndex() const {
+ DCHECK(leaf_index_);
+ return leaf_index_.value();
+}
+
+const base::Time& SingleTreeTracker::LeafState::ObservedAt() const {
+ return observed_at_;
+}
+
+SingleTreeTracker::SCTInclusionStatus SingleTreeTracker::LeafState::State()
+ const {
+ return state_;
+}
+
+void SingleTreeTracker::LeafState::SetState(
+ SingleTreeTracker::SCTInclusionStatus state) {
+ DCHECK_NE(state, SCT_NOT_OBSERVED);
+ DCHECK_NE(state, state_);
+ state_ = state;
+}
+
+bool SingleTreeTracker::OrderByTimestamp::operator()(
+ const MerkleTreeLeaf& lhs,
+ const MerkleTreeLeaf& rhs) const {
+ // This comparator should only used for containers where leaves from
+ // different logs are not mixed.
+ DCHECK_EQ(lhs.log_id, rhs.log_id);
+
+ if (lhs.timestamp != rhs.timestamp)
+ return lhs.timestamp < rhs.timestamp;
+
+ const LogEntry& lhs_entry = lhs.log_entry;
+ const LogEntry& rhs_entry = rhs.log_entry;
+ if (lhs_entry.type != rhs_entry.type)
+ return lhs_entry.type < rhs_entry.type;
+
+ if (lhs_entry.type == LogEntry::LOG_ENTRY_TYPE_X509)
+ return lhs_entry.leaf_certificate < rhs_entry.leaf_certificate;
+
+ // lhs_entry.type == LOG_ENTRY_TYPE_PRECERT
+ return lhs_entry.tbs_certificate < rhs_entry.tbs_certificate &&
+ net::SHA256HashValueLessThan()(lhs_entry.issuer_key_hash,
+ rhs_entry.issuer_key_hash);
+}
- return it == entries_status_.end() ? SCT_NOT_OBSERVED : it->second;
+bool SingleTreeTracker::LeafAlreadyEncountered(const MerkleTreeLeaf& leaf) {
+ return observed_entries_.find(leaf) != observed_entries_.end();
}
} // namespace certificate_transparency

Powered by Google App Engine
This is Rietveld 408576698