diff options
| author | Aikar <aikar@aikar.co> | 2013-12-02 20:09:22 -0500 |
|---|---|---|
| committer | Wesley Wolfe <weswolf@aol.com> | 2013-12-02 20:34:01 -0600 |
| commit | 9969a86e839560f6fe4cfbfefa1c5d4f47df0a54 (patch) | |
| tree | 3028a9459f0ceec443da134074dafb04a6dc9280 | |
| parent | c5d8b4393a237c33dd9a75cbe6fe48bd1d00d04c (diff) | |
| download | craftbukkit-9969a86e839560f6fe4cfbfefa1c5d4f47df0a54.tar craftbukkit-9969a86e839560f6fe4cfbfefa1c5d4f47df0a54.tar.gz craftbukkit-9969a86e839560f6fe4cfbfefa1c5d4f47df0a54.tar.lz craftbukkit-9969a86e839560f6fe4cfbfefa1c5d4f47df0a54.tar.xz craftbukkit-9969a86e839560f6fe4cfbfefa1c5d4f47df0a54.zip | |
Use strict lookup for op permission refresh. Fixes BUKKIT-4698
Adding or removing operators was mistakenly using a loose player lookup
method, which would cause a permission refreshes on an online player whos
name starts with the name of the (offline) opped player.
Add/Remove op operations are exact name match only and the permission
refresh will behave the same way.
| -rw-r--r-- | src/main/java/net/minecraft/server/PlayerList.java | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/main/java/net/minecraft/server/PlayerList.java b/src/main/java/net/minecraft/server/PlayerList.java index 92e5c893..534073b3 100644 --- a/src/main/java/net/minecraft/server/PlayerList.java +++ b/src/main/java/net/minecraft/server/PlayerList.java @@ -819,7 +819,7 @@ public abstract class PlayerList { this.operators.add(s.toLowerCase()); // CraftBukkit start - Player player = server.server.getPlayer(s); + Player player = server.server.getPlayerExact(s); if (player != null) { player.recalculatePermissions(); } @@ -830,7 +830,7 @@ public abstract class PlayerList { this.operators.remove(s.toLowerCase()); // CraftBukkit start - Player player = server.server.getPlayer(s); + Player player = server.server.getPlayerExact(s); if (player != null) { player.recalculatePermissions(); } |