Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes 32 Seg Faults with LiveTV on some platforms #472

Closed
jksj461 opened this issue Feb 8, 2022 · 4 comments
Closed

Fixes 32 Seg Faults with LiveTV on some platforms #472

jksj461 opened this issue Feb 8, 2022 · 4 comments
Assignees
Labels
Milestone

Comments

@jksj461
Copy link

jksj461 commented Feb 8, 2022

LiveTV-SegFault.txt
The file contains the gdb output for a segfault that occurs on any attempt to access live tv on a remote Intel Desktop.
Note LiveTV is working on Android 32 (Nvidia Shield & Fire Stick Max) also on a slow Intel Notebook.
It could be noteworthy that the log shows two files being accessed :-
'myth://tv/20681_20220208170148.ts
myth://tv/20681_20220208170150.ts
normally the target file only contains 2 secs of data but with the frontend running under the debugger the backend continues to write valid data .
The file 20681_20220208171500.ts does contain a valid recording

@bennettpeter
Copy link
Member

I tried to recreate this but I am unable to. Did you start Live TV from the LiveTV menu or from the program guide? I tried both and it works for me. If you tried from the menu please try also from the program guide and see if that makes a difference.

@jksj461
Copy link
Author

jksj461 commented Feb 9, 2022

High it segfaults instantly with live tv accessed by either method. The same build mythtv-light_32~Pre-3519-g0caf3ac0ec-0_amd64_focal.deb runs happily on an Intel Notebook. Which does suggest that there is a bad runtime link of some kind on this machine, if the multi file creates mentioned above is a red herring.
I installed the full debug version from the Ubuntu ppa to compare. No change in behaviour but the trace may offer more clues.
The line

0x00007ffff793c0ba in StreamHasRequiredParameters (Stream=0x555556379680, Context=0x555556468540) at decoders/avformatdecoder.cpp:238
238 decoders/avformatdecoder.cpp: No such file or directory.
May be significant

LiveTV-SegFaultPPA.txt

@paul-h
Copy link
Contributor

paul-h commented Feb 9, 2022

@bennettpeter This appears to be a result of the change to support ffmpeg 4.4.2 from 3 months ago

It looks like internal may be NULL or invalid here? https://github.com/MythTV/mythtv/blame/master/mythtv/libs/libmythtv/decoders/avformatdecoder.cpp#L238

@stuarta stuarta added the version:v32-fixes fixes/32 label Feb 9, 2022
@stuarta stuarta added this to the 32.0 milestone Feb 9, 2022
@bennettpeter
Copy link
Member

There was a change from Stream->nb_decoded_frames to Stream->internal->nb_decoded_frames due to ffmpeg moving some fields to internal. If internal null is indeed causing the bug then we should comment that check as has been done to the ones above it.

@bennettpeter bennettpeter self-assigned this Feb 9, 2022
bennettpeter added a commit that referenced this issue Feb 9, 2022
Remove unnecessary audio check which was being done before audio is
completely initialized and was checking for DTS audio which would be unlikley
in a Live TV broadcast.

Fixes #472

(cherry picked from commit e25055c)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants