2025-07-26 06:29 CEST

View Issue Details Jump to Notes ]
IDProjectCategoryView StatusLast Update
0000908Spring engineGeneralpublic2012-06-27 02:29
ReporterKDR_11k 
Assigned Toabma 
PrioritynormalSeveritymajorReproducibilityalways
StatusresolvedResolutionfixed 
Product Version 
Target VersionFixed in Version 
Summary0000908: Units return friendly fire
DescriptionA unit that is damaged by a weapon fired by an allied (same ally team but not same team) unit will target that unit and shoot at it. It's easiest to reproduce by using a mod with buildlasers (e.g. SimBase, Kernel Panic), putting two teams into one allyteam and driving one team's units through the other team's buildlasers.
Additional InformationThis is doubly weird as even if the targetting allows picking an ally the code should make the unit stop shooting at the next check (unless it's a usertarget).
TagsNo tags attached.
Checked infolog.txt for Errors
Attached Files

-Relationships
+Relationships

-Notes

~0002190

tvo (reporter)

One occurence was described here:

http://spring.clan-sy.com/phpbb/viewtopic.php?p=270586

~0002217

imbaczek (reporter)

Last edited: 2008-05-02 20:43

My simple tests show that gs->Ally doesn't work in CUnit::SetLastAttacker. I'm confused what gs->Ally should mean. Currently it is true only for a=b, at least that's what my gdb says.

edit: nvm, adding gs->AlliedTeams, this should fix it.

~0002218

Kloot (developer)

The check in SetLastAttacker should probably read

if (gs->Ally(team, attacker->team) || gs->Ally(attacker->team, team)) {
    return;
}

since the allies matrix is not symmetric.

~0002219

imbaczek (reporter)

did just that. r5824.
+Notes

-Issue History
Date Modified Username Field Change
2008-04-29 14:35 KDR_11k New Issue
2008-04-30 13:18 tvo Note Added: 0002190
2008-05-02 20:39 imbaczek Note Added: 0002217
2008-05-02 20:43 imbaczek Note Edited: 0002217
2008-05-02 20:47 Kloot Note Added: 0002218
2008-05-02 20:59 imbaczek Note Added: 0002219
2008-05-02 20:59 imbaczek Status new => resolved
2012-06-27 02:27 abma Status resolved => assigned
2012-06-27 02:27 abma Assigned To => abma
2012-06-27 02:29 abma Status assigned => resolved
2012-06-27 02:29 abma Resolution open => fixed
+Issue History