[testing yavdr] Andauernd Absturz des VDR beim Systemstart

Moin,

der Frank mit seinem Handy ist schuld

Spass bei Seite...das Fritzbox Plugin will eine Nachricht anzeigen, danach geht der Skin auf die Bretter. Was mich nur wundert: es wird cNopacityDisplayReplay::SetMessage() aufgerufen. Das wird aber eigentlich nur aufgerufen, wenn eine Aufnahme abgespielt wird. "Normalerweise" sollte cNopacityDisplayMessage::SetMessage() aufgerufen werden?!

Passiert das direkt beim Start des VDR?

Ciao Louis

PS: ist nOpacity im yaVDR testing ungepatcht? Oder ist ggf. das Unterdrücken der ersten Ausgabe beim Start herausgepatcht?

Ciao Louis

Zitat von louis

Oder ist ggf. das Unterdrücken der ersten Ausgabe beim Start herausgepatcht?

In den Paketen im yaVDR PPA aktuell nicht - so ein Patch war im April mal kurzzeitig im Paket in testing. Ich habe den aber wieder herausgenommen, weil es (wie zu erwarten) Probleme gab.

Moin,

irgendwie habe ich heute morgen das falsche Crashlog angeschaut...war wohl noch zu früh Was ich bisher geschrieben habe war Blödsinn, in dem Crashlog das ich angesehen habe, war das Fritzbox Plugin involviert, und es hat ein Frank angerufen

Mit dem Crashlog ist klar, warum es knallt. Da fehlt in cNopacityDisplayChannel::SetMessage die Abfrage, ob es sich um die erste Ausgabe handelt. Füge einfach in der Zeile 133 der displaychannel.c ganz am Anfang der Funktion SetMessage folgendes hinzu:

if (!doOutput)
    return;

Dann ist der Crash weg.

Ciao Louis

Also so:

Index: vdr-plugin-skinnopacity-1.1.3.git20140524.1729/displaychannel.c
===================================================================
--- vdr-plugin-skinnopacity-1.1.3.git20140524.1729.orig/displaychannel.c        2014-05-24 19:29:36.000000000 +0200
+++ vdr-plugin-skinnopacity-1.1.3.git20140524.1729/displaychannel.c     2014-07-29 09:04:02.347931573 +0200
@@ -130,6 +130,8 @@
 
 
 void cNopacityDisplayChannel::SetMessage(eMessageType Type, const char *Text) {
+    if (!doOutput)
+        return;
     channelView->ClearChannelLogo();
     channelView->ClearChannelName();
     channelView->ClearEPGInfo();

Oder ist es evtl. besser das erst nach dem Aufräumen vor Zeile 140 zu machen? http://projects.vdr-developer.…ree/displaychannel.c#n140

Hi Seahawk,

Zitat von seahawk1986

Also so:

Diff

Index: vdr-plugin-skinnopacity-1.1.3.git20140524.1729/displaychannel.c
===================================================================
--- vdr-plugin-skinnopacity-1.1.3.git20140524.1729.orig/displaychannel.c        2014-05-24 19:29:36.000000000 +0200
+++ vdr-plugin-skinnopacity-1.1.3.git20140524.1729/displaychannel.c     2014-07-29 09:04:02.347931573 +0200
@@ -130,6 +130,8 @@
 
 
 void cNopacityDisplayChannel::SetMessage(eMessageType Type, const char *Text) {
+    if (!doOutput)
+        return;
     channelView->ClearChannelLogo();
     channelView->ClearChannelName();
     channelView->ClearEPGInfo();

Alles anzeigen

Genau so!

Zitat von seahawk1986

Oder ist es evtl. besser das erst nach dem Aufräumen vor Zeile 140 zu machen? http://projects.vdr-developer.…ree/displaychannel.c#n140

Nein, das channelView Objekt ist NULL, da es aufgrund des ersten Aufrufs gar nicht erzeugt worden ist. Deshalb crasht es ja

Ciao Louis

Es gibt ein neues Paket in testing mit dem Patch - bitte mal ausprobieren.

Das ist ja ein schneller Service

Ciao Louis

Aber?