author | tron |
Fri, 30 Dec 2005 14:28:32 +0000 | |
changeset 2809 | 5bec9d1a7d6b |
parent 2772 | 96e47991fd1d |
child 2819 | 710436dd4288 |
permissions | -rw-r--r-- |
2186 | 1 |
/* $Id$ */ |
2 |
||
0 | 3 |
#include "stdafx.h" |
1891
92a3b0aa0946
(svn r2397) - CodeChange: rename all "ttd" files to "openttd" files.
Darkvater
parents:
1877
diff
changeset
|
4 |
#include "openttd.h" |
507
8aa8100b0b22
(svn r815) Include strings.h only in the files which need it.
tron
parents:
467
diff
changeset
|
5 |
#include "table/strings.h" |
2163
637ec3c361f5
(svn r2673) Include functions.h directly, not globally via openttd.h
tron
parents:
2159
diff
changeset
|
6 |
#include "functions.h" |
679
e959706a3e4d
(svn r1117) Move map arrays and some related macros into their own files map.c and map.h
tron
parents:
652
diff
changeset
|
7 |
#include "map.h" |
0 | 8 |
#include "gui.h" |
9 |
#include "command.h" |
|
10 |
#include "player.h" |
|
543
e3b43338096b
(svn r942) -Merged branch/network back into the trunk
truelight
parents:
534
diff
changeset
|
11 |
#include "network.h" |
2159
3b634157c3b2
(svn r2669) Shuffle some more stuff around to reduce dependencies
tron
parents:
1980
diff
changeset
|
12 |
#include "variables.h" |
0 | 13 |
|
1820
9b6458526480
(svn r2324) Introduce _cmd_text for passing strings with a command instead of abusing _decode_parameters as text buffer. This should prevent several possible buffer overruns and is a bit cleaner to use. As bonus it reduces the size of most command packets by 79 bytes.
tron
parents:
1804
diff
changeset
|
14 |
const char* _cmd_text = NULL; |
9b6458526480
(svn r2324) Introduce _cmd_text for passing strings with a command instead of abusing _decode_parameters as text buffer. This should prevent several possible buffer overruns and is a bit cleaner to use. As bonus it reduces the size of most command packets by 79 bytes.
tron
parents:
1804
diff
changeset
|
15 |
|
0 | 16 |
#define DEF_COMMAND(yyyy) int32 yyyy(int x, int y, uint32 flags, uint32 p1, uint32 p2) |
17 |
||
18 |
DEF_COMMAND(CmdBuildRailroadTrack); |
|
19 |
DEF_COMMAND(CmdRemoveRailroadTrack); |
|
20 |
DEF_COMMAND(CmdBuildSingleRail); |
|
21 |
DEF_COMMAND(CmdRemoveSingleRail); |
|
22 |
||
23 |
DEF_COMMAND(CmdLandscapeClear); |
|
24 |
||
25 |
DEF_COMMAND(CmdBuildBridge); |
|
26 |
||
27 |
DEF_COMMAND(CmdBuildRailroadStation); |
|
28 |
DEF_COMMAND(CmdRemoveFromRailroadStation); |
|
29 |
DEF_COMMAND(CmdConvertRail); |
|
30 |
||
1227
3552f20fcfcb
(svn r1731) - Fix: [ 1106930 ] BugFix: placing signals with 2x1 drags workaround is completely rewritten. Also features checks for hacked/modified clients. Thanks a lot Hackykid!
darkvater
parents:
1217
diff
changeset
|
31 |
DEF_COMMAND(CmdBuildSingleSignal); |
3552f20fcfcb
(svn r1731) - Fix: [ 1106930 ] BugFix: placing signals with 2x1 drags workaround is completely rewritten. Also features checks for hacked/modified clients. Thanks a lot Hackykid!
darkvater
parents:
1217
diff
changeset
|
32 |
DEF_COMMAND(CmdRemoveSingleSignal); |
0 | 33 |
|
34 |
DEF_COMMAND(CmdTerraformLand); |
|
35 |
||
36 |
DEF_COMMAND(CmdPurchaseLandArea); |
|
37 |
DEF_COMMAND(CmdSellLandArea); |
|
38 |
||
39 |
DEF_COMMAND(CmdBuildTunnel); |
|
40 |
||
41 |
DEF_COMMAND(CmdBuildTrainDepot); |
|
395
4c990f33dab7
(svn r587) -newgrf: Rename all /Checkpoint/i tokens to 'Waypoint's. The name actually makes some sense and is also compatible with TTDPatch (pasky).
darkvater
parents:
215
diff
changeset
|
42 |
DEF_COMMAND(CmdBuildTrainWaypoint); |
4c990f33dab7
(svn r587) -newgrf: Rename all /Checkpoint/i tokens to 'Waypoint's. The name actually makes some sense and is also compatible with TTDPatch (pasky).
darkvater
parents:
215
diff
changeset
|
43 |
DEF_COMMAND(CmdRenameWaypoint); |
4c990f33dab7
(svn r587) -newgrf: Rename all /Checkpoint/i tokens to 'Waypoint's. The name actually makes some sense and is also compatible with TTDPatch (pasky).
darkvater
parents:
215
diff
changeset
|
44 |
DEF_COMMAND(CmdRemoveTrainWaypoint); |
0 | 45 |
|
1217
ab9f02a224ab
(svn r1721) -Feature: It is now possible to build multiple road stations (up to 8) on
celestar
parents:
1093
diff
changeset
|
46 |
DEF_COMMAND(CmdBuildRoadStop); |
0 | 47 |
|
48 |
DEF_COMMAND(CmdBuildLongRoad); |
|
49 |
DEF_COMMAND(CmdRemoveLongRoad); |
|
50 |
DEF_COMMAND(CmdBuildRoad); |
|
51 |
DEF_COMMAND(CmdRemoveRoad); |
|
52 |
||
53 |
DEF_COMMAND(CmdBuildRoadDepot); |
|
54 |
||
55 |
DEF_COMMAND(CmdBuildAirport); |
|
56 |
||
57 |
DEF_COMMAND(CmdBuildDock); |
|
58 |
||
59 |
DEF_COMMAND(CmdBuildShipDepot); |
|
60 |
||
61 |
DEF_COMMAND(CmdBuildBuoy); |
|
62 |
||
63 |
DEF_COMMAND(CmdPlantTree); |
|
64 |
||
65 |
DEF_COMMAND(CmdBuildRailVehicle); |
|
66 |
DEF_COMMAND(CmdMoveRailVehicle); |
|
67 |
||
68 |
DEF_COMMAND(CmdStartStopTrain); |
|
69 |
||
70 |
DEF_COMMAND(CmdSellRailWagon); |
|
71 |
||
1794
44f9deff97ed
(svn r2298) - CodeChange: removed CmdAbuses: CmdSetTownNameType(), CmdStartNewGame(), CmdCreateScenario(), CmdSetNewLandscapeType() and CmdGenRandomNewGame().
Darkvater
parents:
1786
diff
changeset
|
72 |
DEF_COMMAND(CmdSendTrainToDepot); |
0 | 73 |
DEF_COMMAND(CmdForceTrainProceed); |
74 |
DEF_COMMAND(CmdReverseTrainDirection); |
|
75 |
||
76 |
DEF_COMMAND(CmdModifyOrder); |
|
77 |
DEF_COMMAND(CmdSkipOrder); |
|
78 |
DEF_COMMAND(CmdDeleteOrder); |
|
79 |
DEF_COMMAND(CmdInsertOrder); |
|
80 |
DEF_COMMAND(CmdChangeTrainServiceInt); |
|
81 |
DEF_COMMAND(CmdRestoreOrderIndex); |
|
82 |
||
83 |
DEF_COMMAND(CmdBuildIndustry); |
|
84 |
||
85 |
DEF_COMMAND(CmdBuildCompanyHQ); |
|
86 |
DEF_COMMAND(CmdSetPlayerFace); |
|
87 |
DEF_COMMAND(CmdSetPlayerColor); |
|
88 |
||
89 |
DEF_COMMAND(CmdIncreaseLoan); |
|
90 |
DEF_COMMAND(CmdDecreaseLoan); |
|
91 |
||
92 |
DEF_COMMAND(CmdWantEnginePreview); |
|
93 |
||
94 |
DEF_COMMAND(CmdNameVehicle); |
|
95 |
DEF_COMMAND(CmdRenameEngine); |
|
96 |
||
97 |
DEF_COMMAND(CmdChangeCompanyName); |
|
98 |
DEF_COMMAND(CmdChangePresidentName); |
|
99 |
||
100 |
DEF_COMMAND(CmdRenameStation); |
|
101 |
||
102 |
DEF_COMMAND(CmdSellAircraft); |
|
103 |
DEF_COMMAND(CmdStartStopAircraft); |
|
104 |
DEF_COMMAND(CmdBuildAircraft); |
|
105 |
DEF_COMMAND(CmdSendAircraftToHangar); |
|
106 |
DEF_COMMAND(CmdChangeAircraftServiceInt); |
|
107 |
DEF_COMMAND(CmdRefitAircraft); |
|
108 |
||
109 |
DEF_COMMAND(CmdPlaceSign); |
|
110 |
DEF_COMMAND(CmdRenameSign); |
|
111 |
||
112 |
DEF_COMMAND(CmdBuildRoadVeh); |
|
113 |
DEF_COMMAND(CmdStartStopRoadVeh); |
|
114 |
DEF_COMMAND(CmdSellRoadVeh); |
|
115 |
DEF_COMMAND(CmdSendRoadVehToDepot); |
|
116 |
DEF_COMMAND(CmdTurnRoadVeh); |
|
117 |
DEF_COMMAND(CmdChangeRoadVehServiceInt); |
|
118 |
||
119 |
DEF_COMMAND(CmdPause); |
|
120 |
||
121 |
DEF_COMMAND(CmdBuyShareInCompany); |
|
122 |
DEF_COMMAND(CmdSellShareInCompany); |
|
123 |
DEF_COMMAND(CmdBuyCompany); |
|
124 |
||
125 |
DEF_COMMAND(CmdBuildTown); |
|
126 |
||
127 |
DEF_COMMAND(CmdRenameTown); |
|
128 |
DEF_COMMAND(CmdDoTownAction); |
|
129 |
||
130 |
DEF_COMMAND(CmdSetRoadDriveSide); |
|
131 |
||
132 |
DEF_COMMAND(CmdChangeDifficultyLevel); |
|
543
e3b43338096b
(svn r942) -Merged branch/network back into the trunk
truelight
parents:
534
diff
changeset
|
133 |
DEF_COMMAND(CmdChangePatchSetting); |
0 | 134 |
|
135 |
DEF_COMMAND(CmdStartStopShip); |
|
136 |
DEF_COMMAND(CmdSellShip); |
|
137 |
DEF_COMMAND(CmdBuildShip); |
|
138 |
DEF_COMMAND(CmdSendShipToDepot); |
|
139 |
DEF_COMMAND(CmdChangeShipServiceInt); |
|
140 |
DEF_COMMAND(CmdRefitShip); |
|
141 |
||
142 |
DEF_COMMAND(CmdCloneOrder); |
|
143 |
||
144 |
DEF_COMMAND(CmdClearArea); |
|
145 |
||
543
e3b43338096b
(svn r942) -Merged branch/network back into the trunk
truelight
parents:
534
diff
changeset
|
146 |
DEF_COMMAND(CmdGiveMoney); |
0 | 147 |
DEF_COMMAND(CmdMoneyCheat); |
148 |
DEF_COMMAND(CmdBuildCanal); |
|
147
d0c8100d18cf
(svn r148) -Feature: Company HQ can now be moved somewhere else (cost 1% of company value). Water floods HQ.
darkvater
parents:
58
diff
changeset
|
149 |
DEF_COMMAND(CmdBuildLock); |
0 | 150 |
|
151 |
DEF_COMMAND(CmdPlayerCtrl); |
|
152 |
||
153 |
DEF_COMMAND(CmdLevelLand); |
|
154 |
||
155 |
DEF_COMMAND(CmdRefitRailVehicle); |
|
156 |
||
1227
3552f20fcfcb
(svn r1731) - Fix: [ 1106930 ] BugFix: placing signals with 2x1 drags workaround is completely rewritten. Also features checks for hacked/modified clients. Thanks a lot Hackykid!
darkvater
parents:
1217
diff
changeset
|
157 |
DEF_COMMAND(CmdBuildSignalTrack); |
3552f20fcfcb
(svn r1731) - Fix: [ 1106930 ] BugFix: placing signals with 2x1 drags workaround is completely rewritten. Also features checks for hacked/modified clients. Thanks a lot Hackykid!
darkvater
parents:
1217
diff
changeset
|
158 |
DEF_COMMAND(CmdRemoveSignalTrack); |
58
b9fdcc9b5c90
(svn r59) -Feature: Added Autosignals, just like Autorail. Can copy signal style, convert signal<->semaphore, etc. Big thanks to betatesters Dribbel and Testman57 (Darkvater)
darkvater
parents:
0
diff
changeset
|
159 |
|
842 | 160 |
DEF_COMMAND(CmdReplaceVehicle); |
812
79c99885ad00
(svn r1283) -Add: AutoRenew is now a client-side patch instead of a game-side patch
truelight
parents:
679
diff
changeset
|
161 |
|
2244 | 162 |
DEF_COMMAND(CmdCloneVehicle); |
163 |
||
164 |
||
0 | 165 |
/* The master command table */ |
1804
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
166 |
static const Command _command_proc_table[] = { |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
167 |
{CmdBuildRailroadTrack, 0}, /* 0 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
168 |
{CmdRemoveRailroadTrack, 0}, /* 1 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
169 |
{CmdBuildSingleRail, 0}, /* 2 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
170 |
{CmdRemoveSingleRail, 0}, /* 3 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
171 |
{CmdLandscapeClear, 0}, /* 4 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
172 |
{CmdBuildBridge, 0}, /* 5 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
173 |
{CmdBuildRailroadStation, 0}, /* 6 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
174 |
{CmdBuildTrainDepot, 0}, /* 7 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
175 |
{CmdBuildSingleSignal, 0}, /* 8 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
176 |
{CmdRemoveSingleSignal, 0}, /* 9 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
177 |
{CmdTerraformLand, 0}, /* 10 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
178 |
{CmdPurchaseLandArea, 0}, /* 11 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
179 |
{CmdSellLandArea, 0}, /* 12 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
180 |
{CmdBuildTunnel, 0}, /* 13 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
181 |
{CmdRemoveFromRailroadStation, 0}, /* 14 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
182 |
{CmdConvertRail, 0}, /* 15 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
183 |
{CmdBuildTrainWaypoint, 0}, /* 16 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
184 |
{CmdRenameWaypoint, 0}, /* 17 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
185 |
{CmdRemoveTrainWaypoint, 0}, /* 18 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
186 |
{NULL, 0}, /* 19 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
187 |
{NULL, 0}, /* 20 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
188 |
{CmdBuildRoadStop, 0}, /* 21 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
189 |
{NULL, 0}, /* 22 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
190 |
{CmdBuildLongRoad, 0}, /* 23 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
191 |
{CmdRemoveLongRoad, 0}, /* 24 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
192 |
{CmdBuildRoad, 0}, /* 25 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
193 |
{CmdRemoveRoad, 0}, /* 26 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
194 |
{CmdBuildRoadDepot, 0}, /* 27 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
195 |
{NULL, 0}, /* 28 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
196 |
{CmdBuildAirport, 0}, /* 29 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
197 |
{CmdBuildDock, 0}, /* 30 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
198 |
{CmdBuildShipDepot, 0}, /* 31 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
199 |
{CmdBuildBuoy, 0}, /* 32 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
200 |
{CmdPlantTree, 0}, /* 33 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
201 |
{CmdBuildRailVehicle, 0}, /* 34 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
202 |
{CmdMoveRailVehicle, 0}, /* 35 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
203 |
{CmdStartStopTrain, 0}, /* 36 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
204 |
{NULL, 0}, /* 37 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
205 |
{CmdSellRailWagon, 0}, /* 38 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
206 |
{CmdSendTrainToDepot, 0}, /* 39 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
207 |
{CmdForceTrainProceed, 0}, /* 40 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
208 |
{CmdReverseTrainDirection, 0}, /* 41 */ |
147
d0c8100d18cf
(svn r148) -Feature: Company HQ can now be moved somewhere else (cost 1% of company value). Water floods HQ.
darkvater
parents:
58
diff
changeset
|
209 |
|
1804
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
210 |
{CmdModifyOrder, 0}, /* 42 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
211 |
{CmdSkipOrder, 0}, /* 43 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
212 |
{CmdDeleteOrder, 0}, /* 44 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
213 |
{CmdInsertOrder, 0}, /* 45 */ |
0 | 214 |
|
1804
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
215 |
{CmdChangeTrainServiceInt, 0}, /* 46 */ |
147
d0c8100d18cf
(svn r148) -Feature: Company HQ can now be moved somewhere else (cost 1% of company value). Water floods HQ.
darkvater
parents:
58
diff
changeset
|
216 |
|
1804
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
217 |
{CmdBuildIndustry, 0}, /* 47 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
218 |
{CmdBuildCompanyHQ, 0}, /* 48 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
219 |
{CmdSetPlayerFace, 0}, /* 49 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
220 |
{CmdSetPlayerColor, 0}, /* 50 */ |
1796
cae31916ae54
(svn r2300) - CodeChange: check the last number of commands, now only the refit ones remain, and some server-only commands.
Darkvater
parents:
1794
diff
changeset
|
221 |
|
1804
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
222 |
{CmdIncreaseLoan, 0}, /* 51 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
223 |
{CmdDecreaseLoan, 0}, /* 52 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
224 |
|
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
225 |
{CmdWantEnginePreview, 0}, /* 53 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
226 |
|
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
227 |
{CmdNameVehicle, 0}, /* 54 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
228 |
{CmdRenameEngine, 0}, /* 55 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
229 |
|
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
230 |
{CmdChangeCompanyName, 0}, /* 56 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
231 |
{CmdChangePresidentName, 0}, /* 57 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
232 |
|
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
233 |
{CmdRenameStation, 0}, /* 58 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
234 |
|
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
235 |
{CmdSellAircraft, 0}, /* 59 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
236 |
{CmdStartStopAircraft, 0}, /* 60 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
237 |
|
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
238 |
{CmdBuildAircraft, 0}, /* 61 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
239 |
{CmdSendAircraftToHangar, 0}, /* 62 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
240 |
{CmdChangeAircraftServiceInt, 0}, /* 63 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
241 |
{CmdRefitAircraft, 0}, /* 64 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
242 |
|
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
243 |
{CmdPlaceSign, 0}, /* 65 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
244 |
{CmdRenameSign, 0}, /* 66 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
245 |
|
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
246 |
{CmdBuildRoadVeh, 0}, /* 67 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
247 |
{CmdStartStopRoadVeh, 0}, /* 68 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
248 |
{CmdSellRoadVeh, 0}, /* 69 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
249 |
{CmdSendRoadVehToDepot, 0}, /* 70 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
250 |
{CmdTurnRoadVeh, 0}, /* 71 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
251 |
{CmdChangeRoadVehServiceInt, 0}, /* 72 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
252 |
|
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
253 |
{CmdPause, CMD_SERVER}, /* 73 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
254 |
|
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
255 |
{CmdBuyShareInCompany, 0}, /* 74 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
256 |
{CmdSellShareInCompany, 0}, /* 75 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
257 |
{CmdBuyCompany, 0}, /* 76 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
258 |
|
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
259 |
{CmdBuildTown, CMD_OFFLINE}, /* 77 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
260 |
{NULL, 0}, /* 78 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
261 |
{NULL, 0}, /* 79 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
262 |
{CmdRenameTown, CMD_SERVER}, /* 80 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
263 |
{CmdDoTownAction, 0}, /* 81 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
264 |
|
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
265 |
{CmdSetRoadDriveSide, CMD_SERVER}, /* 82 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
266 |
{NULL, 0}, /* 83 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
267 |
{NULL, 0}, /* 84 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
268 |
{CmdChangeDifficultyLevel, CMD_SERVER}, /* 85 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
269 |
|
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
270 |
{CmdStartStopShip, 0}, /* 86 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
271 |
{CmdSellShip, 0}, /* 87 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
272 |
{CmdBuildShip, 0}, /* 88 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
273 |
{CmdSendShipToDepot, 0}, /* 89 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
274 |
{CmdChangeShipServiceInt, 0}, /* 90 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
275 |
{CmdRefitShip, 0}, /* 91 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
276 |
|
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
277 |
{NULL, 0}, /* 92 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
278 |
{NULL, 0}, /* 93 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
279 |
{NULL, 0}, /* 94 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
280 |
{NULL, 0}, /* 95 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
281 |
{NULL, 0}, /* 96 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
282 |
{NULL, 0}, /* 97 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
283 |
{NULL, 0}, /* 98 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
284 |
|
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
285 |
{CmdCloneOrder, 0}, /* 99 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
286 |
|
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
287 |
{CmdClearArea, 0}, /* 100 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
288 |
{NULL, 0}, /* 101 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
289 |
|
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
290 |
{CmdMoneyCheat, CMD_OFFLINE}, /* 102 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
291 |
{CmdBuildCanal, 0}, /* 103 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
292 |
{CmdPlayerCtrl, 0}, /* 104 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
293 |
|
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
294 |
{CmdLevelLand, 0}, /* 105 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
295 |
|
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
296 |
{CmdRefitRailVehicle, 0}, /* 106 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
297 |
{CmdRestoreOrderIndex, 0}, /* 107 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
298 |
{CmdBuildLock, 0}, /* 108 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
299 |
{NULL, 0}, /* 109 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
300 |
{CmdBuildSignalTrack, 0}, /* 110 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
301 |
{CmdRemoveSignalTrack, 0}, /* 111 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
302 |
{NULL, 0}, /* 112 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
303 |
{CmdGiveMoney, 0}, /* 113 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
304 |
{CmdChangePatchSetting, CMD_SERVER}, /* 114 */ |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
305 |
{CmdReplaceVehicle, 0}, /* 115 */ |
2244 | 306 |
{CmdCloneVehicle, 0}, /* 116 */ |
0 | 307 |
}; |
308 |
||
903
f860c7234167
(svn r1389) -Add: [Network] Added packet protection. No longer a client or server
truelight
parents:
889
diff
changeset
|
309 |
/* This function range-checks a cmd, and checks if the cmd is not NULL */ |
959
b031d88c76f3
(svn r1451) Fix some of the signed/unsigned comparison warnings
tron
parents:
926
diff
changeset
|
310 |
bool IsValidCommand(uint cmd) |
903
f860c7234167
(svn r1389) -Add: [Network] Added packet protection. No longer a client or server
truelight
parents:
889
diff
changeset
|
311 |
{ |
2645
964bd8fe3ce2
(svn r3187) Simplify overly complicated ifs, especially if (foo) return false; else return true; is confusing
tron
parents:
2425
diff
changeset
|
312 |
cmd &= 0xFF; |
903
f860c7234167
(svn r1389) -Add: [Network] Added packet protection. No longer a client or server
truelight
parents:
889
diff
changeset
|
313 |
|
2645
964bd8fe3ce2
(svn r3187) Simplify overly complicated ifs, especially if (foo) return false; else return true; is confusing
tron
parents:
2425
diff
changeset
|
314 |
return |
964bd8fe3ce2
(svn r3187) Simplify overly complicated ifs, especially if (foo) return false; else return true; is confusing
tron
parents:
2425
diff
changeset
|
315 |
cmd < lengthof(_command_proc_table) && |
964bd8fe3ce2
(svn r3187) Simplify overly complicated ifs, especially if (foo) return false; else return true; is confusing
tron
parents:
2425
diff
changeset
|
316 |
_command_proc_table[cmd].proc != NULL; |
903
f860c7234167
(svn r1389) -Add: [Network] Added packet protection. No longer a client or server
truelight
parents:
889
diff
changeset
|
317 |
} |
f860c7234167
(svn r1389) -Add: [Network] Added packet protection. No longer a client or server
truelight
parents:
889
diff
changeset
|
318 |
|
1804
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
319 |
byte GetCommandFlags(uint cmd) {return _command_proc_table[cmd & 0xFF].flags;} |
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
320 |
|
0 | 321 |
int32 DoCommandByTile(TileIndex tile, uint32 p1, uint32 p2, uint32 flags, uint procc) |
322 |
{ |
|
926
bd4312619522
(svn r1414) Move TileIndex, TILE_MASK and GET_TILE_[XY] to map.h and turn the latter into inline functions names Tile[XY]
tron
parents:
903
diff
changeset
|
323 |
return DoCommand(TileX(tile) * 16, TileY(tile) * 16, p1, p2, flags, procc); |
193
0a7025304867
(svn r194) -Codechange: stripping trailing-spaces. Please keep this that way!
truelight
parents:
147
diff
changeset
|
324 |
} |
0 | 325 |
|
326 |
||
2304
f3309898aeb0
(svn r2828) Only command.c needs to know about _docommand_recursive, so make it static. Tell me if there's a problem, because i removed it from network_server.c, but afaict it should be ok
tron
parents:
2244
diff
changeset
|
327 |
static int _docommand_recursive; |
f3309898aeb0
(svn r2828) Only command.c needs to know about _docommand_recursive, so make it static. Tell me if there's a problem, because i removed it from network_server.c, but afaict it should be ok
tron
parents:
2244
diff
changeset
|
328 |
|
0 | 329 |
int32 DoCommand(int x, int y, uint32 p1, uint32 p2, uint32 flags, uint procc) |
330 |
{ |
|
331 |
int32 res; |
|
332 |
CommandProc *proc; |
|
193
0a7025304867
(svn r194) -Codechange: stripping trailing-spaces. Please keep this that way!
truelight
parents:
147
diff
changeset
|
333 |
|
1776
fa776997bba2
(svn r2280) - Fix: Check if the passed tile to DoCommand is within the current mapsize
Darkvater
parents:
1728
diff
changeset
|
334 |
/* Do not even think about executing out-of-bounds tile-commands */ |
2809
5bec9d1a7d6b
(svn r3357) Fix an off-by-one error in checking if a coordinate passed to a command is on the map
tron
parents:
2772
diff
changeset
|
335 |
if (TileVirtXY(x, y) >= MapSize()) { |
1820
9b6458526480
(svn r2324) Introduce _cmd_text for passing strings with a command instead of abusing _decode_parameters as text buffer. This should prevent several possible buffer overruns and is a bit cleaner to use. As bonus it reduces the size of most command packets by 79 bytes.
tron
parents:
1804
diff
changeset
|
336 |
_cmd_text = NULL; |
9b6458526480
(svn r2324) Introduce _cmd_text for passing strings with a command instead of abusing _decode_parameters as text buffer. This should prevent several possible buffer overruns and is a bit cleaner to use. As bonus it reduces the size of most command packets by 79 bytes.
tron
parents:
1804
diff
changeset
|
337 |
return CMD_ERROR; |
9b6458526480
(svn r2324) Introduce _cmd_text for passing strings with a command instead of abusing _decode_parameters as text buffer. This should prevent several possible buffer overruns and is a bit cleaner to use. As bonus it reduces the size of most command packets by 79 bytes.
tron
parents:
1804
diff
changeset
|
338 |
} |
1776
fa776997bba2
(svn r2280) - Fix: Check if the passed tile to DoCommand is within the current mapsize
Darkvater
parents:
1728
diff
changeset
|
339 |
|
1804
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
340 |
proc = _command_proc_table[procc].proc; |
0 | 341 |
|
2749
46aba09ca365
(svn r3294) - Fix: use INVALID_STRING_ID instead of -1.
Darkvater
parents:
2715
diff
changeset
|
342 |
if (_docommand_recursive == 0) _error_message = INVALID_STRING_ID; |
0 | 343 |
|
344 |
_docommand_recursive++; |
|
345 |
||
346 |
// only execute the test call if it's toplevel, or we're not execing. |
|
347 |
if (_docommand_recursive == 1 || !(flags & DC_EXEC) || (flags & DC_FORCETEST) ) { |
|
348 |
res = proc(x, y, flags&~DC_EXEC, p1, p2); |
|
1691
fcd9fefaed02
(svn r2195) Add CmdFailed() as the One True Way(tm) to check if a command failed.
tron
parents:
1623
diff
changeset
|
349 |
if (CmdFailed(res)) { |
0 | 350 |
if (res & 0xFFFF) _error_message = res & 0xFFFF; |
351 |
goto error; |
|
352 |
} |
|
353 |
||
354 |
if (_docommand_recursive == 1) { |
|
355 |
if (!(flags&DC_QUERY_COST) && res != 0 && !CheckPlayerHasMoney(res)) |
|
356 |
goto error; |
|
357 |
} |
|
358 |
||
359 |
if (!(flags & DC_EXEC)) { |
|
360 |
_docommand_recursive--; |
|
1820
9b6458526480
(svn r2324) Introduce _cmd_text for passing strings with a command instead of abusing _decode_parameters as text buffer. This should prevent several possible buffer overruns and is a bit cleaner to use. As bonus it reduces the size of most command packets by 79 bytes.
tron
parents:
1804
diff
changeset
|
361 |
_cmd_text = NULL; |
0 | 362 |
return res; |
363 |
} |
|
364 |
} |
|
365 |
||
889
7f8630bfea41
(svn r1375) -Fix: [1050990] Buying trains sometimes accounted for incorrectly. Was the result of the cost getting reset in a recursive call of docommand. That is fixed. In addition all cost-commands are typed explicitely. Please do not forget to do so or your costs will be credited to construction if you are unlucky.
darkvater
parents:
842
diff
changeset
|
366 |
/* Execute the command here. All cost-relevant functions set the expenses type |
7f8630bfea41
(svn r1375) -Fix: [1050990] Buying trains sometimes accounted for incorrectly. Was the result of the cost getting reset in a recursive call of docommand. That is fixed. In addition all cost-commands are typed explicitely. Please do not forget to do so or your costs will be credited to construction if you are unlucky.
darkvater
parents:
842
diff
changeset
|
367 |
* themselves with "SET_EXPENSES_TYPE(...);" at the beginning of the function */ |
0 | 368 |
res = proc(x, y, flags, p1, p2); |
1691
fcd9fefaed02
(svn r2195) Add CmdFailed() as the One True Way(tm) to check if a command failed.
tron
parents:
1623
diff
changeset
|
369 |
if (CmdFailed(res)) { |
0 | 370 |
if (res & 0xFFFF) _error_message = res & 0xFFFF; |
371 |
error: |
|
372 |
_docommand_recursive--; |
|
1820
9b6458526480
(svn r2324) Introduce _cmd_text for passing strings with a command instead of abusing _decode_parameters as text buffer. This should prevent several possible buffer overruns and is a bit cleaner to use. As bonus it reduces the size of most command packets by 79 bytes.
tron
parents:
1804
diff
changeset
|
373 |
_cmd_text = NULL; |
0 | 374 |
return CMD_ERROR; |
375 |
} |
|
376 |
||
377 |
// if toplevel, subtract the money. |
|
378 |
if (--_docommand_recursive == 0) { |
|
379 |
SubtractMoneyFromPlayer(res); |
|
2772
96e47991fd1d
(svn r3319) - Fix (regression): "Unnamed Competitors". A result of revision r3224 which allowed AI's for multiplayer. Old AI's didn't function correctly anymore. The update of last-built tile is moved back, but to a slightly different place so it is only executed when the command is actually successfully executed. This code shouldn't be neccessary, but alas, the old AI doesn't use DoCommandDP() functions.
Darkvater
parents:
2767
diff
changeset
|
380 |
// XXX - Old AI hack which doesn't use DoCommandDP; update last build coord of player |
96e47991fd1d
(svn r3319) - Fix (regression): "Unnamed Competitors". A result of revision r3224 which allowed AI's for multiplayer. Old AI's didn't function correctly anymore. The update of last-built tile is moved back, but to a slightly different place so it is only executed when the command is actually successfully executed. This code shouldn't be neccessary, but alas, the old AI doesn't use DoCommandDP() functions.
Darkvater
parents:
2767
diff
changeset
|
381 |
if ( (x|y) != 0 && _current_player < MAX_PLAYERS) { |
96e47991fd1d
(svn r3319) - Fix (regression): "Unnamed Competitors". A result of revision r3224 which allowed AI's for multiplayer. Old AI's didn't function correctly anymore. The update of last-built tile is moved back, but to a slightly different place so it is only executed when the command is actually successfully executed. This code shouldn't be neccessary, but alas, the old AI doesn't use DoCommandDP() functions.
Darkvater
parents:
2767
diff
changeset
|
382 |
GetPlayer(_current_player)->last_build_coordinate = TileVirtXY(x, y); |
96e47991fd1d
(svn r3319) - Fix (regression): "Unnamed Competitors". A result of revision r3224 which allowed AI's for multiplayer. Old AI's didn't function correctly anymore. The update of last-built tile is moved back, but to a slightly different place so it is only executed when the command is actually successfully executed. This code shouldn't be neccessary, but alas, the old AI doesn't use DoCommandDP() functions.
Darkvater
parents:
2767
diff
changeset
|
383 |
} |
0 | 384 |
} |
385 |
||
1820
9b6458526480
(svn r2324) Introduce _cmd_text for passing strings with a command instead of abusing _decode_parameters as text buffer. This should prevent several possible buffer overruns and is a bit cleaner to use. As bonus it reduces the size of most command packets by 79 bytes.
tron
parents:
1804
diff
changeset
|
386 |
_cmd_text = NULL; |
0 | 387 |
return res; |
388 |
} |
|
389 |
||
1093
e8d26c7dc42f
(svn r1594) Convert all undefined parameter lists to (void) and add the appropriate warning flags in the Makefile
tron
parents:
959
diff
changeset
|
390 |
int32 GetAvailableMoneyForCommand(void) |
0 | 391 |
{ |
1794
44f9deff97ed
(svn r2298) - CodeChange: removed CmdAbuses: CmdSetTownNameType(), CmdStartNewGame(), CmdCreateScenario(), CmdSetNewLandscapeType() and CmdGenRandomNewGame().
Darkvater
parents:
1786
diff
changeset
|
392 |
PlayerID pid = _current_player; |
147
d0c8100d18cf
(svn r148) -Feature: Company HQ can now be moved somewhere else (cost 1% of company value). Water floods HQ.
darkvater
parents:
58
diff
changeset
|
393 |
if (pid >= MAX_PLAYERS) return 0x7FFFFFFF; // max int |
1962
51ee4f459268
(svn r2468) -Codechange: Got rid of DEREF_PLAYER and replaced it by GetPlayer
celestar
parents:
1891
diff
changeset
|
394 |
return GetPlayer(pid)->player_money; |
0 | 395 |
} |
396 |
||
397 |
// toplevel network safe docommand function for the current player. must not be called recursively. |
|
398 |
// the callback is called when the command succeeded or failed. |
|
399 |
bool DoCommandP(TileIndex tile, uint32 p1, uint32 p2, CommandCallback *callback, uint32 cmd) |
|
400 |
{ |
|
401 |
int32 res = 0,res2; |
|
402 |
CommandProc *proc; |
|
403 |
uint32 flags; |
|
404 |
bool notest; |
|
405 |
||
926
bd4312619522
(svn r1414) Move TileIndex, TILE_MASK and GET_TILE_[XY] to map.h and turn the latter into inline functions names Tile[XY]
tron
parents:
903
diff
changeset
|
406 |
int x = TileX(tile) * 16; |
bd4312619522
(svn r1414) Move TileIndex, TILE_MASK and GET_TILE_[XY] to map.h and turn the latter into inline functions names Tile[XY]
tron
parents:
903
diff
changeset
|
407 |
int y = TileY(tile) * 16; |
0 | 408 |
|
1776
fa776997bba2
(svn r2280) - Fix: Check if the passed tile to DoCommand is within the current mapsize
Darkvater
parents:
1728
diff
changeset
|
409 |
/* Do not even think about executing out-of-bounds tile-commands */ |
1820
9b6458526480
(svn r2324) Introduce _cmd_text for passing strings with a command instead of abusing _decode_parameters as text buffer. This should prevent several possible buffer overruns and is a bit cleaner to use. As bonus it reduces the size of most command packets by 79 bytes.
tron
parents:
1804
diff
changeset
|
410 |
if (tile > MapSize()) { |
9b6458526480
(svn r2324) Introduce _cmd_text for passing strings with a command instead of abusing _decode_parameters as text buffer. This should prevent several possible buffer overruns and is a bit cleaner to use. As bonus it reduces the size of most command packets by 79 bytes.
tron
parents:
1804
diff
changeset
|
411 |
_cmd_text = NULL; |
9b6458526480
(svn r2324) Introduce _cmd_text for passing strings with a command instead of abusing _decode_parameters as text buffer. This should prevent several possible buffer overruns and is a bit cleaner to use. As bonus it reduces the size of most command packets by 79 bytes.
tron
parents:
1804
diff
changeset
|
412 |
return false; |
9b6458526480
(svn r2324) Introduce _cmd_text for passing strings with a command instead of abusing _decode_parameters as text buffer. This should prevent several possible buffer overruns and is a bit cleaner to use. As bonus it reduces the size of most command packets by 79 bytes.
tron
parents:
1804
diff
changeset
|
413 |
} |
1776
fa776997bba2
(svn r2280) - Fix: Check if the passed tile to DoCommand is within the current mapsize
Darkvater
parents:
1728
diff
changeset
|
414 |
|
193
0a7025304867
(svn r194) -Codechange: stripping trailing-spaces. Please keep this that way!
truelight
parents:
147
diff
changeset
|
415 |
assert(_docommand_recursive == 0); |
0 | 416 |
|
417 |
_error_message = INVALID_STRING_ID; |
|
418 |
_error_message_2 = cmd >> 16; |
|
419 |
_additional_cash_required = 0; |
|
420 |
||
1834
33612fb9c7d7
(svn r2339) - Fix: on a dedicated server, after the joining of a player, the player was set to player 0 instead of remaining OWNER_SPECTATOR. Fix this, and allow the dedicated server to execute commands (so 'patch <value>') now works and not only when a client is connected.
Darkvater
parents:
1820
diff
changeset
|
421 |
/** Spectator has no rights except for the dedicated server which |
33612fb9c7d7
(svn r2339) - Fix: on a dedicated server, after the joining of a player, the player was set to player 0 instead of remaining OWNER_SPECTATOR. Fix this, and allow the dedicated server to execute commands (so 'patch <value>') now works and not only when a client is connected.
Darkvater
parents:
1820
diff
changeset
|
422 |
* is a spectator but is the server, so can do anything */ |
33612fb9c7d7
(svn r2339) - Fix: on a dedicated server, after the joining of a player, the player was set to player 0 instead of remaining OWNER_SPECTATOR. Fix this, and allow the dedicated server to execute commands (so 'patch <value>') now works and not only when a client is connected.
Darkvater
parents:
1820
diff
changeset
|
423 |
if (_current_player == OWNER_SPECTATOR && !_network_dedicated) { |
0 | 424 |
ShowErrorMessage(_error_message, _error_message_2, x, y); |
1820
9b6458526480
(svn r2324) Introduce _cmd_text for passing strings with a command instead of abusing _decode_parameters as text buffer. This should prevent several possible buffer overruns and is a bit cleaner to use. As bonus it reduces the size of most command packets by 79 bytes.
tron
parents:
1804
diff
changeset
|
425 |
_cmd_text = NULL; |
0 | 426 |
return false; |
427 |
} |
|
428 |
||
429 |
flags = 0; |
|
430 |
if (cmd & CMD_AUTO) flags |= DC_AUTO; |
|
431 |
if (cmd & CMD_NO_WATER) flags |= DC_NO_WATER; |
|
432 |
||
433 |
// get pointer to command handler |
|
434 |
assert((cmd & 0xFF) < lengthof(_command_proc_table)); |
|
1804
7810fc0aa941
(svn r2308) - Fix: enforce server-only and/or offline commands by giving them flags in the process table. This also fixes bug "[ 1190944 ] Many commands not checked for security"
Darkvater
parents:
1802
diff
changeset
|
435 |
proc = _command_proc_table[cmd & 0xFF].proc; |
1877
e7318705dd04
(svn r2383) - Fix: Monkey-testing turned up some command crashes.
Darkvater
parents:
1838
diff
changeset
|
436 |
if (proc == NULL) { |
e7318705dd04
(svn r2383) - Fix: Monkey-testing turned up some command crashes.
Darkvater
parents:
1838
diff
changeset
|
437 |
_cmd_text = NULL; |
e7318705dd04
(svn r2383) - Fix: Monkey-testing turned up some command crashes.
Darkvater
parents:
1838
diff
changeset
|
438 |
return false; |
e7318705dd04
(svn r2383) - Fix: Monkey-testing turned up some command crashes.
Darkvater
parents:
1838
diff
changeset
|
439 |
} |
0 | 440 |
|
826 | 441 |
// Some commands have a different output in dryrun than the realrun |
543
e3b43338096b
(svn r942) -Merged branch/network back into the trunk
truelight
parents:
534
diff
changeset
|
442 |
// e.g.: if you demolish a whole town, the dryrun would say okay. |
e3b43338096b
(svn r942) -Merged branch/network back into the trunk
truelight
parents:
534
diff
changeset
|
443 |
// but by really destroying, your rating drops and at a certain point |
e3b43338096b
(svn r942) -Merged branch/network back into the trunk
truelight
parents:
534
diff
changeset
|
444 |
// it will fail. so res and res2 are different |
467
6f3e282fdce4
(svn r697) Fix the town road removal crash _still_ present there.
pasky
parents:
395
diff
changeset
|
445 |
// CMD_REMOVE_ROAD: This command has special local authority |
6f3e282fdce4
(svn r697) Fix the town road removal crash _still_ present there.
pasky
parents:
395
diff
changeset
|
446 |
// restrictions which may cause the test run to fail (the previous |
6f3e282fdce4
(svn r697) Fix the town road removal crash _still_ present there.
pasky
parents:
395
diff
changeset
|
447 |
// road fragments still stay there and the town won't let you |
6f3e282fdce4
(svn r697) Fix the town road removal crash _still_ present there.
pasky
parents:
395
diff
changeset
|
448 |
// disconnect the road system), but the exec will succeed and this |
6f3e282fdce4
(svn r697) Fix the town road removal crash _still_ present there.
pasky
parents:
395
diff
changeset
|
449 |
// fact will trigger an assertion failure. --pasky |
193
0a7025304867
(svn r194) -Codechange: stripping trailing-spaces. Please keep this that way!
truelight
parents:
147
diff
changeset
|
450 |
notest = |
0a7025304867
(svn r194) -Codechange: stripping trailing-spaces. Please keep this that way!
truelight
parents:
147
diff
changeset
|
451 |
(cmd & 0xFF) == CMD_CLEAR_AREA || |
0a7025304867
(svn r194) -Codechange: stripping trailing-spaces. Please keep this that way!
truelight
parents:
147
diff
changeset
|
452 |
(cmd & 0xFF) == CMD_CONVERT_RAIL || |
0 | 453 |
(cmd & 0xFF) == CMD_LEVEL_LAND || |
1701
e46ac9ca353e
(svn r2205) - Fixup of revision 2200, not twice CMD_REMOVE_LONG_ROAD :p. Thanks TrueLight.
Darkvater
parents:
1696
diff
changeset
|
454 |
(cmd & 0xFF) == CMD_REMOVE_ROAD || |
1696
b3979425aa5a
(svn r2200) - Fix [ 1179892 ] click & drag removal of road assertion fail. Add CMD_REMOVE_LONG_ROAD to the 'notest' section as it might have difficulties with town ratings, etc. Read the comment for more info (mgasterix)
Darkvater
parents:
1691
diff
changeset
|
455 |
(cmd & 0xFF) == CMD_REMOVE_LONG_ROAD; |
0 | 456 |
|
457 |
_docommand_recursive = 1; |
|
458 |
||
459 |
// cost estimation only? |
|
2425
99e1e8430a28
(svn r2951) - Fix: [ 1259345 ] Changing engine in netgame opens train window for everyone
Darkvater
parents:
2304
diff
changeset
|
460 |
if (_shift_pressed && IsLocalPlayer() && !(cmd & (CMD_NETWORK_COMMAND | CMD_SHOW_NO_ERROR))) { |
0 | 461 |
// estimate the cost. |
462 |
res = proc(x, y, flags, p1, p2); |
|
1691
fcd9fefaed02
(svn r2195) Add CmdFailed() as the One True Way(tm) to check if a command failed.
tron
parents:
1623
diff
changeset
|
463 |
if (CmdFailed(res)) { |
0 | 464 |
if (res & 0xFFFF) _error_message = res & 0xFFFF; |
465 |
ShowErrorMessage(_error_message, _error_message_2, x, y); |
|
466 |
} else { |
|
467 |
ShowEstimatedCostOrIncome(res, x, y); |
|
468 |
} |
|
469 |
||
470 |
_docommand_recursive = 0; |
|
1820
9b6458526480
(svn r2324) Introduce _cmd_text for passing strings with a command instead of abusing _decode_parameters as text buffer. This should prevent several possible buffer overruns and is a bit cleaner to use. As bonus it reduces the size of most command packets by 79 bytes.
tron
parents:
1804
diff
changeset
|
471 |
_cmd_text = NULL; |
0 | 472 |
return false; |
473 |
} |
|
474 |
||
193
0a7025304867
(svn r194) -Codechange: stripping trailing-spaces. Please keep this that way!
truelight
parents:
147
diff
changeset
|
475 |
|
543
e3b43338096b
(svn r942) -Merged branch/network back into the trunk
truelight
parents:
534
diff
changeset
|
476 |
if (!((cmd & CMD_NO_TEST_IF_IN_NETWORK) && _networking)) { |
0 | 477 |
// first test if the command can be executed. |
478 |
res = proc(x,y, flags, p1, p2); |
|
1691
fcd9fefaed02
(svn r2195) Add CmdFailed() as the One True Way(tm) to check if a command failed.
tron
parents:
1623
diff
changeset
|
479 |
if (CmdFailed(res)) { |
0 | 480 |
if (res & 0xFFFF) _error_message = res & 0xFFFF; |
481 |
goto show_error; |
|
482 |
} |
|
543
e3b43338096b
(svn r942) -Merged branch/network back into the trunk
truelight
parents:
534
diff
changeset
|
483 |
// no money? Only check if notest is off |
2767 | 484 |
if (!notest && res != 0 && !CheckPlayerHasMoney(res)) goto show_error; |
0 | 485 |
} |
486 |
||
543
e3b43338096b
(svn r942) -Merged branch/network back into the trunk
truelight
parents:
534
diff
changeset
|
487 |
#ifdef ENABLE_NETWORK |
1838
522076c550c0
(svn r2343) - Fix (regression): dedicated server caused desyncs with its commands, because the player sent over the network is OWNER_SPECTATOR as well, which on clients cannot execute anything. So *hack*hack* it into temporarily player 0.
Darkvater
parents:
1834
diff
changeset
|
488 |
/** If we are in network, and the command is not from the network |
522076c550c0
(svn r2343) - Fix (regression): dedicated server caused desyncs with its commands, because the player sent over the network is OWNER_SPECTATOR as well, which on clients cannot execute anything. So *hack*hack* it into temporarily player 0.
Darkvater
parents:
1834
diff
changeset
|
489 |
* send it to the command-queue and abort execution |
522076c550c0
(svn r2343) - Fix (regression): dedicated server caused desyncs with its commands, because the player sent over the network is OWNER_SPECTATOR as well, which on clients cannot execute anything. So *hack*hack* it into temporarily player 0.
Darkvater
parents:
1834
diff
changeset
|
490 |
* If we are a dedicated server temporarily switch local player, otherwise |
522076c550c0
(svn r2343) - Fix (regression): dedicated server caused desyncs with its commands, because the player sent over the network is OWNER_SPECTATOR as well, which on clients cannot execute anything. So *hack*hack* it into temporarily player 0.
Darkvater
parents:
1834
diff
changeset
|
491 |
* the other parties won't be able to execute our command and will desync. |
522076c550c0
(svn r2343) - Fix (regression): dedicated server caused desyncs with its commands, because the player sent over the network is OWNER_SPECTATOR as well, which on clients cannot execute anything. So *hack*hack* it into temporarily player 0.
Darkvater
parents:
1834
diff
changeset
|
492 |
* @todo Rewrite dedicated server to something more than a dirty hack! |
522076c550c0
(svn r2343) - Fix (regression): dedicated server caused desyncs with its commands, because the player sent over the network is OWNER_SPECTATOR as well, which on clients cannot execute anything. So *hack*hack* it into temporarily player 0.
Darkvater
parents:
1834
diff
changeset
|
493 |
*/ |
543
e3b43338096b
(svn r942) -Merged branch/network back into the trunk
truelight
parents:
534
diff
changeset
|
494 |
if (_networking && !(cmd & CMD_NETWORK_COMMAND)) { |
1838
522076c550c0
(svn r2343) - Fix (regression): dedicated server caused desyncs with its commands, because the player sent over the network is OWNER_SPECTATOR as well, which on clients cannot execute anything. So *hack*hack* it into temporarily player 0.
Darkvater
parents:
1834
diff
changeset
|
495 |
if (_network_dedicated) _local_player = 0; |
543
e3b43338096b
(svn r942) -Merged branch/network back into the trunk
truelight
parents:
534
diff
changeset
|
496 |
NetworkSend_Command(tile, p1, p2, cmd, callback); |
1838
522076c550c0
(svn r2343) - Fix (regression): dedicated server caused desyncs with its commands, because the player sent over the network is OWNER_SPECTATOR as well, which on clients cannot execute anything. So *hack*hack* it into temporarily player 0.
Darkvater
parents:
1834
diff
changeset
|
497 |
if (_network_dedicated) _local_player = OWNER_SPECTATOR; |
543
e3b43338096b
(svn r942) -Merged branch/network back into the trunk
truelight
parents:
534
diff
changeset
|
498 |
_docommand_recursive = 0; |
1820
9b6458526480
(svn r2324) Introduce _cmd_text for passing strings with a command instead of abusing _decode_parameters as text buffer. This should prevent several possible buffer overruns and is a bit cleaner to use. As bonus it reduces the size of most command packets by 79 bytes.
tron
parents:
1804
diff
changeset
|
499 |
_cmd_text = NULL; |
543
e3b43338096b
(svn r942) -Merged branch/network back into the trunk
truelight
parents:
534
diff
changeset
|
500 |
return true; |
0 | 501 |
} |
543
e3b43338096b
(svn r942) -Merged branch/network back into the trunk
truelight
parents:
534
diff
changeset
|
502 |
#endif /* ENABLE_NETWORK */ |
0 | 503 |
|
504 |
// update last build coordinate of player. |
|
1962
51ee4f459268
(svn r2468) -Codechange: Got rid of DEREF_PLAYER and replaced it by GetPlayer
celestar
parents:
1891
diff
changeset
|
505 |
if ( tile != 0 && _current_player < MAX_PLAYERS) GetPlayer(_current_player)->last_build_coordinate = tile; |
0 | 506 |
|
889
7f8630bfea41
(svn r1375) -Fix: [1050990] Buying trains sometimes accounted for incorrectly. Was the result of the cost getting reset in a recursive call of docommand. That is fixed. In addition all cost-commands are typed explicitely. Please do not forget to do so or your costs will be credited to construction if you are unlucky.
darkvater
parents:
842
diff
changeset
|
507 |
/* Actually try and execute the command. If no cost-type is given |
7f8630bfea41
(svn r1375) -Fix: [1050990] Buying trains sometimes accounted for incorrectly. Was the result of the cost getting reset in a recursive call of docommand. That is fixed. In addition all cost-commands are typed explicitely. Please do not forget to do so or your costs will be credited to construction if you are unlucky.
darkvater
parents:
842
diff
changeset
|
508 |
* use the construction one */ |
7f8630bfea41
(svn r1375) -Fix: [1050990] Buying trains sometimes accounted for incorrectly. Was the result of the cost getting reset in a recursive call of docommand. That is fixed. In addition all cost-commands are typed explicitely. Please do not forget to do so or your costs will be credited to construction if you are unlucky.
darkvater
parents:
842
diff
changeset
|
509 |
_yearly_expenses_type = EXPENSES_CONSTRUCTION; |
0 | 510 |
res2 = proc(x,y, flags|DC_EXEC, p1, p2); |
193
0a7025304867
(svn r194) -Codechange: stripping trailing-spaces. Please keep this that way!
truelight
parents:
147
diff
changeset
|
511 |
|
826 | 512 |
// If notest is on, it means the result of the test can be different than |
543
e3b43338096b
(svn r942) -Merged branch/network back into the trunk
truelight
parents:
534
diff
changeset
|
513 |
// the real command.. so ignore the test |
652
0f553238c98a
(svn r1085) -Fix: [Network] [ 1084834 ] If IF_IN_NETWORK flag was on, an error
truelight
parents:
601
diff
changeset
|
514 |
if (!notest && !((cmd & CMD_NO_TEST_IF_IN_NETWORK) && _networking)) { |
0 | 515 |
assert(res == res2); // sanity check |
516 |
} else { |
|
1702
6402cfbb9772
(svn r2206) - Regression Fix: slight typo of res2 into res, this also fixes recent bug "[ 1183396 ] Train can't find depot, it gives money in Multiplayer"
Darkvater
parents:
1701
diff
changeset
|
517 |
if (CmdFailed(res2)) { |
0 | 518 |
if (res2 & 0xFFFF) _error_message = res2 & 0xFFFF; |
519 |
goto show_error; |
|
520 |
} |
|
521 |
} |
|
522 |
||
523 |
SubtractMoneyFromPlayer(res2); |
|
524 |
||
2425
99e1e8430a28
(svn r2951) - Fix: [ 1259345 ] Changing engine in netgame opens train window for everyone
Darkvater
parents:
2304
diff
changeset
|
525 |
if (IsLocalPlayer() && _game_mode != GM_EDITOR) { |
0 | 526 |
if (res2 != 0) |
527 |
ShowCostOrIncomeAnimation(x, y, GetSlopeZ(x, y), res2); |
|
528 |
if (_additional_cash_required) { |
|
534
17ab2f22ff74
(svn r901) Small step in the process to clean up the DPARAM mess:
tron
parents:
507
diff
changeset
|
529 |
SetDParam(0, _additional_cash_required); |
0 | 530 |
ShowErrorMessage(STR_0003_NOT_ENOUGH_CASH_REQUIRES, _error_message_2, x,y); |
531 |
if (res2 == 0) goto callb_err; |
|
532 |
} |
|
533 |
} |
|
534 |
||
535 |
_docommand_recursive = 0; |
|
193
0a7025304867
(svn r194) -Codechange: stripping trailing-spaces. Please keep this that way!
truelight
parents:
147
diff
changeset
|
536 |
|
0 | 537 |
if (callback) callback(true, tile, p1, p2); |
1820
9b6458526480
(svn r2324) Introduce _cmd_text for passing strings with a command instead of abusing _decode_parameters as text buffer. This should prevent several possible buffer overruns and is a bit cleaner to use. As bonus it reduces the size of most command packets by 79 bytes.
tron
parents:
1804
diff
changeset
|
538 |
_cmd_text = NULL; |
0 | 539 |
return true; |
540 |
||
541 |
show_error: |
|
542 |
// show error message if the command fails? |
|
2425
99e1e8430a28
(svn r2951) - Fix: [ 1259345 ] Changing engine in netgame opens train window for everyone
Darkvater
parents:
2304
diff
changeset
|
543 |
if (IsLocalPlayer() && _error_message_2 != 0) |
0 | 544 |
ShowErrorMessage(_error_message, _error_message_2, x,y); |
545 |
||
546 |
callb_err: |
|
547 |
_docommand_recursive = 0; |
|
193
0a7025304867
(svn r194) -Codechange: stripping trailing-spaces. Please keep this that way!
truelight
parents:
147
diff
changeset
|
548 |
|
0 | 549 |
if (callback) callback(false, tile, p1, p2); |
1820
9b6458526480
(svn r2324) Introduce _cmd_text for passing strings with a command instead of abusing _decode_parameters as text buffer. This should prevent several possible buffer overruns and is a bit cleaner to use. As bonus it reduces the size of most command packets by 79 bytes.
tron
parents:
1804
diff
changeset
|
550 |
_cmd_text = NULL; |
0 | 551 |
return false; |
552 |
} |