Some maps containing 16 players from Stratagus 2.1 Pud

Bug #672845 reported by Zombie Ryushu on 2010-11-09
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Wargus
Fix Released
Undecided
Pali

Bug Description

I have some Puds containing 16 players that are causing segfaults in Stratagus. I will attach the offending map.

Related branches

Zombie Ryushu (zombie-ryushu) wrote :
Zombie Ryushu (zombie-ryushu) wrote :

Other half of map.

Zombie Ryushu (zombie-ryushu) wrote :

These are maps *I* made.

Pali (pali) wrote :

In your map is written: File generated automatically from pudconvert
So, can you attach original map too?

Zombie Ryushu (zombie-ryushu) wrote :

Yes I can. I have a few maps like this actually.

mattn (martin-gerhardy) wrote :

sorry for being off-topic - but if you have several self-made maps, why not releasing them under gpl and we add it to the wargus repository. it would be cool to have more multiplayer maps imo.

Pali (pali) wrote :

Problem is in pudconvert or original map. In converted sms map is line(s) CreateUnit() without unit name. So stratagus crashed.

unit = CreateUnit("", 0, {34, 55})

I try find out why pudconvert write unit without name...

Pali (pali) wrote :

Problem is that UnitScriptNames[] in pud.h is not complete. And contains empty strings.

Pali (pali) wrote :

In your map is only one type of unknow unit (but on more positions). I checked all positions in WAR2EDIT and on specified postions are no units! I dont know where is problem, but I think it is bug in WAR2EDIT or your map.

Unit type is 54 for player 0 (e.g. first player) on positions: [34, 55], [41, 56], [39, 56], [33, 56], [35, 56], [54, 57], [34, 57], [55, 58], [53, 58], [54, 59], [52, 60], [42, 62], [44, 62], [32, 63], [43, 64], [40, 64], [46, 64], [31, 64], [33, 64], [32, 65]

I patched pudconvert to ignore unknow units. Next I convert your map and chcecked original map in WAR2EDIT with Stratagus game. I cannot find differences.

Patch is in bzr now. Fixed pudconvert will be in new release.

Changed in wargus:
status: New → Confirmed
status: Confirmed → Triaged
assignee: nobody → Pali (pali)
status: Triaged → In Progress
status: In Progress → Fix Committed
Zombie Ryushu (zombie-ryushu) wrote :

I'd like to share a few points that were fixed in Stratagus 2.1 but broken in Stratagus 2.2. In Stratagus 2.1, a few changes were made to deal with some "special" units. One was the presence of "Attack Peasants." and I think those are your unknown Units. Attack Peasants were Peasants that behave like enemy grunts and attack other units.

Another thing was that there were support for 16 players (You can have two of each character color. e.g. Red Orcs, and Red Humans, Blue Orcs and Blue Humans.

Good for two purposes: 16 players per match, and support for Dual Race maps.

Pali (pali) on 2010-11-23
Changed in wargus:
milestone: none → 2.2.5.5
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers