From f2260d56a49e252698c77a76ee6a00900be9ce3e Mon Sep 17 00:00:00 2001 From: JeromSar Date: Fri, 26 Aug 2016 12:39:56 +0200 Subject: [PATCH] Fix potential IOOBE in /stfu. Fixes #1750 --- .../totalfreedommod/command/Command_stfu.java | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/src/main/java/me/totalfreedom/totalfreedommod/command/Command_stfu.java b/src/main/java/me/totalfreedom/totalfreedommod/command/Command_stfu.java index 353246b..13e8bfb 100644 --- a/src/main/java/me/totalfreedom/totalfreedommod/command/Command_stfu.java +++ b/src/main/java/me/totalfreedom/totalfreedommod/command/Command_stfu.java @@ -23,7 +23,7 @@ public class Command_stfu extends FreedomCommand return false; } - if (args[0].equalsIgnoreCase("list")) + if (args[0].equals("list")) { msg("Muted players:"); FPlayer info; @@ -45,7 +45,7 @@ public class Command_stfu extends FreedomCommand return true; } - if (args[0].equalsIgnoreCase("purge")) + if (args[0].equals("purge")) { FUtil.adminAction(sender.getName(), "Unmuting all players.", true); FPlayer info; @@ -63,7 +63,7 @@ public class Command_stfu extends FreedomCommand return true; } - if (args[0].equalsIgnoreCase("all")) + if (args[0].equals("all")) { FUtil.adminAction(sender.getName(), "Muting all non-Superadmins", true); @@ -83,10 +83,16 @@ public class Command_stfu extends FreedomCommand return true; } + // -s option (smite) boolean smite = args[0].equals("-s"); if (smite) { args = ArrayUtils.subarray(args, 1, args.length); + + if (args.length < 1) + { + return false; + } } final Player player = getPlayer(args[0]);