Fixes NullReferenceException in AspNet.OpenAuthSecurityManager.

This was reported at: http://stackoverflow.com/questions/12235395/openauth-requestauthentication-throws-null-ref-in-vs2012-web-forms-template/12240150#12240150
It is due to fields being left uninitialized by the constructors yet assumed to be non-null by all or some public methods.

1 files changed, 3 insertions, 12 deletions

diff --git a/src/DotNetOpenAuth.AspNet/OpenAuthSecurityManager.cs b/src/DotNetOpenAuth.AspNet/OpenAuthSecurityManager.cs
index 6a898a1..2d18b7b 100644
--- a/src/DotNetOpenAuth.AspNet/OpenAuthSecurityManager.cs
+++ b/src/DotNetOpenAuth.AspNet/OpenAuthSecurityManager.cs
@@ -64,15 +64,6 @@ namespace DotNetOpenAuth.AspNet {
 		/// <param name="requestContext">
 		/// The request context. 
 		/// </param>
-		public OpenAuthSecurityManager(HttpContextBase requestContext)
-			: this(requestContext, provider: null, dataProvider: null) { }
-
-		/// <summary>
-		/// Initializes a new instance of the <see cref="OpenAuthSecurityManager"/> class.
-		/// </summary>
-		/// <param name="requestContext">
-		/// The request context. 
-		/// </param>
 		/// <param name="provider">
 		/// The provider. 
 		/// </param>
@@ -81,9 +72,9 @@ namespace DotNetOpenAuth.AspNet {
 		/// </param>
 		public OpenAuthSecurityManager(
 			HttpContextBase requestContext, IAuthenticationClient provider, IOpenAuthDataProvider dataProvider) {
-			if (requestContext == null) {
-				throw new ArgumentNullException("requestContext");
-			}
+			Requires.NotNull(requestContext, "requestContext");
+			Requires.NotNull(provider, "provider");
+			Requires.NotNull(dataProvider, "dataProvider");
 
 			this.requestContext = requestContext;
 			this.dataProvider = dataProvider;