From 67f17eb4bb1c2b18dd80dc50146f60cf18872f6f Mon Sep 17 00:00:00 2001 From: hartvig Date: Sun, 20 Jun 2010 09:43:14 +0000 Subject: [PATCH] Fixes 27687 [TFS Changeset #70186] --- umbraco/providers/UsersMembershipProvider.cs | 17 +++++++++++++---- 1 file changed, 13 insertions(+), 4 deletions(-) diff --git a/umbraco/providers/UsersMembershipProvider.cs b/umbraco/providers/UsersMembershipProvider.cs index fff5a3124e..1842935a34 100644 --- a/umbraco/providers/UsersMembershipProvider.cs +++ b/umbraco/providers/UsersMembershipProvider.cs @@ -564,11 +564,20 @@ namespace umbraco.providers /// public override bool ValidateUser(string username, string password) { - User user = new User(username); - if (user != null && user.Id != -1) + // we need to wrap this in a try/catch as passing a non existing + // user will throw an exception + try { - if (user.Disabled) return false; - else return user.ValidatePassword(EncodePassword(password)); + User user = new User(username); + if (user != null && user.Id != -1) + { + if (user.Disabled) return false; + else return user.ValidatePassword(EncodePassword(password)); + } + } + catch + { + // nothing to catch here - move on } return false;