FS#7933 - fireworks.c: Duplicate element in autofire_delay_settings

Attached to Project: Rockbox
Opened by Mario Lang (mlang) - Thursday, 11 October 2007, 16:05 GMT
Last edited by Peter D'Hoye (petur) - Thursday, 11 October 2007, 19:33 GMT
Task Type Patches
Category Plugins
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No


The autofire_delay_settings array contains a duplicate element (300ms).
Since autofire_delay_values has only 15 elements, but 16 is assumed due to the duplicate element in ..._settings, we have a possible out-of-bounds access as well.
Remove the duplicate element and change 16 to 15 all over.
This task depends upon

Closed by  Peter D'Hoye (petur)
Thursday, 11 October 2007, 19:33 GMT
Reason for closing:  Accepted
Additional comments about closing:  thanks for spotting