Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#9602 closed Bug Report - Crash (fixed)

Occational fe crash on playback startup / MythSystem?

Reported by: otto@… Owned by: beirdo
Priority: minor Milestone: 0.25
Component: MythTV - General Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Frontend occationally crashes when playback is started. See attached backtrace which suggest that this might be MythSystem? related. Didn't have time to test/report this on ticket #9421.

As shown on version output, local version is "dirty" but those modifications should have nothing to do with this.

Attachments (4)

frontend-version.txt (669 bytes) - added by otto@… 9 years ago.
playback-startup-bt.txt (56.3 KB) - added by otto@… 9 years ago.
fe-log.txt (31.3 KB) - added by otto@… 9 years ago.
0001-Try-to-keep-from-another-apparent-race-condition.patch (1.7 KB) - added by beirdo 9 years ago.

Download all attachments as: .zip

Change History (12)

Changed 9 years ago by otto@…

Attachment: frontend-version.txt added

Changed 9 years ago by otto@…

Attachment: playback-startup-bt.txt added

comment:1 Changed 9 years ago by beirdo

I also need the log output from the frontend, running -v system,extra

comment:2 Changed 9 years ago by beirdo

Owner: set to beirdo
Status: newassigned

Changed 9 years ago by otto@…

Attachment: fe-log.txt added

comment:3 in reply to:  1 Changed 9 years ago by otto@…

Replying to beirdo:

I also need the log output from the frontend, running -v system,extra

Attached the requested log (not from the same crash as the backtrace). Started frontend, started playback of recording twice and the second one crashed.

Changed 9 years ago by beirdo

comment:4 Changed 9 years ago by beirdo

OK, give this patch a try if you could, please. You seem to do quite well at finding the little tiny edge cases that I just can't get to fail here. :)

comment:5 Changed 9 years ago by beirdo

See also #9605. Same crash, different trigger.

comment:6 in reply to:  4 Changed 9 years ago by otto@…

Replying to beirdo:

OK, give this patch a try if you could, please. You seem to do quite well at finding the little tiny edge cases that I just can't get to fail here. :)

Patch seems to fix the issue, no crashes with it :)

comment:7 Changed 9 years ago by beirdo

Milestone: unknown0.25
Resolution: Fixed
Status: assignedclosed

Committed to master in 6fe46da4735. Thanks.

comment:8 Changed 9 years ago by Gavin Hurlbut

Resolution: Fixedfixed

Try to keep from another apparent race condition

Not quite sure why the QMap is going crazy, but let's try reading the AutoCleanup? flag before unlocking so we can mitigate the race some. Also, in both myth_system() and in the signal handler, use deleteLater() so we don't try to double-delete, as that will just cause other messes.

Fixes #9602

Changeset: 6fe46da473563a3586ca9ead45870734abdc0fbe

Note: See TracTickets for help on using tickets.