From 2f0e849c8443b34f6f421f0afc8948b0864ae99d Mon Sep 17 00:00:00 2001
From: ErrorNoInternet <errornointernet@envs.net>
Date: Wed, 1 Jan 2025 15:11:02 -0500
Subject: [PATCH] fix(commands/voice/queue): don't allow negative indices

---
 commands/voice.py | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/commands/voice.py b/commands/voice.py
index edfd5f6..228d2a2 100644
--- a/commands/voice.py
+++ b/commands/voice.py
@@ -1,11 +1,12 @@
 import math
 
 import arguments
-import commands
-import utils
 import youtubedl
 from state import client, players
 
+import commands
+import utils
+
 
 async def queue_or_play(message):
     await ensure_joined(message)
@@ -97,12 +98,13 @@ async def queue_or_play(message):
         await utils.add_check_reaction(message)
         return
     elif i := args.remove_index:
-        try:
-            queued = players[message.guild.id].queue[i - 1]
-            del players[message.guild.id].queue[i - 1]
-            await utils.reply(message, f"**x** {queued.format()}")
-        except:
+        if i <= 0 or i > len(players[message.guild.id].queue):
             await utils.reply(message, "invalid index!")
+            return
+
+        queued = players[message.guild.id].queue[i - 1]
+        del players[message.guild.id].queue[i - 1]
+        await utils.reply(message, f"**x** {queued.format()}")
     elif args.remove_title or args.remove_queuer:
         targets = []
         for queued in players[message.guild.id].queue: