From 28324e5174f22dba7dbcfd152fc3d13703aaed0f Mon Sep 17 00:00:00 2001
From: Dr Scofield
Date: Tue, 9 Jun 2009 12:11:22 +0000
Subject: ARGH!!!! note to self: ALWAYS use String.IsNullOrEmpty(...)!

this patch fixes RemoteAdminPlugin gratuitously updating the user
password even if no update was intended.

---
 OpenSim/ApplicationPlugins/RemoteController/RemoteAdminPlugin.cs | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/OpenSim/ApplicationPlugins/RemoteController/RemoteAdminPlugin.cs b/OpenSim/ApplicationPlugins/RemoteController/RemoteAdminPlugin.cs
index 47240a1..f7171ff 100644
--- a/OpenSim/ApplicationPlugins/RemoteController/RemoteAdminPlugin.cs
+++ b/OpenSim/ApplicationPlugins/RemoteController/RemoteAdminPlugin.cs
@@ -1115,8 +1115,9 @@ namespace OpenSim.ApplicationPlugins.RemoteController
                     if (null == userProfile)
                         throw new Exception(String.Format("avatar {0} {1} does not exist", firstname, lastname));
 
-                    if (null != passwd)
+                    if (!String.IsNullOrEmpty(passwd))
                     {
+                        m_log.DebugFormat("[RADMIN]: UpdateUserAccount: updating password for avatar {0} {1}", firstname, lastname);
                         string md5PasswdHash = Util.Md5Hash(Util.Md5Hash(passwd) + ":" + String.Empty);
                         userProfile.PasswordHash = md5PasswdHash;
                     }
-- 
cgit v1.1