summaryrefslogtreecommitdiff
path: root/src/script/api/script_object.cpp
diff options
context:
space:
mode:
authorglx22 <glx22@users.noreply.github.com>2019-09-07 18:37:01 +0200
committerCharles Pigott <charlespigott@googlemail.com>2019-09-07 17:37:01 +0100
commitb3fd7879596defeb6af78acb4ea45e4418821bdf (patch)
tree08b94b2cd65012e573b0e8c94196daab5aab12ba /src/script/api/script_object.cpp
parent381c2a45875206afe43241d0c76619638cf613fc (diff)
downloadopenttd-b3fd7879596defeb6af78acb4ea45e4418821bdf.tar.xz
Fix #7188: check the validity of command callback for scripts (#7701)
Diffstat (limited to 'src/script/api/script_object.cpp')
-rw-r--r--src/script/api/script_object.cpp20
1 files changed, 20 insertions, 0 deletions
diff --git a/src/script/api/script_object.cpp b/src/script/api/script_object.cpp
index 62a71574f..ac06c4c2d 100644
--- a/src/script/api/script_object.cpp
+++ b/src/script/api/script_object.cpp
@@ -83,6 +83,23 @@ ScriptObject::ActiveInstance::~ActiveInstance()
return GetStorage()->mode_instance;
}
+/* static */ void ScriptObject::SetLastCommand(TileIndex tile, uint32 p1, uint32 p2, uint32 cmd)
+{
+ GetStorage()->last_tile = tile;
+ GetStorage()->last_p1 = p1;
+ GetStorage()->last_p2 = p2;
+ GetStorage()->last_cmd = cmd;
+}
+
+/* static */ bool ScriptObject::CheckLastCommand(TileIndex tile, uint32 p1, uint32 p2, uint32 cmd)
+{
+ if (GetStorage()->last_tile != tile) return false;
+ if (GetStorage()->last_p1 != p1) return false;
+ if (GetStorage()->last_p2 != p2) return false;
+ if (GetStorage()->last_cmd != cmd) return false;
+ return true;
+}
+
/* static */ void ScriptObject::SetDoCommandCosts(Money value)
{
GetStorage()->costs = CommandCost(value);
@@ -304,6 +321,9 @@ ScriptObject::ActiveInstance::~ActiveInstance()
/* Only set p2 when the command does not come from the network. */
if (GetCommandFlags(cmd) & CMD_CLIENT_ID && p2 == 0) p2 = UINT32_MAX;
+ /* Store the command for command callback validation. */
+ if (!estimate_only && _networking && !_generating_world) SetLastCommand(tile, p1, p2, cmd);
+
/* Try to perform the command. */
CommandCost res = ::DoCommandPInternal(tile, p1, p2, cmd, (_networking && !_generating_world) ? ScriptObject::GetActiveInstance()->GetDoCommandCallback() : nullptr, text, false, estimate_only);