diff --git a/CHANGELOG.md b/CHANGELOG.md index db7b154b..e9883b79 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -2,15 +2,19 @@ This is the rolling changelog for TShock for Terraria. Use past tense when adding new entries; sign your name off when you add or change something. This should primarily be things like user changes, not necessarily codebase changes unless it's really relevant or large. -## Upcoming Changes +## Upcoming changes +* Installed new sprinklers! + +## TShock 4.4.0 (Pre-release 11) * New permission `tshock.tp.pylon` to enable teleporting via Teleportation Pylons (@QuiCM) * New permission `tshock.journey.research` to enable sharing research via item sacrifice (@QuiCM) -* Add Emoji event to GetDataHandler. This packet is received when a player tries to display an emote. - * Adding EmojiHandler to handle an exploit. Adding `tshock.sendemoji` permission and checks. Added this permission to guest group by default. -* Handling SyncCavernMonsterType packet to prevent an exploit where players could modify the server's cavern monster types and make the server spawn any NPCs - including bosses - onto other players. +* Add Emoji event to GetDataHandler. This packet is received when a player tries to display an emote. (who?) + * Adding EmojiHandler to handle an exploit. Adding `tshock.sendemoji` permission and checks. Added this permission to guest group by default. (who?) +* Handled SyncCavernMonsterType packet to prevent an exploit where players could modify the server's cavern monster types and make the server spawn any NPCs - including bosses - onto other players. (who?) * Added LandGolfBallInCup event which is accessible for developers to work with, as well as LandGolfBallInCup handler to handle exploits where players could send direct packets to trigger and imitate golf ball cup landing anywhere in the game world. Added two public lists in Handlers.LandGolfBallInCupHandler: GolfBallProjectileIDs and GolfClubItemIDs. (@Patrikkk) -* Add SyncTilePicking event. This is called when a player damages a tile. Implementing SyncTilePickingHandler and patching tile damaging related exploits. (Preventing player sending invalid world position data which disconnects other players.) -* Fix the issue where mobs could not be fished out during bloodmoon because of Bouncer checks. (@Patrikkk) +* Added SyncTilePicking event. This is called when a player damages a tile. Implementing SyncTilePickingHandler and patching tile damaging related exploits. (Preventing player sending invalid world position data which disconnects other players.) +* Fixed the issue where mobs could not be fished out during bloodmoon because of Bouncer checks. (@Patrikkk) + *Fixed the issue where certain fishing rods could not fish out NPCs due to a Bouncer check. (@Patrikkk) * Update for OTAPI 2.0.0.37 and Terraria 1.4.0.5. (@hakusaro, @Patrikkk) * Add additional config options for automatically kicking clients from the server upon breaking anti-cheat thresholds. (@moisterrific) * Add pylon teleportation permission to default group, added `/spawn` permission to admin group, added the new journey mode research permission to trustedadmin, and moved all previous journey mode permissions from owner to trustedadmin. (@moisterrific) diff --git a/TShockAPI/Bouncer.cs b/TShockAPI/Bouncer.cs index f21b60eb..117c8aac 100644 --- a/TShockAPI/Bouncer.cs +++ b/TShockAPI/Bouncer.cs @@ -494,20 +494,36 @@ namespace TShockAPI if (args.Player.TileKillThreshold >= TShock.Config.TileKillThreshold) { + if (TShock.Config.KickOnTileKillThresholdBroken) + { + args.Player.Kick(string.Format("Tile kill threshold exceeded {0}.", TShock.Config.TileKillThreshold)); + } + else + { + args.Player.Disable("Reached TileKill threshold.", DisableFlags.WriteToLogAndConsole); + args.Player.SendTileSquare(tileX, tileY, 4); + } + TShock.Log.ConsoleDebug("Bouncer / OnTileEdit rejected from tile kill threshold from {0}, (value: {1})", args.Player.Name, args.Player.TileKillThreshold); - TShock.Log.ConsoleDebug("If this player wasn't hacking, please report the damage value they were disabled for to TShock so we can improve this!"); - args.Player.Disable("Reached TileKill threshold.", DisableFlags.WriteToLogAndConsole); - args.Player.SendTileSquare(tileX, tileY, 4); + TShock.Log.ConsoleDebug("If this player wasn't hacking, please report the tile kill threshold they were disabled for to TShock so we can improve this!"); args.Handled = true; return; } if (args.Player.TilePlaceThreshold >= TShock.Config.TilePlaceThreshold) { + if (TShock.Config.KickOnTilePlaceThresholdBroken) + { + args.Player.Kick(string.Format("Tile place threshold exceeded {0}.", TShock.Config.TilePlaceThreshold)); + } + else + { + args.Player.Disable("Reached TilePlace threshold.", DisableFlags.WriteToLogAndConsole); + args.Player.SendTileSquare(tileX, tileY, 4); + } + TShock.Log.ConsoleDebug("Bouncer / OnTileEdit rejected from tile place threshold from {0}, (value: {1})", args.Player.Name, args.Player.TilePlaceThreshold); - TShock.Log.ConsoleDebug("If this player wasn't hacking, please report the damage value they were disabled for to TShock so we can improve this!"); - args.Player.Disable("Reached TilePlace threshold.", DisableFlags.WriteToLogAndConsole); - args.Player.SendTileSquare(tileX, tileY, 4); + TShock.Log.ConsoleDebug("If this player wasn't hacking, please report the tile place threshold they were disabled for to TShock so we can improve this!"); args.Handled = true; return; } @@ -761,9 +777,18 @@ namespace TShockAPI if (args.Player.ProjectileThreshold >= TShock.Config.ProjectileThreshold) { - args.Player.Disable("Reached projectile update threshold.", DisableFlags.WriteToLogAndConsole); - TShock.Log.ConsoleDebug("Bouncer / OnNewProjectile rejected from proj update threshold from {0} {1}/{2}", args.Player.Name, args.Player.ProjectileThreshold, TShock.Config.ProjectileThreshold); - args.Player.RemoveProjectile(ident, owner); + if (TShock.Config.KickOnProjectileThresholdBroken) + { + args.Player.Kick(string.Format("Projectile update threshold exceeded {0}.", TShock.Config.ProjectileThreshold)); + } + else + { + args.Player.Disable("Reached projectile update threshold.", DisableFlags.WriteToLogAndConsole); + args.Player.RemoveProjectile(ident, owner); + } + + TShock.Log.ConsoleDebug("Bouncer / OnNewProjectile rejected from projectile update threshold from {0} {1}/{2}", args.Player.Name, args.Player.ProjectileThreshold, TShock.Config.ProjectileThreshold); + TShock.Log.ConsoleDebug("If this player wasn't hacking, please report the projectile update threshold they were disabled for to TShock so we can improve this!"); args.Handled = true; return; } @@ -817,21 +842,20 @@ namespace TShockAPI return; } - if (damage > TShock.Config.MaxDamage && !args.Player.HasPermission(Permissions.ignoredamagecap)) + if (damage >= TShock.Config.MaxDamage && !args.Player.HasPermission(Permissions.ignoredamagecap)) { if (TShock.Config.KickOnDamageThresholdBroken) { args.Player.Kick(string.Format("NPC damage exceeded {0}.", TShock.Config.MaxDamage)); - TShock.Log.ConsoleDebug("Bouncer / OnNPCStrike rejected from kodtb from {0} {1}/{2}", args.Player.Name, damage, TShock.Config.MaxDamage); - args.Handled = true; - return; } else { - TShock.Log.ConsoleDebug("Bouncer / OnNPCStrike rejected from dtb from {0} {1}/{2}", args.Player.Name, damage, TShock.Config.MaxDamage); args.Player.Disable(String.Format("NPC damage exceeded {0}.", TShock.Config.MaxDamage), DisableFlags.WriteToLogAndConsole); + args.Player.SendData(PacketTypes.NpcUpdate, "", id); } - args.Player.SendData(PacketTypes.NpcUpdate, "", id); + + TShock.Log.ConsoleDebug("Bouncer / OnNPCStrike rejected from damage threshold from {0} {1}/{2}", args.Player.Name, damage, TShock.Config.MaxDamage); + TShock.Log.ConsoleDebug("If this player wasn't hacking, please report the damage threshold they were disabled for to TShock so we can improve this!"); args.Handled = true; return; } @@ -1114,9 +1138,18 @@ namespace TShockAPI if (args.Player.TileLiquidThreshold >= TShock.Config.TileLiquidThreshold) { + if (TShock.Config.KickOnTileLiquidThresholdBroken) + { + args.Player.Kick(string.Format("Reached TileLiquid threshold {0}.", TShock.Config.TileLiquidThreshold)); + } + else + { + args.Player.Disable("Reached TileLiquid threshold.", DisableFlags.WriteToLogAndConsole); + args.Player.SendTileSquare(tileX, tileY, 1); + } + TShock.Log.ConsoleDebug("Bouncer / OnLiquidSet rejected from liquid threshold from {0} {1}/{2}", args.Player.Name, args.Player.TileLiquidThreshold, TShock.Config.TileLiquidThreshold); - args.Player.Disable("Reached TileLiquid threshold.", DisableFlags.WriteToLogAndConsole); - args.Player.SendTileSquare(tileX, tileY, 1); + TShock.Log.ConsoleDebug("If this player wasn't hacking, please report the tile liquid threshold they were disabled for to TShock so we can improve this!"); args.Handled = true; return; } @@ -1440,25 +1473,33 @@ namespace TShockAPI // Why 0.2? // @bartico6: Because heal other player only happens when you are using the spectre armor with the hood, // and the healing you can do with that is 20% of your damage. - if (amount > TShock.Config.MaxDamage * 0.2) + if (amount >= TShock.Config.MaxDamage * 0.2) { - TShock.Log.ConsoleDebug("Bouncer / OnUpdateNPCHome 0.2 check from {0}", args.Player.Name); + TShock.Log.ConsoleDebug("Bouncer / OnHealOtherPlayer 0.2 check from {0}", args.Player.Name); args.Player.Disable("HealOtherPlayer cheat attempt!", DisableFlags.WriteToLogAndConsole); args.Handled = true; return; } - if (args.Player.HealOtherThreshold > TShock.Config.HealOtherThreshold) + if (args.Player.HealOtherThreshold >= TShock.Config.HealOtherThreshold) { - TShock.Log.ConsoleDebug("Bouncer / OnUpdateNPCHome rejected heal other threshold from {0} {1}/{2}", args.Player.Name, args.Player.HealOtherThreshold, TShock.Config.HealOtherThreshold); - args.Player.Disable("Reached HealOtherPlayer threshold.", DisableFlags.WriteToLogAndConsole); + if (TShock.Config.KickOnHealOtherThresholdBroken) + { + args.Player.Kick(string.Format("HealOtherPlayer threshold exceeded {0}.", TShock.Config.HealOtherThreshold)); + } + else + { + args.Player.Disable("Reached HealOtherPlayer threshold.", DisableFlags.WriteToLogAndConsole); + } + TShock.Log.ConsoleDebug("Bouncer / OnHealOtherPlayer rejected heal other threshold from {0} {1}/{2}", args.Player.Name, args.Player.HealOtherThreshold, TShock.Config.HealOtherThreshold); + TShock.Log.ConsoleDebug("If this player wasn't hacking, please report the HealOtherPlayer threshold they were disabled for to TShock so we can improve this!"); args.Handled = true; return; } if (args.Player.IsBeingDisabled() || args.Player.IsBouncerThrottled()) { - TShock.Log.ConsoleDebug("Bouncer / OnUpdateNPCHome rejected disabled/throttled from {0}", args.Player.Name); + TShock.Log.ConsoleDebug("Bouncer / OnHealOtherPlayer rejected disabled/throttled from {0}", args.Player.Name); args.Handled = true; return; } @@ -1906,8 +1947,8 @@ namespace TShockAPI /// internal void OnFishOutNPC(object sender, GetDataHandlers.FishOutNPCEventArgs args) { - /// Getting recent projectiles of the player which are named Bobber. All bobbers have the same Name. - var projectile = args.Player.RecentlyCreatedProjectiles.FirstOrDefault(p => Main.projectile[p.Index].Name == "Bobber"); + /// Getting recent projectiles of the player and selecting the first that is a bobber. + var projectile = args.Player.RecentlyCreatedProjectiles.FirstOrDefault(p => Main.projectile[p.Index].bobber); if (!FishingRodItemIDs.Contains(args.Player.SelectedItem.type)) { diff --git a/TShockAPI/Commands.cs b/TShockAPI/Commands.cs index 4fc949b7..4e8e773d 100644 --- a/TShockAPI/Commands.cs +++ b/TShockAPI/Commands.cs @@ -1970,7 +1970,8 @@ namespace TShockAPI "snowmen", "pirates", "pumpkinmoon", - "frostmoon" + "frostmoon", + "martians" }; private static void ManageWorldEvent(CommandArgs args) diff --git a/TShockAPI/ConfigFile.cs b/TShockAPI/ConfigFile.cs index 89b3c702..996823b1 100644 --- a/TShockAPI/ConfigFile.cs +++ b/TShockAPI/ConfigFile.cs @@ -272,22 +272,42 @@ namespace TShockAPI /// Disables a player and reverts their actions if this number of tile kills is exceeded within 1 second. [Description("Disables a player and reverts their actions if this number of tile kills is exceeded within 1 second.")] public int TileKillThreshold = 60; + + /// Whether or not to kick users when they surpass the TileKill threshold. + [Description("Whether or not to kick users when they surpass the TileKill threshold.")] + public bool KickOnTileKillThresholdBroken = false; /// Disables a player and reverts their actions if this number of tile places is exceeded within 1 second. [Description("Disables a player and reverts their actions if this number of tile places is exceeded within 1 second.")] public int TilePlaceThreshold = 32; + + /// Whether or not to kick users when they surpass the TilePlace threshold. + [Description("Whether or not to kick users when they surpass the TilePlace threshold.")] + public bool KickOnTilePlaceThresholdBroken = false; /// Disables a player if this number of liquid sets is exceeded within 1 second. [Description("Disables a player if this number of liquid sets is exceeded within 1 second.")] public int TileLiquidThreshold = 50; + + /// Whether or not to kick users when they surpass the TileLiquid threshold. + [Description("Whether or not to kick users when they surpass the TileLiquid threshold.")] + public bool KickOnTileLiquidThresholdBroken = false; /// Disable a player if this number of projectiles is created within 1 second. [Description("Disable a player if this number of projectiles is created within 1 second.")] public int ProjectileThreshold = 50; + + /// Whether or not to kick users when they surpass the Projectile threshold. + [Description("Whether or not to kick users when they surpass the Projectile threshold.")] + public bool KickOnProjectileThresholdBroken = false; /// Disables a player if this number of HealOtherPlayer packets is sent within 1 second. [Description("Disables a player if this number of HealOtherPlayer packets is sent within 1 second.")] public int HealOtherThreshold = 50; + + /// Whether or not to kick users when they surpass the HealOther threshold. + [Description("Whether or not to kick users when they surpass the HealOther threshold.")] + public bool KickOnHealOtherThresholdBroken = false; /// Whether or not to ignore shrapnel from crystal bullets for the projectile threshold count. [Description("Whether or not to ignore shrapnel from crystal bullets for the projectile threshold count.")] @@ -452,6 +472,10 @@ namespace TShockAPI /// Disables a player if this number of tiles is painted within 1 second. [Description("Disables a player if this number of tiles is painted within 1 second.")] public int TilePaintThreshold = 15; + + /// Whether or not to kick users when they surpass the TilePaint threshold. + [Description("Whether or not to kick users when they surpass the TilePaint threshold.")] + public bool KickOnTilePaintThresholdBroken = false; /// Forces Halloween-only events to occur all year. [Description("Forces Halloween-only events to occur all year.")] diff --git a/TShockAPI/DB/GroupManager.cs b/TShockAPI/DB/GroupManager.cs index 3cff37a1..1462411d 100644 --- a/TShockAPI/DB/GroupManager.cs +++ b/TShockAPI/DB/GroupManager.cs @@ -76,7 +76,8 @@ namespace TShockAPI.DB Permissions.summonboss, Permissions.whisper, Permissions.wormhole, - Permissions.canpaint)); + Permissions.canpaint, + Permissions.pylon)); AddDefaultGroup("vip", "default", string.Join(",", @@ -123,7 +124,8 @@ namespace TShockAPI.DB Permissions.tpnpc, Permissions.tppos, Permissions.tpsilent, - Permissions.userinfo)); + Permissions.userinfo, + Permissions.spawn)); AddDefaultGroup("trustedadmin", "admin", string.Join(",", @@ -155,7 +157,20 @@ namespace TShockAPI.DB Permissions.invade, Permissions.startdd2, Permissions.uploaddata, - Permissions.uploadothersdata)); + Permissions.uploadothersdata, + Permissions.journey_timefreeze, + Permissions.journey_timeset, + Permissions.journey_timespeed, + Permissions.journey_godmode, + Permissions.journey_windstrength, + Permissions.journey_windfreeze, + Permissions.journey_rainstrength, + Permissions.journey_rainfreeze, + Permissions.journey_placementrange, + Permissions.journey_setdifficulty, + Permissions.journey_biomespreadfreeze, + Permissions.journey_setspawnrate, + Permissions.journey_contributeresearch)); AddDefaultGroup("owner", "trustedadmin", string.Join(",", @@ -174,19 +189,7 @@ namespace TShockAPI.DB Permissions.settempgroup, Permissions.spawnrate, Permissions.tpoverride, - Permissions.createdumps, - Permissions.journey_timefreeze, - Permissions.journey_timeset, - Permissions.journey_timespeed, - Permissions.journey_godmode, - Permissions.journey_windstrength, - Permissions.journey_windfreeze, - Permissions.journey_rainstrength, - Permissions.journey_rainfreeze, - Permissions.journey_placementrange, - Permissions.journey_setdifficulty, - Permissions.journey_biomespreadfreeze, - Permissions.journey_setspawnrate)); + Permissions.createdumps)); } // Load Permissions from the DB diff --git a/TShockAPI/Handlers/SendTileSquareHandler.cs b/TShockAPI/Handlers/SendTileSquareHandler.cs index 39a7e751..2bf0c729 100644 --- a/TShockAPI/Handlers/SendTileSquareHandler.cs +++ b/TShockAPI/Handlers/SendTileSquareHandler.cs @@ -72,7 +72,7 @@ namespace TShockAPI.Handlers Debug.VisualiseTileSetDiff(args.TileX, args.TileY, args.Size, args.Size, tiles); - IterateTileSquare(tiles, processed, args); + IterateTileSquare(tiles, processed, args); // Uncommenting this function will send the same tile square 10 blocks above you for visualisation. This will modify your world and overwrite existing blocks. // Use in test worlds only. @@ -139,7 +139,7 @@ namespace TShockAPI.Handlers NetTile[,] newTiles; int width = data.Width; int height = data.Height; - int offset = 0; + int offsetY = 0; if (newTile.Type == TileID.TrapdoorClosed) { @@ -148,7 +148,13 @@ namespace TShockAPI.Handlers // So we capture all 6 possible tiles and offset ourselves 1 tile above the closed trapdoor to capture the entire 2x3 area width = 2; height = 3; - offset = -1; + offsetY = -1; + } + + // Ensure the tile object fits inside the square before processing it + if (!DoesTileObjectFitInTileSquare(x, y, width, height, size, offsetY, processed)) + { + continue; } newTiles = new NetTile[width, height]; @@ -157,11 +163,11 @@ namespace TShockAPI.Handlers { for (int j = 0; j < height; j++) { - newTiles[i, j] = tiles[x + i, y + j + offset]; - processed[x + i, y + j] = true; + newTiles[i, j] = tiles[x + i, y + j + offsetY]; + processed[x + i, y + j + offsetY] = true; } } - ProcessTileObject(newTile.Type, realX, realY + offset, width, height, newTiles, args); + ProcessTileObject(newTile.Type, realX, realY + offsetY, width, height, newTiles, args); continue; } @@ -392,7 +398,7 @@ namespace TShockAPI.Handlers /// /// /// - static NetTile[,] ReadNetTilesFromStream(System.IO.MemoryStream stream, int size) + static NetTile[,] ReadNetTilesFromStream(System.IO.MemoryStream stream, short size) { NetTile[,] tiles = new NetTile[size, size]; for (int x = 0; x < size; x++) @@ -420,8 +426,8 @@ namespace TShockAPI.Handlers return true; } - // 5x5 is the largest vanilla-sized tile square. Anything larger than this should not be seen in the vanilla game and should be rejected - if (args.Size > 5) + // 7x7 is the largest vanilla-sized tile square (used for lamp posts). Anything larger than this should not be sent by the vanilla game and should be rejected + if (args.Size > 7) { TShock.Log.ConsoleDebug("Bouncer / SendTileSquare rejected from non-vanilla tilemod from {0}", args.Player.Name); return true; @@ -444,6 +450,44 @@ namespace TShockAPI.Handlers return false; } + /// + /// Checks if a tile object fits inside the dimensions of a tile square + /// + /// + /// + /// + /// + /// + /// + /// + /// + static bool DoesTileObjectFitInTileSquare(int x, int y, int width, int height, int size, int offsetY, bool[,] processed) + { + // If the starting y position of this tile object is at (x, 0) and the y offset is negative, we'll be accessing tiles outside the square + if (y + offsetY < 0) + { + TShock.Log.ConsoleDebug("Bouncer / SendTileSquareHandler - rejected tile object because object dimensions fall outside the tile square (negative y value)"); + return false; + } + + if (x + width >= size || y + height + offsetY >= size) + { + // This is ugly, but we want to mark all these tiles as processed so that we're not hitting this check multiple times for one dodgy tile object + for (int i = x; i < size; i++) + { + for (int j = Math.Max(0, y + offsetY); j < size; j++) // This is also ugly. Using Math.Max to make sure y + offsetY >= 0 + { + processed[i, j] = true; + } + } + + TShock.Log.ConsoleDebug("Bouncer / SendTileSquareHandler - rejected tile object because object dimensions fall outside the tile square (excessive size)"); + return false; + } + + return true; + } + class Debug { ///