Opened 3 years ago
Closed 3 years ago
Last modified 3 years ago
[patch] corrupted read buffer refs #9511
|Reported by:||Jiri Fojtasek <jiri.fojtasek@…>||Owned by:||janne|
|Component:||MythTV - Video Playback||Version:||Master Head|
Replying to markk:
Jiri - I'm closing this ticket as fixed, though recognise that there are probably outstanding issues.
If you are still seeing issues with livetv hitting the end of the file during normal playback, please open another ticket as that is a separate issue that needs a proper root cause analysis.
Thanks for the time and effort taken in debugging the issue and producing the patches.
c3e1879e0c284b3c2267 introduced new problem and its stream corruption when you hit pause/play few times (sometimes leading in to playback crash), or during program switch/jump.
Your solution is wrong because when the buffer is paused and decoder trying to read from ringbuffer ReadDirect? is used instead and reading data from physical file without readahead buffer and read position adjustment. This will cause readahead buffer corruption when playback is resumed.
Attached patch reverting c3e1879e0c284b3c2267 and fixing wait condition by another way than in my previous patch.
Change History (5)
Changed 3 years ago by Jiri Fojtasek <jiri.fojtasek@…>
comment:1 Changed 3 years ago by stuartm
- Cc stuartm added
- Milestone changed from unknown to 0.25
comment:2 Changed 3 years ago by markk
comment:3 Changed 3 years ago by markk
- Resolution set to Fixed
- Status changed from new to closed
comment:4 Changed 3 years ago by Mark Kendall
- Resolution changed from Fixed to fixed