Modify

Ticket #9602 (closed Bug Report - Crash: fixed)

Opened 2 years ago

Last modified 2 years ago

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

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

Change History

Changed 2 years ago by otto@…

Changed 2 years ago by otto@…

comment:1 follow-up: ↓ 3 Changed 2 years ago by beirdo

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

comment:2 Changed 2 years ago by beirdo

  • Owner set to beirdo
  • Status changed from new to assigned

Changed 2 years ago by otto@…

comment:3 in reply to: ↑ 1 Changed 2 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 2 years ago by beirdo

comment:4 follow-up: ↓ 6 Changed 2 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 2 years ago by beirdo

See also #9605. Same crash, different trigger.

comment:6 in reply to: ↑ 4 Changed 2 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 2 years ago by beirdo

  • Status changed from assigned to closed
  • Resolution set to Fixed
  • Milestone changed from unknown to 0.25

Committed to master in 6fe46da4735. Thanks.

comment:8 Changed 2 years ago by Gavin Hurlbut

  • Resolution changed from Fixed to fixed

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

View

Add a comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.