[vdr] Crash in cDvbSubtitleConverter::Action()

Klaus Schmidinger Klaus.Schmidinger at tvdr.de
Thu Aug 15 13:18:15 CEST 2013


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.



More information about the vdr mailing list