empire/empserver
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Oops when unit_carrier_change() arguments OLD or NEW are invalid

Browse source
This commit is contained in:
Markus Armbruster 2008-10-18 14:21:45 -04:00
parent 488b80841b
commit c0c416b011
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
src/lib/common/cargo.c
View file

@ -186,9 +186,9 @@ unit_carrier_change(struct empobj *cargo, int type, int old, int new)
{ {
if (CANT_HAPPEN(type < 0 || type > EF_NUKE)) if (CANT_HAPPEN(type < 0 || type > EF_NUKE))
return; return;
if (old >= 0) if (old >= 0 && !CANT_HAPPEN(old >= nclink[type]))
clink_rem(&clink[type][old], cargo->ef_type, cargo->uid); clink_rem(&clink[type][old], cargo->ef_type, cargo->uid);
if (new >= 0) if (new >= 0 && !CANT_HAPPEN(new >= nclink[type]))
clink_add(&clink[type][new], cargo->ef_type, cargo->uid); clink_add(&clink[type][new], cargo->ef_type, cargo->uid);
} }