From 6ed110f862cfc600ceb6ad874579b8d08ae95ef5 Mon Sep 17 00:00:00 2001
From: Lucas Nicodemus <shank@shanked.me>
Date: Mon, 11 Dec 2017 00:28:08 -0700
Subject: [PATCH] Change a bounds check to handle a packet instead of letting
 it go.

This change reverts mod to line 1525 of GetDataHandlers.cs in
672d360e9dbe6c93997f4d17ac0f5e2c987db6ea by Deathamx.

We couldn't think of a reason why a bounds check would ever return
false, thus not handling the packet and letting the server accept
it. Therefore, it's now true.

In the future, we need to document handling better.
---
 TShockAPI/Bouncer.cs | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/TShockAPI/Bouncer.cs b/TShockAPI/Bouncer.cs
index dd1257db..5b802cef 100644
--- a/TShockAPI/Bouncer.cs
+++ b/TShockAPI/Bouncer.cs
@@ -59,8 +59,7 @@ namespace TShockAPI
 		{
 			if (args.ProjectileIndex > Main.maxProjectiles || args.ProjectileIndex < 0)
 			{
-				// TODO: Should this be /true/ to stop the server from processing it?
-				args.Handled = false;
+				args.Handled = true;
 				return;
 			}