View Issue Details

IDProjectCategoryView StatusLast Update
0002590FSSCPgameplaypublic2012-07-24 10:06
Reporterniffiwan Assigned Toniffiwan  
PrioritynormalSeveritycrashReproducibilityalways
Status resolvedResolutionfixed 
PlatformallOSallOS Versionall
Product Version3.6.14 RC6 
Target Version3.6.14Fixed in Version3.6.14 
Summary0002590: Starting Wings are allowed to have 6 ships which causes crashes when restarting the mission
DescriptionThe wings in $Starting Wings: are allowed to have 6 ships. However the ship & loadout selection screens assume a maximum of 4 ships per wing. This causes issues with the loadout screen itself, as well as causes the mission to crash on restart.

A simple option would be to not allow missions with more than MAX_WING_SLOTS (4) ships in starting wings to parse. A more complicated option (& a feature) would be to increase MAX_WING_SLOTS to 6.
Steps To ReproduceGet clippingtest.fs2 from the RC4 release thread.
http://www.hard-light.net/forums/index.php?topic=79747.msg1579245#msg1579245
(or see attached to this

Open the mission in the simulator and switch to the ship selection screen.
Note that "Gamma Wing" only has two ships and one is locked.

Start the mission, press escape and select "restart mission". FSO will crash.
Additional InformationWe should provide an updated mission in the RC4 thread before we stop this mission from parsing. Otherwise we're shooting ourselves in the foot regarding testing of the new sound code.
TagsNo tags attached.

Relationships

related to 0002653 resolvedkarajorma MAX_WING_SLOTS warning is triggering on a wing of 4 ships. 

Activities

niffiwan

2012-01-29 02:51

developer  

clippingtest.fs2 (17,351 bytes)

niffiwan

2012-02-01 09:22

developer   ~0013220

Fix committed to trunk@8413.

Zacam

2012-02-14 00:48

administrator   ~0013309

Fix committed to fs2_open_3_6_14@8484.

karajorma

2012-05-19 02:27

administrator   ~0013565

Fix committed to trunk@8795.

Zacam

2012-07-02 01:01

administrator   ~0013760

Fix committed to fs2_open_3_6_14@8936.

niffiwan

2012-07-05 13:07

developer   ~0013832

I think that karajorma's fix has re-introduced the original bug :( This is a reminder for me to review this...

niffiwan

2012-07-24 03:25

developer   ~0013882

derp - j will be out of bounds when slot is assigned after j has reached MAX_WING_SLOTS... needs another fix

niffiwan

2012-07-24 10:06

developer   ~0013885

Should be fixed in r9052

Related Changesets

fs2open: trunk r8413

2012-02-01 04:23

niffiwan


Ported: N/A

Details Diff
Fix for mantis 0002590: fix array overruns caused by starting wings have > MAX_WING_SLOTS ships Affected Issues
0002590
mod - /trunk/fs2_open/code/missionui/missionshipchoice.cpp Diff File
mod - /trunk/fs2_open/code/missionui/missionscreencommon.cpp Diff File

fs2open: fs2_open_3_6_14 r8484

2012-02-13 19:48

Zacam


Ported: N/A

Details Diff
Backport: Trunk r8413; Fix for mantis 0002590: fix array overruns caused by starting wings have > MAX_WING_SLOTS ships Affected Issues
0002590
mod - /branches/fs2_open_3_6_14/code/missionui/missionshipchoice.cpp Diff File
mod - /branches/fs2_open_3_6_14/code/missionui/missionscreencommon.cpp Diff File

fs2open: trunk r8795

2012-05-18 22:28

karajorma


Ported: N/A

Details Diff
Niffiwan's fix for 2590 seems to have introduced a bug of it's own. Easy enough to fix it though. Affected Issues
0002590
mod - /trunk/fs2_open/code/missionui/missionshipchoice.cpp Diff File

fs2open: fs2_open_3_6_14 r8936

2012-07-01 21:02

Zacam


Ported: N/A

Details Diff
Backport: Trunk r8795; Niffiwan's fix for 2590 seems to have introduced a bug of it's own. Easy enough to fix it though. Affected Issues
0002590
mod - /branches/fs2_open_3_6_14/code/missionui/missionshipchoice.cpp Diff File

fs2open: trunk r8995

2012-07-06 21:39

niffiwan


Ported: N/A

Details Diff
Fix number 3 for mantis 0002590 - move array overflow detection to correct place in loops Affected Issues
0002590
mod - /trunk/fs2_open/code/missionui/missionscreencommon.cpp Diff File
mod - /trunk/fs2_open/code/missionui/missionshipchoice.cpp Diff File

fs2open: fs2_open_3_6_14 r9033

2012-07-23 21:37

Zacam


Ported: N/A

Details Diff
Backport: Trunk 8995; Fix number 3 for mantis 0002590 - move array overflow detection to correct place in loops Affected Issues
0002590
mod - /branches/fs2_open_3_6_14/code/missionui/missionscreencommon.cpp Diff File
mod - /branches/fs2_open_3_6_14/code/missionui/missionshipchoice.cpp Diff File

Issue History

Date Modified Username Field Change
2012-01-29 02:51 niffiwan New Issue
2012-01-29 02:51 niffiwan File Added: clippingtest.fs2
2012-01-29 02:52 niffiwan Assigned To => niffiwan
2012-01-29 02:52 niffiwan Status new => assigned
2012-01-29 04:41 niffiwan Description Updated
2012-02-01 09:22 niffiwan Changeset attached => fs2open trunk r8413
2012-02-01 09:22 niffiwan Note Added: 0013220
2012-02-01 09:22 niffiwan Status assigned => resolved
2012-02-01 09:22 niffiwan Resolution open => fixed
2012-02-14 00:48 Zacam Changeset attached => fs2open fs2_open_3_6_14 r8484
2012-02-14 00:48 Zacam Note Added: 0013309
2012-05-19 02:27 karajorma Changeset attached => fs2open trunk r8795
2012-05-19 02:27 karajorma Note Added: 0013565
2012-07-02 01:01 Zacam Changeset attached => fs2open fs2_open_3_6_14 r8936
2012-07-02 01:01 Zacam Note Added: 0013760
2012-07-05 13:07 niffiwan Note Added: 0013832
2012-07-05 13:07 niffiwan Status resolved => feedback
2012-07-05 13:07 niffiwan Resolution fixed => reopened
2012-07-05 13:07 niffiwan Relationship added related to 0002653
2012-07-05 13:07 niffiwan Status feedback => assigned
2012-07-07 01:37 niffiwan Changeset attached => fs2open trunk r8995
2012-07-07 01:39 niffiwan Product Version 3.6.14 RC4 => 3.6.14 RC6
2012-07-07 01:39 niffiwan Status assigned => resolved
2012-07-07 01:39 niffiwan Fixed in Version => 3.6.14
2012-07-07 01:39 niffiwan Resolution reopened => fixed
2012-07-24 01:32 Zacam Changeset attached => fs2open fs2_open_3_6_14 r9033
2012-07-24 03:25 niffiwan Note Added: 0013882
2012-07-24 03:25 niffiwan Status resolved => feedback
2012-07-24 03:25 niffiwan Resolution fixed => reopened
2012-07-24 10:06 niffiwan Note Added: 0013885
2012-07-24 10:06 niffiwan Status feedback => assigned
2012-07-24 10:06 niffiwan Status assigned => resolved
2012-07-24 10:06 niffiwan Resolution reopened => fixed