Fixed corrupt packets caused by invalid tile IDs

CHANGELOG.md

@@ -19,6 +19,7 @@ This is the rolling changelog for TShock for Terraria. Use past tense when addin
 * Players can no longer quick stack items into region protected chests (@ProfessorXZ)
 * Rope placement is no longer blocked by range checks (@ProfessorXZ)
 * The Drill Containment Unit breaks blocks properly now (@ProfessorXZ)
+* Fixed item duplications caused by range checks and invalid netIDs (@ProfessorXZ)
 * Fixed Expert mode coin duplication (@ProfessorXZ)
 * Players are no longer able to place liquids using LoadNetModule packet (@ProfessorXZ)
 * Explosives are no longer blocked by range checks (@ProfessorXZ)

TShockAPI/GetDataHandlers.cs

@@ -1907,6 +1907,12 @@ namespace TShockAPI
 
 				var style = args.Data.ReadInt8();
 
+				if (editData < 0)
+				{
+					args.Player.SendTileSquare(tileX, tileY, 4);
+					return true;
+				}
+
 				if (OnTileEdit(args.Player, tileX, tileY, action, type, editData, style))
 					return true;
 				if (!TShock.Utils.TilePlacementValid(tileX, tileY))