[Freeciv-Dev] [bug #20700] assert (index) = 0 (index) MAP_INDEX_SIZE failed

2013-04-05 Thread Marko Lindqvist
Update of bug #20700 (project freeciv):

  Status:  Ready For Test = Fixed  
 Assigned to:None = cazfi  
 Open/Closed:Open = Closed 


___

Reply to this item at:

  http://gna.org/bugs/?20700

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #20700] assert (index) = 0 (index) MAP_INDEX_SIZE failed

2013-04-03 Thread Marko Lindqvist
Follow-up Comment #4, bug #20700 (project freeciv):

After total of about 45 hours of autogame execution I've both confirmed that
the issue was the one I noticed, and the already submitted patch seems to
work. (Change comes to play first time much earlier where even the old code
luckily didn't crash, changing how the autogame proceeds, so fix is not tested
against exact old crashing situation)

___

Reply to this item at:

  http://gna.org/bugs/?20700

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #20700] assert (index) = 0 (index) MAP_INDEX_SIZE failed

2013-04-01 Thread Marko Lindqvist
URL:
  http://gna.org/bugs/?20700

 Summary: assert (index) = 0  (index)  MAP_INDEX_SIZE
failed
 Project: Freeciv
Submitted by: cazfi
Submitted on: Tue 02 Apr 2013 02:05:14 AM EEST
Category: None
Severity: 3 - Normal
Priority: 5 - Normal
  Status: None
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Release: r22640
 Discussion Lock: Any
Operating System: None
 Planned Release: 

___

Details:

Attached autogame asserts (but only at turn 393 and it has a lot of players
for server to handle, so be prepared to long wait if you run the autogame)


 bt full
#0  0x7668defb in raise (sig=optimized out)
at ../nptl/sysdeps/unix/sysv/linux/pt-raise.c:42
No locals.
#1  0x00670b52 in fc_assert_fail (file=0x71e181 common/map.h, 
function=0x71eb00 index_to_map_pos_y, line=718, 
assertion=0x71e158 (index) = 0  (index)  MAP_INDEX_SIZE, 
message=0x74ef94 nologmsg:%s) at utility/log.c:520
level = LOG_FATAL
#2  0x0057befe in index_to_map_pos_y (index=78302080)
at common/map.h:718
map_x = 32767
map_y = -163131160
__FUNCTION__ = index_to_map_pos_y
#3  0x0057e695 in unit_tile_link (punit=0x4a38b60)
at common/featured_text.c:1126
buf = [l tgt=\tile\ x=88 y=47]AEGIS Cruiser[/l]\000\000\000],
'\000' repeats 82 times
tag_name = 0x71e512 l
#4  0x00497cab in sell_random_unit (pplayer=0x1755660, 
punitlist=0x493b490) at server/cityturn.c:1914
punit = 0x4a38b60
gold_upkeep = 2
r = 2
__FUNCTION__ = sell_random_unit
#5  0x0049801c in player_balance_treasury_units_and_buildings (
pplayer=0x1755660) at server/cityturn.c:1971
pimprlist = 0x481df30
punitlist = 0x493b490
sell_unit = true
__FUNCTION__ = player_balance_treasury_units_and_buildings
#6  0x004939c0 in update_city_activities (pplayer=0x1755660)
at server/cityturn.c:581
cities = 0x7fffdec8
i = 0
r = 0
buf = '\000' repeats 96 times,
PEe\001\000\000\000\000PEe\001\000\000\000\000PEe\001\000\000\000\000\330\336\377\377\377\177\000\000$,
'\000' repeats 11 times, %, '\000' repeats 11 times,
p\340\377\377\377\177\000\000\023\307d\000\000\000\000\000(\215\271\000\000\000\000\000\240\340\377\377\377\177\000\000\267\324d\000\000\000\000
n = 27
gold = 11
__FUNCTION__ = update_city_activities
#7  0x0043d24a in end_phase () at server/srv_main.c:1029
MY_i = 98
pplayer = 0x1755660
__FUNCTION__ = end_phase
#8  0x00440e64 in srv_running () at server/srv_main.c:2337
eot_timer = 0x1646be0
save_counter = 19
i = 0
is_new_turn = true
need_send_pending_events = false
__FUNCTION__ = srv_running
#9  0x004420ee in srv_main () at server/srv_main.c:2788
__FUNCTION__ = srv_main
#10 0x004366df in main (argc=6, argv=0x7fffe368)
at server/civserver.c:454
inx = 6
showhelp = false
showvers = false
option = 0xb968a0  \261\232
__FUNCTION__ = main




___

File Attachments:


---
Date: Tue 02 Apr 2013 02:05:14 AM EEST  Name: mapindex.serv  Size: 183B   By:
cazfi

http://gna.org/bugs/download.php?file_id=17651

___

Reply to this item at:

  http://gna.org/bugs/?20700

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #20700] assert (index) = 0 (index) MAP_INDEX_SIZE failed

2013-04-01 Thread Marko Lindqvist
Follow-up Comment #2, bug #20700 (project freeciv):

I don't know if it's the problem I'm seeing, but there's apparent problem with
the code when multiple units are being sold.

If there's both transport and its cargo in the list of potential units to
sell, selling transport first will cause, due to recursive wiping, cargo also
to get destroyed, but to remain in the list of potential units to sell.

___

Reply to this item at:

  http://gna.org/bugs/?20700

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #20700] assert (index) = 0 (index) MAP_INDEX_SIZE failed

2013-04-01 Thread Marko Lindqvist
Update of bug #20700 (project freeciv):

Category:None = general
  Status:None = Ready For Test 
 Planned Release: = 2.3.5, 2.4.0, 2.5.0

___

Follow-up Comment #3:

 If there's both transport and its cargo in the list of potential
 units to sell, selling transport first will cause, due to
 recursive wiping, cargo also to get destroyed, but to remain in
 the list of potential units to sell.

Untested patch for that one.

(file #17652, file #17653)
___

Additional Item Attachment:

File name: SellCargoBeforeTransport.patch Size:3 KB
File name: SellCargoBeforeTransport-S2_3.patch Size:3 KB


___

Reply to this item at:

  http://gna.org/bugs/?20700

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev