themaster1970sf/TShock
1
0
Fork 0
Code Issues Pull requests Projects Releases 1 Packages Wiki Activity Actions

SyncTilePickingHandler - Remove redundant check. Properly document method.

Browse source
This commit is contained in:
Patrikkk 2020-06-02 19:26:25 +02:00
parent e738d8e794
commit 7487afafb6
1 changed files with 1 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

9
TShockAPI/Handlers/SyncTilePickingHandler.cs
View file

@ -11,19 +11,12 @@ namespace TShockAPI.Handlers
class SyncTilePickingHandler : IPacketHandler<SyncTilePickingEventArgs> class SyncTilePickingHandler : IPacketHandler<SyncTilePickingEventArgs>
{ {
/// <summary> /// <summary>
/// Invoked when player damages a tile. /// Invoked when player damages a tile. Rejects the packet if its out of world bounds.
/// </summary> /// </summary>
/// <param name="sender"></param> /// <param name="sender"></param>
/// <param name="args"></param> /// <param name="args"></param>
public void OnReceive(object sender, SyncTilePickingEventArgs args) public void OnReceive(object sender, SyncTilePickingEventArgs args)
{ {
if (args.PlayerIndex != args.Player.Index)
{
TShock.Log.ConsoleDebug($"SyncTilePickingHandler: SyncTilePicking packet rejected for ID spoofing. Expected {args.Player.Index}, received {args.PlayerIndex} from {args.Player.Name}.");
args.Handled = true;
return;
}
if (args.TileX > Main.maxTilesX || args.TileX < 0 if (args.TileX > Main.maxTilesX || args.TileX < 0
|| args.TileY > Main.maxTilesY || args.TileY < 0) || args.TileY > Main.maxTilesY || args.TileY < 0)
{ {