- Status Closed
- Percent Complete
- Task Type Bugs
- Category User Interface → Themes
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Daily build (which?)
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#10453 - WPS bitmaps (backdrop) corrupted with recent build.
Using current build r21979 on a H300 (with default settings) i noticed some WPS corruption (see attached screendump).
This does not happen in the uisimulator.
With daily build r21977 this does not happen. So the recent cuesheet changes seem to cause this.
Closed by jdgordon
2009-07-20 15:50
Reason for closing: Fixed
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
2009-07-20 15:50
Reason for closing: Fixed
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
21982.
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
I think r21978 is the cause.
See it on my e200 too. Cuesheet support not enabled.
I suspect the function audio_finish_load_track in playback.c to be the cause. It puts a temporary struct cuesheet on the stack. That the struct is close to 50 kB large, and since the audio thread stack is 5 kB, there ought to be a pretty big stack overflow. Kind of surprised it doesn’t cause more problems. :)
The same thing here (Sansa e200).
I disabled the bit of code with the struct cuesheet and that removed the WPS corruption.
There IS a stack overflow when cuesheet support is enabled and the current or next song has a cuesheet.
untested fix… try it out and commit if it works, or ill test after work today