- Status Closed
- Percent Complete
- Task Type Bugs
- Category User Interface → Simulator
- Assigned To No-one
- Operating System Sansa e200
- Severity Low
- Priority Very Low
- Reported Version Daily build (which?)
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#12234 - Simulator crashes when playing mp3 file with cuesheet
Playing an mp3 file with cuesheet causes the simulator (linux and w32) to crash. The same setup on the actual device works properly.
The position of the mp3 file in the playlist can change the results (i.e. proper operation).
The last working revision is r29784 and the first failing rev (that builds) is r29789
Linux sim r30322 fails with the following gdb message:
Program received signal SIGSEGV, Segmentation fault.
cuesheet_subtrack_changed (id3=0x81a3790)
at /home/mc2739/rockbox/apps/cuesheet.c:377
377 || (cue→curr_track_idx < cue→track_count - 1
Closed by MikeS
2011-08-22 00:18
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
2011-08-22 00:18
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
r30337 - OP reports the patch
effective
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
if “The position of the mp3 file in the playlist can change the results (i.e. proper operation).” affects anything then you must give a better repro setup.
Test setup used r30322 with no modifications and no configuration changes with the exception of enabling cuesheet support.
I have two mp3 files in separate folders. One of the mp3 files has a cuesheet that divides the song into 3 subtracks. There is no album art in the folders or id3 tags.
I have created two playlists. the first playlist has the mp3 with the cuesheet first, and the other playlist has it last.
When the first playlist is started, the cuesheet mp3 plays properly and subtracks are shown. Skipping to the next/previous subtrack or track also works properly. If repeat all is enabled, skipping forward from the second track to the first works properly.
To trigger the crash,
1. Start the sim - make sure cuesheet support is enabled
2. Start the second playlist - the first song should be playing
3. Skip forward to play the second song (with cuesheet) - if the first track plays through completely, when the second track (with cuesheet) starts, there is no crash.
4. The sim should crash
or,
1. Start the sim - make sure cuesheet support is enabled
2. Start the first playlist and skip to the second track
3. Exit the sim and restart - this is required to trigger the crash - if the first song has played, it will not crash
4. Enable Repeat All
5. Resume playback - the second song should be playing
6. Skip forward to play the first song (with cuesheet) - if you skip backward into the first song, it will not crash
7. The sim should crash
This sets the cuesheet pointer to NULL when it has to read directly from the buffer to do lookaheads for manual skips. id3→cuesheet is set during buffering so that it is known that a cuesheet exists for later on when the track becomes current. It wouldn’t be valid when looking ahead, which might bother the sim with memory protection but not the target. I hadn’t actually run this change myself.