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

Unified Diff: components/rappor/byte_vector_utils.h

Issue 49753002: RAPPOR implementation (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 10 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/rappor/byte_vector_utils.h
diff --git a/components/rappor/byte_vector_utils.h b/components/rappor/byte_vector_utils.h
new file mode 100644
index 0000000000000000000000000000000000000000..197fbc9a344b9e1c511ccef0a4aa8243def954cb
--- /dev/null
+++ b/components/rappor/byte_vector_utils.h
@@ -0,0 +1,97 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef COMPONENTS_RAPPOR_BYTE_VECTOR_UTILS_H_
+#define COMPONENTS_RAPPOR_BYTE_VECTOR_UTILS_H_
+
+#include <stddef.h>
+#include <stdint.h>
+#include <vector>
+
+#include "components/rappor/rappor_parameters.h"
+#include "crypto/hmac.h"
+
+namespace rappor {
+
+// A vector of 8-bit integers used to store a set of binary bits.
+typedef std::vector<uint8_t> ByteVector;
+
+// Computes a bitwise OR of byte vectors and stores the result in rhs.
+ByteVector* ByteVectorOr(const ByteVector& lhs, ByteVector* rhs);
+
+// Merges the contents of lhs and rhs vectors according to a mask vector.
+// Equivalent to (lhs & ~mask) | (rhs & mask). Stores the result in rhs.
+ByteVector* ByteVectorMerge(const ByteVector& mask,
+ const ByteVector& lhs,
+ ByteVector* rhs);
+
+// Counts the number of bits set in the byte vector.
+int CountBits(const ByteVector& vector);
+
+// A utility object for generating random binary data with different
+// likelihood of bits being true, using entropy from crypto::RandBytes().
+class ByteVectorGenerator {
+ public:
+ explicit ByteVectorGenerator(size_t byte_count);
+
+ ~ByteVectorGenerator();
+
+ // Generates a random byte vector where the bits are independent random
+ // variables which are true with the given |probability|.
+ ByteVector GetWeightedRandomByteVector(Probability probability);
+
+ protected:
+ // Generates a random vector of bytes from a uniform distribution.
+ virtual ByteVector GetRandomByteVector();
+
+ size_t byte_count_;
+
+ DISALLOW_COPY_AND_ASSIGN(ByteVectorGenerator);
+};
+
+// A ByteVectorGenerator that uses a psuedo-random function to generate a
+// deterministically random bits. This class only implements a single request
+// from HMAC_DRBG and streams up to 2^19 bits from that request.
+// Ref: http://csrc.nist.gov/publications/nistpubs/800-90A/SP800-90A.pdf
+// We're using our own PRNG instead of crypto::RandBytes because we need to
+// generate a repeatable sequence of bits from the same seed. Conservatively,
+// we're choosing to use HMAC_DRBG here, as it is one of the best studied
+// and standardized ways of generating deterministic, unpredictable sequences
+// based on a secret seed.
+class HmacByteVectorGenerator : public ByteVectorGenerator {
+ public:
+ // Constructor takes the size of the vector to generate, along with a
+ // secret value to seed the pseudo-random number generator. |secret| is
+ // treated as an array of bytes.
+ HmacByteVectorGenerator(size_t byte_count,
+ const std::string& entropy_input,
+ const std::string& personalization_string);
+
+ ~HmacByteVectorGenerator();
+
+ static std::string GenerateEntropyInput();
+
+ // Key size required for 128-bit security strength (including nonce).
+ static const size_t kEntropyInputSize;
+
+ protected:
+ virtual ByteVector GetRandomByteVector() OVERRIDE;
+
+ private:
+ // HMAC_DRBG_Update
Alexei Svitkine (slow) 2014/02/12 18:13:05 Expand comment to briefly explain what this does (
Steven Holte 2014/02/12 22:28:50 Done.
+ void Update(const std::string& provided_data, const ByteVector& key);
+
+ // HMAC initalized with the value of "Key" HMAC_DRBG_Initialize.
+ crypto::HMAC hmac_;
+ // The "V" value from HMAC_DRBG.
+ ByteVector value_;
+ // Total number of bytes streamed from the HMAC_DRBG Generate Process
Alexei Svitkine (slow) 2014/02/12 18:13:05 Nit: add . at the end
Steven Holte 2014/02/12 22:28:50 Done.
+ size_t requested_bytes_;
Alexei Svitkine (slow) 2014/02/12 18:13:05 Nit: I'd use a different name for this, "requested
Steven Holte 2014/02/12 22:28:50 Done.
+
+ DISALLOW_COPY_AND_ASSIGN(HmacByteVectorGenerator);
+};
+
+} // namespace rappor
+
+#endif // COMPONENTS_RAPPOR_BYTE_VECTOR_UTILS_H_

Powered by Google App Engine
This is Rietveld 408576698