On 14.08.2013 22:14, Ville Skyttä wrote:
Hello,
I'm occasionally seeing crashes in cDvbSubtitleConverter::Action(),
example backtrace below. Unfortunately I don't have a reliable
reproducer nor am I even sure if this happens on VDR startup or
shutdown, but I think it's one of those, maybe the latter :)
Perhaps something like this would avoid it (guessing, haven't tried
anything yet):
- if (cDvbSubtitleBitmaps *sb = bitmaps->First()) {
+ if (cDevice::PrimaryDevice() && cDvbSubtitleBitmaps *sb =
bitmaps->First()) {
A cDvbSubtitleConverter is only created for replay on the primary
device. So when the primary device is destroyed when shutting down VDR,
its destructor first deletes the dvbSubtitleConverter:
cDevice::~cDevice()
{
Detach(player);
DetachAllReceivers();
delete liveSubtitle;
delete dvbSubtitleConverter;
}
The destructor of cDvbSubtitleConverter in turn does a Cancel(3), which
stops its Action() loop.
Are you getting any log messages that stopping the "subtitleConverter"
thread didn't work?
Besides, the additional check you suggested wouldn't make any difference,
because the static cDevice::primaryDevice pointer is never set back to NULL
at shutdown. Maybe we should add
if (this == primaryDevice)
primaryDevice = NULL;
at the end of cDevice::~cDevice(), but since shutting down the devices is pretty
much the last thing done when leaving VDR I don't think this is going to make much
of a difference.
Klaus
#0 0x0000000000000000 in ?? ()
No symbol table info available.
#1 0x00007fcebbe30636 in cDvbSubtitleConverter::Action
(this=0x7fcea0381cd0) at dvbsubtitle.c:935
STC = <optimized out>
Delta = <optimized out>
sb = 0x7fcea0333bd0
ThreadLock = {thread = 0x7fcea0381cd0, locked = true}
WaitMs = 100
LastSetupLevel = 0
Timeout = {begin = 15644639}
#2 0x00007fcebbea234f in cThread::StartThread (Thread=0x7fcea0381cd0)
at thread.c:262
No locals.
#3 0x00007fcebb6f0c53 in start_thread () from /lib64/libpthread.so.0
No symbol table info available.
#4 0x00007fceba11213d in clone () from /lib64/libc.so.6
No symbol table info available.
_______________________________________________
vdr mailing list
vdr@xxxxxxxxxxx
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
