Opened 23 months ago
Closed 23 months ago
Last modified 22 months ago
#13421 closed Bug Report - General (fixed)
Segfault in Frontend master: Previously Recorded
Reported by: | jpilk | Owned by: | Peter Bennett |
---|---|---|---|
Priority: | minor | Milestone: | 30.1 |
Component: | MythTV - General | Version: | v30-fixes |
Severity: | medium | Keywords: | |
Cc: | Ticket locked: | no |
Description
I have what feels like a new segfault in frontend master, reproducible in F28, el7 and Firestick 4k. F28 says it has created a core dump, el7 reports and exits quickly, Firestick (20190224) needs a device restart.
Code 1, PID 8, UID 0, Value 0 in F28, 0xXXXXXX in el7
The steps to reproduce are ones that I have used quite often in el7, rarely in F28 and only in today's test on the Firestick.
Manage Recordings > Schedule Recordings > Programme Guide > highlight a recording that has not been previously recorded.
E > Schedule Info > Previously Recorded
No recording found, press right or left arrow
'Loading' popup appears and segfault follows. If recordings were identified, it lists them, and will continue on command.
In F28 a file appears briefly in /tmp. I've looked at the Debug wiki but hope the steps above might point to the cause of the problem.
Attachments (1)
Change History (9)
comment:1 Changed 23 months ago by
comment:2 Changed 23 months ago by
This bug was introduced between commit a29633136d (Jun 17 2017) and a1a5064ab4 (Dec 31 2017).
comment:3 Changed 23 months ago by
The whole "previously recorded" code was revised in that time frame. Ticket #13135 etc
https://code.mythtv.org/trac/ticket/13135#comment:32
is when the popup was added. But it seems unlikely that I haven't hit the arrow key instead of Escape since then.
comment:4 Changed 23 months ago by
Git bisect says that's the one. Commit 61c6429d7b. I can recreate it there, but not in the previous commit.
Gdb says the code is trying to access the first item in an empty array.
Changed 23 months ago by
Attachment: | 20190306_1851_tickets_13397_13421.patch added |
---|
comment:5 Changed 23 months ago by
This works but maybe I should change it to check the array length rather.
comment:6 Changed 23 months ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
In c08b7ae0e/mythtv:
comment:8 Changed 22 months ago by
Milestone: | needs_triage → 30.1 |
---|
It works when I use the Escape key. Perhaps the "newness" was just using arrow (or Enter?) instead.