Segfault in Frontend master: Previously Recorded

[jpilk]

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.

[jpilk]

It works when I use the Escape key. Perhaps the "newness" was just using arrow (or Enter?) instead.

[David Hampton]

This bug was introduced between commit a29633136d (Jun 17 2017) and a1a5064ab4 (Dec 31 2017).

[jpilk]

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.

[David Hampton]

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.

[Peter Bennett]

This fixes two tickets #13397 & #13421

[Peter Bennett]

This works but maybe I should change it to check the array length rather.

[Peter Bennett <pbennett@…>]

In c08b7ae0e/mythtv:

Error: Processor CommitTicketReference failed

GIT backend not available

[Peter Bennett <pbennett@…>]

In 042c180902/mythtv:

Error: Processor CommitTicketReference failed

GIT backend not available