game crashes/assertion failure

Bug #530767 reported by Victor Pelt
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
widelands
Fix Committed
Critical
Timowi

Bug Description

I was playing. then when attacking around an enemy's headquarter the game crashes. when i try to load the autosave it fails with an assertion failure

Tags: crash military
Revision history for this message
Victor Pelt (victor-pelt) wrote :

savegame that fails

Revision history for this message
Timowi (timo-wingender) wrote :

Crashes because of a flag on field with wrong owner. Probably because of a conquered building with not enough land.

Writing Flag Data ... widelands: /home/timo/widelands/widelands/src/map_io/widelands_map_flag_data_packet.cc:143: virtual void Widelands::Map_Flag_Data_Packet::Write(FileSystem&, Widelands::Editor_Game_Base&, Widelands::Map_Map_Object_Saver&): Assertion `field->get_owned_by() == flag->owner().player_number()' failed.

savegame was written with unofficial-bzr-trunk-4995

Changed in widelands:
status: New → Confirmed
Revision history for this message
Victor Pelt (victor-pelt) wrote :

yep i was conquering the HQ, means we should check for some other option of conquering might give bad results

SirVer (sirver)
tags: added: crash military
Revision history for this message
Sigra (sigra) wrote :
SirVer (sirver)
Changed in widelands:
importance: Undecided → Critical
milestone: none → build15-rc1
Revision history for this message
Nicolai Hähnle (nha) wrote :

Here's another savegame that fails with slightly different symptoms, but the underlying problem (incorrect assignment of field ownership) appears to be the same. Here's the error message we get:

flags: 2246 (at (27, 83), owned by player 1): neighbour node (27, 84): is owned by player 3

Revision history for this message
Sigra (sigra) wrote :

We do not need more savegames attached to this duplicate report of a well-understood bug. What we need is a rewrite of the map object loading code, making sure that it does not call any init() functions. (By the way, it is called *node* ownership, to be precise.)

Timowi (timo-wingender)
Changed in widelands:
assignee: nobody → Timowi (timo-wingender)
Timowi (timo-wingender)
Changed in widelands:
status: Confirmed → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.