From 1ffd443d5aaec408170eaec31923a1cbbe1bb929 Mon Sep 17 00:00:00 2001
From: Phallacy <Dragoonmac@gmail.com>
Date: Tue, 12 Feb 2019 22:30:26 -0800
Subject: [PATCH] fixed nul user check to be first per justaman

---
 .../Library/DefaultAuthenticationProvider.cs             | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/Emby.Server.Implementations/Library/DefaultAuthenticationProvider.cs b/Emby.Server.Implementations/Library/DefaultAuthenticationProvider.cs
index 92346c65aa..255fd82527 100644
--- a/Emby.Server.Implementations/Library/DefaultAuthenticationProvider.cs
+++ b/Emby.Server.Implementations/Library/DefaultAuthenticationProvider.cs
@@ -32,15 +32,16 @@ namespace Emby.Server.Implementations.Library
 
         //This is the verson that we need to use for local users. Because reasons.
         public Task<ProviderAuthenticationResult> Authenticate(string username, string password, User resolvedUser)
-        {
-            ConvertPasswordFormat(resolvedUser);
-            byte[] passwordbytes = Encoding.UTF8.GetBytes(password);
-            bool success = false;
+        {
+            bool success = false;
             if (resolvedUser == null)
             {
                 success = false;
                 throw new Exception("Invalid username or password");
             }
+            ConvertPasswordFormat(resolvedUser);
+            byte[] passwordbytes = Encoding.UTF8.GetBytes(password);
+           
             if (!resolvedUser.Password.Contains("$"))
             {
                 ConvertPasswordFormat(resolvedUser);