vdr-plugin-dynamite: bug fix

MarkusE · 16. Februar 2023

Hallo,

In https://github.com/MarkusEh/vd…amite/tree/markus/patches ist vdr-2.6.3-dynamite.patch .

In diesem Patch ist ein Fehler korrigiert, der (vermutlich) in allen anderen Versionen des Patches ist.

Zum Update:

Den alten Patch entfernen, und mit vdr-2.6.3-dynamite.patch patchen. Dieser Patch sollte im Prinzip auch für vdr 2.4.x funktionieren.
Wer vorher vdr-2.4.6-dynamite.diff hatte, braucht die Plugins nach dem upgrade auf vdr-2.6.3-dynamite.patch nicht neu zu übersetzen, da sich das VDR Interface dadurch nicht ändert.

Zum Fehler:

Der Fehler kann zu "video data stream broken" führen. Siehe z.B. Fehler bei: 1. Aufnahme auf ZDF HD, danach 2. Aufnahme auf rbb Berlin HD (während die 1. Aufnahme noch aufnimmt) .
Der Fehler sollte nur auftreten, wenn der Patch und das Plugin läuft. Wer also nur den Patch hat, ist nicht betroffen.

Warum ein so alter Fehler bisher nicht gefunden wurde?

Da kann ich nur spekulieren. Vermutlich, weil er nur selten auftritt. Und weil bei "video data stream broken" erstmal niemand den Fehler im dynamite Patch sucht.

~ Markus

mini73 · 17. Februar 2023

Rein aus Interesse: Magst du mir beschreiben, was du im Patch geändert hast?

MarkusE · 17. Februar 2023

Alt (also mit dem "alten" Patch):

Code

void cDevice::DetachAllReceivers(void)                                                                                  {                                                                                                                                                                                         cMutexLock MutexLock(&mutexReceiver);                                                                                   for (int i = 0; i < MAXRECEIVERS; i++)
      Detach(receiver[i], false);
  ReleaseCamSlot();
}

Neu (also mit dem korrigierten Patch)

Code

void cDevice::DetachAllReceivers(void)                                                                                  {                                                                                                                         if (parentDevice)                                                                                                         return parentDevice->cDevice::DetachAllReceivers();                                                                   cMutexLock MutexLock(&mutexReceiver);                                                                                   for (int i = 0; i < MAXRECEIVERS; i++)
      Detach(receiver[i], false);
  ReleaseCamSlot();
}

SurfaceCleanerZ · 17. Februar 2023

Hi,

Hmm für mich sehen beide Einträge gleich aus.

MfG Stefan

MarkusE · 17. Februar 2023

Code

if (parentDevice)                                                                                                         return parentDevice->cDevice::DetachAllReceivers();

hat gefehlt.

MarkusE · 17. Februar 2023

Wer es testen / ausprobieren möchte:

Aufzeichnung auf sat1 starten, prio 50
Aufzeichnung auf tele5 starten, prio 51
Aufzeichnung auf ARD starten, prio 52
...

Weitermachen, bis kein Empfänger mehr frei ist. Also wer 2 Empfänger hat, startet 3 Aufzeichnungen.

Korrektes Verhalten:

VDR beendet ("stopped") die Aufzeichnung auf sat1.

Fehlerhaftes Verhalten:

VDR beendet ("stopped") die Aufzeichnung auf sat1 nicht, schaltet aber den Empfänger auf ARD. -> "video data stream broken"

MarkusE · 17. Februar 2023

Aus reiner Neugierde:

Der Fehler kam mit vdr-1.7.39-dynamite.patch .

In älteren Versionen des Patches (vdr-1.7.37-dynamite.patch und älter) war der noch nicht drin.

Da stand noch

Code

+  if (parentDevice)
+     return parentDevice->DetachAllReceivers();

. Das funktioniert aber mit der aktuellen Version des Plugins nicht mehr.

mini73 · 18. Februar 2023

Vielleicht hatte es einen Grund, dass das raus fiel, vielleicht war es aber auch einfach ein Fehler...

MarkusE · 18. Februar 2023

Hi,

> Vielleicht hatte es einen Grund, dass das raus fiel, vielleicht war es aber auch einfach ein Fehler...

Kurzfassung:

Ja, es gab einen Grund. Dieser ist mit vdr-2.6.3-dynamite.patch beseitigt

Für die Entwickler:

In neueren Versionen des dynamite plugins:

Code

void cDynamicDevice::DetachAllReceivers(void)                                                                                                 
{                                                                                                                                             
  if (subDevice)                                                                                                                              
     return subDevice->DetachAllReceivers();                                                                                                  
  cDevice::DetachAllReceivers();                                                                                                              
}

Mit vdr-1.7.37-dynamite.patch:

Code

void cDevice::DetachAllReceivers(void)
{
  if (parentDevice)
    return parentDevice->DetachAllReceivers();
...
}

Führt das zu einem "endlosen" gegenseitigen Aufruf. Bis der Stack voll ist, und der Kernel VDR beendet.

Das Entfernen von "if (parentDevice) DetachAllReceivers();" in "cDevice::DetachAllReceivers" verhindert zwar den "endlosen" gegenseitigen Aufruf . Führt aber dazu, dass die Receiver nicht mehr gelöscht werden . Mit den oben beschriebenen Folgen.

Deshalb steht in vdr-2.6.3-dynamite.patch auch:

Code

void cDevice::DetachAllReceivers(void)
{
  if (parentDevice)
    return parentDevice->cDevice::DetachAllReceivers();
...
}

Damit wird der "endlose" gegenseitige Aufruf verhindert, und alle Receiver werden gelöscht .

~Markus

seahawk1986 · 18. Februar 2023

Ich habe den Patch mal in https://launchpad.net/~seahawk…archive/ubuntu/vdr-2.6.4/ eingebaut.

vdr-plugin-dynamite: bug fix

Jetzt mitmachen!

Teilen