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

Unified Diff: extensions/browser/api/cast_channel/cast_auth_util.h

Issue 2709523008: [Cast Channel] Add support for nonce challenge to Cast channel authentication. (Closed)
Patch Set: Rebase-only Created 3 years, 9 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: extensions/browser/api/cast_channel/cast_auth_util.h
diff --git a/extensions/browser/api/cast_channel/cast_auth_util.h b/extensions/browser/api/cast_channel/cast_auth_util.h
index 1f66395413963a1ea21c95dc40503927dcaf36da..d578c569ac69d3233fd5e6112dbc25c0c1c90233 100644
--- a/extensions/browser/api/cast_channel/cast_auth_util.h
+++ b/extensions/browser/api/cast_channel/cast_auth_util.h
@@ -46,6 +46,7 @@ struct AuthResult {
ERROR_TLS_CERT_EXPIRED,
ERROR_CRL_INVALID,
ERROR_CERT_REVOKED,
+ ERROR_SENDER_NONCE_MISMATCH,
};
enum PolicyType { POLICY_NONE = 0, POLICY_AUDIO_ONLY = 1 << 0 };
@@ -67,11 +68,38 @@ struct AuthResult {
unsigned int channel_policies;
};
+class AuthContext {
+ public:
+ ~AuthContext();
+
+ // Get an auth challenge context.
+ // The same context must be used in the challenge and reply.
+ static AuthContext Create();
+
+ // Verifies the nonce received in the response is equivalent to the one sent.
+ // Returns success if |nonce_response| matches nonce_
+ AuthResult VerifySenderNonce(const std::string& nonce_response) const;
+
+ // The nonce challenge.
+ const std::string& nonce() const { return nonce_; }
+
+ private:
+ explicit AuthContext(const std::string& nonce);
+
+ const std::string nonce_;
+};
+
// Authenticates the given |challenge_reply|:
// 1. Signature contained in the reply is valid.
// 2. Certficate used to sign is rooted to a trusted CA.
AuthResult AuthenticateChallengeReply(const CastMessage& challenge_reply,
- const net::X509Certificate& peer_cert);
+ const net::X509Certificate& peer_cert,
+ const AuthContext& auth_context);
+
+// Performs a quick check of the TLS certificate for time validity requirements.
+AuthResult VerifyTLSCertificate(const net::X509Certificate& peer_cert,
+ std::string* peer_cert_der,
+ const base::Time& verification_time);
// Auth-library specific implementation of cryptographic signature
// verification routines. Verifies that |response| contains a

Powered by Google App Engine
This is Rietveld 408576698