Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#8506 closed defect (fixed)

mythmusic: Segfault when exiting playlist editor

Reported by: loniedavid@… Owned by: Isaac Richards
Priority: major Milestone: 0.24
Component: Plugin - MythMusic Version: Master Head
Severity: low Keywords:
Cc: Ticket locked: no

Description

To reproduce: 1) Open mythmusic 2) Open "Edit Playlist" 3) Select some tracks 4) Hit escape ...segfault. Backtrace attached.

This is using current SVN trunk (r24896).

It appears to happen when prepping the visualization, and it still happens after disabling all visualizations.

I'm setting this as major since it makes mythmusic completely useless (can't load any tracks into playlist). Of course, feel free to change this if it is incorrect.

If more information/testing is needed let me know.

Attachments (2)

gdb.txt​ (8.3 KB ) - added by loniedavid@… 15 years ago.
gdb backtrace
disable_visuals.patch​ (1.2 KB ) - added by Matthew Wire <devel@…> 15 years ago.
Temporary workaround

Download all attachments as: .zip

Change History (10)

by loniedavid@…, 15 years ago

Attachment: gdb.txt​ added

gdb backtrace

comment:1 by robertm, 15 years ago

Component: MythTV - General → Plugin - MythMusic
Priority: major → trivial
Severity: high → low

Please read the ticket howto, priorities and severity are to be left alone. FWIW MythMusic is in the midst of a rewrite (though we appreciate the backtrace) so it is not likely to be stable during this development cycle.

comment:2 by JYA, 15 years ago

Priority: trivial → major

Have you performed a make distclean on both mythtv and mythplugins ? the libmyth API has changed a fair amount lately, and you have to make sure everything is properly recompiled.

Changing priority, as a crash is still a crash..

comment:3 by loniedavid@…, 15 years ago

I did a make clean distclean before grabbing the backtrace, so stale files shouldn't be an issue.

comment:4 by JYA, 15 years ago

I've identified the issue... This occurs when you haven't defined any visualizers.

Go in the Mytmusic preferences, in the player settings and enable some visualizers.. Will leave it as is for the time being as this is likely being rewritten

comment:5 by Matthew Wire <devel@…>, 15 years ago

I reported #8683 which (correctly) was marked as a duplicate of this bug.

Unfortunately it doesn't seem to matter if I have any visualizers defined or not - it still crashes. I've done some debugging and it seems to fail in the for loop of prepareVisuals() in output.cpp.

I'm attaching a patch which works around the issue for me and allows me to play CDs again - it just disables the calls to dispatchVisuals and prepareVisuals in audiooutputbase.cpp.

by Matthew Wire <devel@…>, 15 years ago

Attachment: disable_visuals.patch​ added

Temporary workaround

comment:6 by paulh, 15 years ago

(In [25551]) MythMusic: Tidy up the visualiser starting and stopping. Refs #8506, #8567.

comment:7 by paulh, 15 years ago

Resolution: → fixed
Status: new → closed

(In [25560]) MythMusic: Make sure the decoder is stopped when updating the playlist and fix a leak in MusicNode. Fixes #8506, #8567. Refs #8683.

comment:8 by stuartm, 15 years ago

Milestone: unknown → 0.24
Note: See TracTickets for help on using tickets.