Author: sveinung
Date: Thu Oct 8 17:42:16 2015
New Revision: 30027
URL: http://svn.gna.org/viewcvs/freeciv?rev=30027&view=rev
Log:
Don't dir2char() invalid directions
Use direction8_is_valid() in case a non -1 invalid value somehow has ended
up in an unit order. This avoids triggering an assert while saving.
See patch #6394
Modified:
trunk/server/savegame3.c
Modified: trunk/server/savegame3.c
URL:
http://svn.gna.org/viewcvs/freeciv/trunk/server/savegame3.c?rev=30027&r1=30026&r2=30027&view=diff
==============================================================================
--- trunk/server/savegame3.c (original)
+++ trunk/server/savegame3.c Thu Oct 8 17:42:16 2015
@@ -5541,7 +5541,7 @@
case ORDER_PERFORM_ACTION:
action_buf[j] = num2char(punit->orders.list[j].action);
tgt_vec[j] = punit->orders.list[j].target;
- if (punit->orders.list[j].dir != -1) {
+ if (direction8_is_valid(punit->orders.list[j].dir)) {
/* The action target is on another tile. */
dir_buf[j] = dir2char(punit->orders.list[j].dir);
}
_______________________________________________
Freeciv-commits mailing list
Freeciv-commits@gna.org
https://mail.gna.org/listinfo/freeciv-commits
