Change a bounds check to handle a packet instead of letting it go.

This change reverts mod to line 1525 of GetDataHandlers.cs in
672d360e9d 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.
This commit is contained in:
Lucas Nicodemus 2017-12-11 00:28:08 -07:00
parent 4b08d61673
commit 6ed110f862

View file

@ -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;
}