diff --git a/CHANGELOG.md b/CHANGELOG.md
index 34992433..c423850c 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -11,6 +11,9 @@ This is the rolling changelog for TShock for Terraria. Use past tense when addin
* Do not insert tabs into this file, under any circumstances, ever.
* Do not forget to sign every line you change with your name. (@hakusaro)
* If there is no section called "Upcoming changes" below this line, please add one with `## Upcoming changes` as the first line, and then a bulleted item directly after with the first change.
+## Upcoming changes
+* Added hook `GetDataHandlers.OnReleaseNpc` to handling ReleaseNPC packet and a bouncer to stops unregistered and logged out players on SSC servers from releasing critters NPC. The bouncer has additional filter to stops players who tried to release different critter using crafted packet, e.g. using bunny item to release golden bunny. (@tru321)
+* Added filter in `GetDataHandlers.HandleCatchNpc` that stops unregistered and logged out players on SSC servers to catch critters. (@tru321)
## Upcoming changes
* Fixed rejection check inside of `HandlePaintTile` to account for the Paint Sprayer (or Architect Gizmo Pack) being inside your inventory, rather than on an accessory slot. (@drunderscore)
diff --git a/TShockAPI/Bouncer.cs b/TShockAPI/Bouncer.cs
index d0042b3f..22017c09 100644
--- a/TShockAPI/Bouncer.cs
+++ b/TShockAPI/Bouncer.cs
@@ -104,6 +104,7 @@ namespace TShockAPI
GetDataHandlers.NPCAddBuff += OnNPCAddBuff;
GetDataHandlers.NPCHome += OnUpdateNPCHome;
GetDataHandlers.HealOtherPlayer += OnHealOtherPlayer;
+ GetDataHandlers.ReleaseNPC += OnReleaseNPC;
GetDataHandlers.PlaceObject += OnPlaceObject;
GetDataHandlers.PlaceTileEntity += OnPlaceTileEntity;
GetDataHandlers.PlaceItemFrame += OnPlaceItemFrame;
@@ -1828,6 +1829,52 @@ namespace TShockAPI
return;
}
+ ///
+ /// A bouncer for checking NPC released by player
+ ///
+ /// The object that triggered the event.
+ /// The packet arguments that the event has.
+ internal void OnReleaseNPC(object sender, GetDataHandlers.ReleaseNpcEventArgs args)
+ {
+ int x = args.X;
+ int y = args.Y;
+ short type = args.Type;
+ byte style = args.Style;
+
+ // if npc released outside allowed tile
+ if (x >= Main.maxTilesX * 16 - 16 || x < 0 || y >= Main.maxTilesY * 16 - 16 || y < 0)
+ {
+ TShock.Log.ConsoleDebug("Bouncer / OnReleaseNPC rejected out of bounds from {0}", args.Player.Name);
+ args.Handled = true;
+ return;
+ }
+
+ // if player disabled
+ if (args.Player.IsBeingDisabled())
+ {
+ TShock.Log.ConsoleDebug("Bouncer / OnReleaseNPC rejected npc release from {0}", args.Player.Name);
+ args.Handled = true;
+ return;
+ }
+
+ // if released npc not from its item (from crafted packet)
+ // e.g. using bunny item to release golden bunny
+ if (args.Player.TPlayer.lastVisualizedSelectedItem.makeNPC != type && args.Player.TPlayer.lastVisualizedSelectedItem.placeStyle != style)
+ {
+ TShock.Log.ConsoleDebug("Bouncer / OnReleaseNPC released different critter from {0}", args.Player.Name);
+ args.Player.Kick("Released critter was not from its item.", true);
+ args.Handled = true;
+ return;
+ }
+
+ if (args.Player.IsBouncerThrottled())
+ {
+ TShock.Log.ConsoleDebug("Bouncer / OnReleaseNPC rejected throttle from {0}", args.Player.Name);
+ args.Handled = true;
+ return;
+ }
+ }
+
/// Bouncer's PlaceObject hook reverts malicious tile placement.
/// The object that triggered the event.
/// The packet arguments that the event has.
diff --git a/TShockAPI/GetDataHandlers.cs b/TShockAPI/GetDataHandlers.cs
index 89eb94c5..5555a883 100644
--- a/TShockAPI/GetDataHandlers.cs
+++ b/TShockAPI/GetDataHandlers.cs
@@ -135,6 +135,7 @@ namespace TShockAPI
{ PacketTypes.Teleport, HandleTeleport },
{ PacketTypes.PlayerHealOther, HandleHealOther },
{ PacketTypes.CatchNPC, HandleCatchNpc },
+ { PacketTypes.ReleaseNPC, HandleReleaseNpc },
{ PacketTypes.TeleportationPotion, HandleTeleportationPotion },
{ PacketTypes.CompleteAnglerQuest, HandleCompleteAnglerQuest },
{ PacketTypes.NumberOfAnglerQuestsCompleted, HandleNumberOfAnglerQuestsCompleted },
@@ -1675,6 +1676,56 @@ namespace TShockAPI
return args.Handled;
}
+ ///
+ /// The ReleaseNPC event arguments
+ ///
+ public class ReleaseNpcEventArgs : GetDataHandledEventArgs
+ {
+ ///
+ /// The X value of where NPC released
+ ///
+ public int X { get; set; }
+
+ ///
+ /// The Y value of where NPC released
+ ///
+ public int Y { get; set; }
+
+ ///
+ /// The NPC Type that player release
+ ///
+ public short Type { get; set; }
+
+ ///
+ /// The NPC release style
+ ///
+ public byte Style { get; set; }
+ }
+
+ ///
+ /// Called when player release a NPC, for checking critter released from item.
+ ///
+ public static HandlerList ReleaseNPC = new HandlerList();
+ private static bool OnReleaseNpc(TSPlayer player, MemoryStream data, int _x, int _y, short _type, byte _style)
+ {
+ if (ReleaseNPC == null)
+ {
+ return false;
+ }
+
+ var args = new ReleaseNpcEventArgs
+ {
+ Player = player,
+ Data = data,
+ X = _x,
+ Y = _y,
+ Type = _type,
+ Style = _style
+ };
+ ReleaseNPC.Invoke(null, args);
+ return args.Handled;
+ }
+
/// The arguments to the PlaceObject hook.
public class PlaceObjectEventArgs : GetDataHandledEventArgs
{
@@ -3724,10 +3775,31 @@ namespace TShockAPI
NetMessage.SendData((int)PacketTypes.NpcUpdate, -1, -1, NetworkText.Empty, npcID);
return true;
}
+
+ if(args.Player.IsBeingDisabled())
+ {
+ TShock.Log.ConsoleDebug("GetDataHandlers / HandleCatchNpc rejected catch npc {0}", args.Player.Name);
+ return true;
+ }
return false;
}
+ private static bool HandleReleaseNpc(GetDataHandlerArgs args)
+ {
+ var x = args.Data.ReadInt32();
+ var y = args.Data.ReadInt32();
+ var type = args.Data.ReadInt16();
+ var style = args.Data.ReadInt8();
+
+ if (OnReleaseNpc(args.Player, args.Data, x, y, type, style))
+ {
+ return true;
+ }
+
+ return false;
+ }
+
private static bool HandleTeleportationPotion(GetDataHandlerArgs args)
{
var type = args.Data.ReadByte();