From 5e6923e9369a5a531d9b64da45fcfcc91f7687de Mon Sep 17 00:00:00 2001 From: Darkvater Date: Wed, 11 May 2005 00:00:27 +0000 Subject: (svn r2290) - CodeChange: protect the next batch of commands. This brings us to a total of 61, which is 53% :) - CodeChange: To correctly accept engine-prototypes, the best-player checking has been moved to its own function, I hope it functions the same as before. - CodeChange: Added symbolic types of PlayerID, OrderID and EngineID. For engines also added GetEngine() and IsEngineIndex(), similar to the other such functions. - CodeChange: To correctly build industries, some tables have been moved to build_industry.h. The only way to find out currently if an industry is valid in a climate is by looping all industries and checking if it matches. Also to comply with the patch setting build_rawmaterial_industries, it is assumed that these industries do not accept any cargo of any type. This can and probably should changed in the future to some flag in their struct. Also use _opt_ptr instead of _opt. - CodeChange: implemented the HQ checking code inspired by MarkR2 in "[ 1190944 ] Many commands not checked for security". Unfortunately it is impossible to prevent only deleting a HQ by a modified client atm. - CodeChange: For insert order and modify order their parameters are implicitely truncated to 8 bits, instead of the 16 bits said in the comments. --- vehicle.c | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) (limited to 'vehicle.c') diff --git a/vehicle.c b/vehicle.c index 07437ce11..91d76ce44 100644 --- a/vehicle.c +++ b/vehicle.c @@ -1621,7 +1621,11 @@ void MaybeReplaceVehicle(Vehicle *v) _current_player = OWNER_NONE; } - +/** Give a custom name to your vehicle + * @param x,y unused + * @param p1 vehicle ID to name + * @param p2 unused + */ int32 CmdNameVehicle(int x, int y, uint32 flags, uint32 p1, uint32 p2) { Vehicle *v; @@ -1631,12 +1635,10 @@ int32 CmdNameVehicle(int x, int y, uint32 flags, uint32 p1, uint32 p2) v = GetVehicle(p1); - if (!CheckOwnership(v->owner)) - return CMD_ERROR; + if (!CheckOwnership(v->owner)) return CMD_ERROR; str = AllocateNameUnique((const char*)_decode_parameters, 2); - if (str == 0) - return CMD_ERROR; + if (str == 0) return CMD_ERROR; if (flags & DC_EXEC) { StringID old_str = v->string_id; -- cgit v1.2.3-70-g09d2