diff options
author | Jeff Ames | 2008-03-19 09:36:13 +0000 |
---|---|---|
committer | Jeff Ames | 2008-03-19 09:36:13 +0000 |
commit | a0e1be3280e4be49cf0ca8d211b97ef8c1ddee2f (patch) | |
tree | be64babdd6f1bcb2a946d637b81739445facdd31 /OpenSim/Region/Environment/PermissionManager.cs | |
parent | * Add a large amount of extra locking to m_parts in SceneObjectGroup (diff) | |
download | opensim-SC_OLD-a0e1be3280e4be49cf0ca8d211b97ef8c1ddee2f.zip opensim-SC_OLD-a0e1be3280e4be49cf0ca8d211b97ef8c1ddee2f.tar.gz opensim-SC_OLD-a0e1be3280e4be49cf0ca8d211b97ef8c1ddee2f.tar.bz2 opensim-SC_OLD-a0e1be3280e4be49cf0ca8d211b97ef8c1ddee2f.tar.xz |
Fixed some comparisons of LLUUIDs to null.
Thanks to DrSchofld for pointing this out.
Diffstat (limited to 'OpenSim/Region/Environment/PermissionManager.cs')
-rw-r--r-- | OpenSim/Region/Environment/PermissionManager.cs | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/OpenSim/Region/Environment/PermissionManager.cs b/OpenSim/Region/Environment/PermissionManager.cs index a23f119..3d008a7 100644 --- a/OpenSim/Region/Environment/PermissionManager.cs +++ b/OpenSim/Region/Environment/PermissionManager.cs | |||
@@ -82,7 +82,7 @@ namespace OpenSim.Region.Environment | |||
82 | } | 82 | } |
83 | 83 | ||
84 | // If there is no master avatar, return false | 84 | // If there is no master avatar, return false |
85 | if (!m_scene.RegionInfo.MasterAvatarAssignedUUID.Equals(null)) | 85 | if (m_scene.RegionInfo.MasterAvatarAssignedUUID != LLUUID.Zero) |
86 | { | 86 | { |
87 | return m_scene.RegionInfo.MasterAvatarAssignedUUID == user; | 87 | return m_scene.RegionInfo.MasterAvatarAssignedUUID == user; |
88 | } | 88 | } |
@@ -96,7 +96,8 @@ namespace OpenSim.Region.Environment | |||
96 | { | 96 | { |
97 | return true; | 97 | return true; |
98 | } | 98 | } |
99 | if (!user.Equals(null)) | 99 | |
100 | if (user != LLUUID.Zero) | ||
100 | { | 101 | { |
101 | LLUUID[] estatemanagers = m_scene.RegionInfo.EstateSettings.estateManagers; | 102 | LLUUID[] estatemanagers = m_scene.RegionInfo.EstateSettings.estateManagers; |
102 | for (int i = 0; i < estatemanagers.Length; i++) | 103 | for (int i = 0; i < estatemanagers.Length; i++) |
@@ -105,11 +106,6 @@ namespace OpenSim.Region.Environment | |||
105 | return true; | 106 | return true; |
106 | } | 107 | } |
107 | } | 108 | } |
108 | // The below is commented out because logically it happens anyway. It's left in for readability | ||
109 | //else | ||
110 | //{ | ||
111 | //return false; | ||
112 | //} | ||
113 | 109 | ||
114 | return false; | 110 | return false; |
115 | } | 111 | } |