[skinnopacity] invalid lock sequence report ... mit vdr-2.4.0 und skinnopacity(gepatcht)

Sehr seltsam, das Ganze. Und Du bist Dir auch sicher, das Du das richtige vdr-Binary benutzt?

Was machst Du genau, wenn dieser Report kommt?

Normalerweise wird dieser Code ja nur benutzt bei der Kanalanzeige. Also beim Drücken der OK-Taste bei Live-TV, oder beim Kanalumschalten. Tritt es dann immer auf?

Ich habe diese Meldung nur ohne den Patch von Klaus. Mit Patch kann ich das nicht nachvollziehen.

Gruß

kamel5

Wenn ich mir den Backtrace hier so anschaue, dann fällt auf, dass da aus dem Konstruktor von cDisplayChannel ProcessKey() und damit Flush() aufgerufen wird, während der Lock auf die Channels gehalten wird.

Probier es mal bitte hiermit:

--- menu.c      2018-05-06 11:44:01.378852317 +0200
+++ menu.c      2018-05-23 23:35:48.140263770 +0200
@@ -4654,8 +4654,10 @@
   displayChannel = Skins.Current()->DisplayChannel(withInfo);
   positioner = NULL;
   channel = NULL;
-  LOCK_CHANNELS_READ;
-  channel = Channels->GetByNumber(cDevice::CurrentChannel());
+  {
+    LOCK_CHANNELS_READ;
+    channel = Channels->GetByNumber(cDevice::CurrentChannel());
+  }
   ProcessKey(FirstKey);
 }

Klaus

Hallo,

ich hatte mit dem patch hier noch das Problem, dass der vdr ab und zu segfaulted.

Das ganze liegt an der Variable following in displaychannel.h, die durch den Patch eingeführt word, diese wird allerdings nie initialisiert, das führt bei mir zu einem sporadischen segfault beim starten.

Wenn ich following = NULL; im constructor von cNopacityView einfüge ist das Problem behoben. Vielleicht hilft das noch jemand.

Marten

Quote

hattest du das Problem, wie hier geschildert mit dem Plugin epgsearch?

Nein, das war gleich beim Start des vdrs, das erste Channelinfo hatte keine gesetzten Informationen.....

Marten

Quote from MartenR

Wenn ich following = NULL; im constructor von cNopacityView einfüge ist das Problem behoben.

Das hatte ich wohl vergessen.

Ich habe die Initialisierung jetzt mal in cNopacityDisplayChannel untergebracht, wo auch present = NULL; steht.

Anbei ein aktueller Gesamtpatch bezogen auf den letzten Git-Stand. Alle anderen Patches bitte ignorieren.

Gruß

kamel5