[ANNOUNCE] VDR Extensions Patch v.54

Habe gerade nochmal in mein 1.5.11 geschaut. Da der 39er Patch ja einige Features mehr als der 37.1 hat, habe ich die Make.config nicht kopiert, sondern auf Basis der Make.config.template neu erstellt. Dabei konnte ich wohl der Versuchung nicht widerstehen ein bissl mehr auszuwählen, hier die Optionen im zzam Patch:

#CHANNELSCAN = 1
CMDSUBMENU = 1
#CUTTERLIMIT = 1
#CUTTERQUEUE = 1
CUTTIME = 1
DDEPGENTRY = 1
DOLBYINREC = 1
#DVBPLAYER = 1
#DVBSETUP = 1
#DVDARCHIVE = 1
#DVDCHAPJUMP = 1
#DVLFRIENDLYFNAMES = 1
#DVLRECSCRIPTADDON = 1
#DVLVIDPREFER = 1
#GRAPHTFT = 1
#HARDLINKCUTTER = 1
#IPTV = 1
JUMPPLAY = 1
LIEMIKUUTIO = 1
#LNBSHARE = 1
MAINMENUHOOKS = 1
#MENUORG = 1
NOEPG = 1
#OSDMAXITEMS = 1
#PINPLUGIN = 1
PLUGINAPI = 1
PLUGINMISSING = 1
#ROTOR = 1
SETTIME = 1
#SETUP = 1
#SOURCECAPS = 1
SORTRECORDS = 1
#SUBMENU = 1
#SYNCEARLY = 1
TIMERCMD = 1
TIMERINFO = 1
VALIDINPUT = 1
WAREAGLEICON = 1
#YAEPG = 1

Und da stürzt der VDR an dieser Stelle nicht ab.

Wenn die Kiste wieder frei ist, werde ich exakt diese Konfiguration mal testen.

Komisch, so ganz verstehe ich es nicht. Mein 1.5.11er + 37.1 ist definitiv mit NO_EPG kompiliert und da hatte ich den Absturz nie, oder wohl offensichtlich nie oft navigiert ???

Die gleichen Einstellungen beim 1.5.12er + 39 bringen (erwartungsgemäß) nix. Ich bastel mir jetzt einen VDR mit Deinem Fix. Sieht so aber auf den ersten Blick nach nem Speicherleck aus.

Hi!

Ich hab mit noepg mir auch mal angeschaut:
Resultat ist hier: noEPG: Aufhebung 999-Zeichen-limit

Schaun wir mal ob wareagle das fixt.

Zzam

Hi,

Ähm... Ich habe ein Prob mit dem Setup-Plugin:

setupmenu.h:58: error: 'cSubMenu' does not name a type
setupmenu.h:59: error: expected ';' before '(' token
make[1]: *** [setup.o] Fehler 1

warum?

Mfg SVen

Zitat

Original von s.krueger
Hi,

Ähm... Ich habe ein Prob mit dem Setup-Plugin:

Code

setupmenu.h:58: error: 'cSubMenu' does not name a type
setupmenu.h:59: error: expected ';' before '(' token
make[1]: *** [setup.o] Fehler 1

warum?

Mfg SVen

Alles anzeigen

Antwote mir mal selber, hatte MENUORG auskommentiert! Naja...

Mfg SVen

Hi,

ich kämpfe auch mit dem noEPG Problem und habe es mal näher angesehen. Die Ursache liegt in der dynamischen Verwaltung von noEPG. Auch in "Aufhebung 999-Zeichen-limit" ist es m.E. nicht gelöst.

Hier

cMenuSetupBase::cMenuSetupBase(void)
{
  data = Setup;
}

nämlich genau beim jedem Aufruf des Setup Menüs wird der 'default' Copy-Konstruktor der cSetup Klasse verwendet, dabei wird eine Kopie der Klasse angelegt. Auch der char* noEPG wird kopiert, nicht jedoch der Speicher auf den er zeigt. Somit hat die Kopie nun eine Referenz auf den Speicher der anderen Instanz. Dieser wird dann im Destruktor frei gegeben und der Zeiger der anderen Instanz zeigt in die Pampa, was dann früher oder später einen Crash zur Folge hat. Daher hilft wie von zulu vorgeschlagen das auskommentieren des free(noEPG) im Destruktor, das gibt zwar ein memleak aber der Speicher bleibt erst mal heile.

Aus meiner Sicht gibt es folgende Lösungen:

Lösung 1: Einen eigenen Copy Konstruktor schreiben welcher den noEPG String kopiert. Dieser Konstruktor müsste dann jedoch alle Member der Klasse umkopieren und jeder neue Member welcher in einer neuen VDR Version hinzukommt müsste wiederum berücksichtigt werden.

Lösung 2: Die Variable noEpg außerhalb der Klasse verwalten (als static deklarieren sollte ggf. auch gehen)

Lösung 3: Die einfachste, noEPG wieder als 'char array' anlegen

Lösung 4: So lassen, den Default Copy Konstruktor aufrufen, und anschließend (ist glaube ich nur die eine Stelle oben) eine Methode von cSetup (ala detachAndInitNoEpg(const char* noEPG)) aufrufen.

Ich tendiere zu 4 ist etwas dirty aber schnell eingebaut uns noEPG beleibt damit dynamisch.

/Edit

Ich habe es mir nochmal angesehen, von dem "Aufhebung 999-Zeichen-limit" gibt es wie ich nun erkannt habe eine zweite Version, in dieser fehlt der free(), daher auch kein Crash, ist aber immer noch gefährlich und ein potenzieller Crash Kandidat da sich die Instanzen der Klasse den Speicherbereich teilen.

Auch hier kann es m.E. krachen:

noEPG=new char[0];
strcpy(noEPG, "");

es werden null Byte Speicher reserviert und ein Byte (die terminierende Null) reinkopiert.

könnte man gegen:

noEpg=strdup("");

ersetzen.

horchi

Hi Zulu!

Ich hab noch Vorschläge den extensions-patch besser zu machen:
1. diff -ruNp verwenden. Dann sieht man in welcher Funktion die Änderungen sind.
2. Eventuell die ifdef-Teile aus Make.config.template in eine andere Datei verschieben und mit include einbinden.

Hast du eigentlich Kontakt zum liemekuutio Autor aufgenommen?

Zzam

Hi Marc,

Deine x-vdr.sh ist ja ein wahnsinn .... Bravo....
Auf die Neue sidux 2007-04 lite von heute Morgen. Er hat mir xine und den VDR zusammen gebaut ohne eingriffe.

Die Nova-S hat er schon ich würde dann gern nochmals das ganze am Schirm.

Da weiß ich nicht mehr wie ich es vor ein Paar Monate auf der Alten Sidux gemacht hatte ?

Danke nochmals.

Gruss Patrice

Super, Patch funktioniert super....

aber irgendwie bekomm ich das noepgmenu nun nicht mehr an's laufen:

g++ -g -O2 -Wall -Woverloaded-virtual -fPIC -c -DUSE_CMDSUBMENU -DUSE_CUTTERLIMIT -DUSE_CUTTERQUEUE -DUSE_CUTTIME -DUSE_DDEPGENTRY -DUSE_DOLBYINREC -DUSE_DVBPLAYER -DUSE_DVDARCHIVE -DUSE_DVLRECSCRIPTADDON -DUSE_DVLVIDPREFER -DUSE_DVLFRIENDLYFNAMES -DUSE_GRAPHTFT -DUSE_JUMPPLAY -DUSE_LIEMIKUUTIO -DUSE_LNBSHARE -DUSE_MAINMENUHOOKS -DUSE_NOEPG -DUSE_OSDMAXITEMS -DUSE_PINPLUGIN -DUSE_PLUGINMISSING -DUSE_ROTOR -DUSE_SETTIME -DUSE_SOURCECAPS -DUSE_SORTRECORDS -DUSE_SYNCEARLY -DUSE_TIMERCMD -DUSE_TIMERINFO -DUSE_TTXTSUBS -DUSE_VALIDINPUT -DUSE_VOLCTRL -DUSE_WAREAGLEICON -D_GNU_SOURCE -DPLUGIN_NAME_I18N='"noepgmenu"' -I../../../../DVB/include -I../../../include -I../../../../DVB/include noepgmenu.c
noepgmenu.c: In member function 'virtual void cMenuChannelItemNoEPG::Set()':
noepgmenu.c:165: error: 'class cChannels' has no member named 'keinEPGScan'
noepgmenu.c: In member function 'eOSState cMenuChannelsNoEPG::noEPGAdd()':
noepgmenu.c:230: error: 'class cSetup' has no member named 'noEPG'
noepgmenu.c:231: error: 'class cSetup' has no member named 'noEPG'
noepgmenu.c:236: error: 'class cSetup' has no member named 'noEPG'
noepgmenu.c:236: error: 'class cSetup' has no member named 'noEPG'
noepgmenu.c: In member function 'eOSState cMenuChannelsNoEPG::noEPGDel()':
noepgmenu.c:250: warning: deprecated conversion from string constant to 'char*'
noepgmenu.c:258: error: 'class cSetup' has no member named 'noEPG'
noepgmenu.c:260: error: 'class cSetup' has no member named 'noEPG'
noepgmenu.c:272: error: 'class cSetup' has no member named 'noEPG'
noepgmenu.c: In member function 'virtual eOSState cMenuChannelsNoEPG::ProcessKey(eKeys)':
noepgmenu.c:294: error: 'class cChannels' has no member named 'keinEPGScan'
make: *** [noepgmenu.o] Fehler 1

Hi,

im Extension Patch ist die aktuellste Version vom NoEpg Patch (von hier
noEPG: Aufhebung 999-Zeichen-limit)
enthalten, in diesen wurde noEPG in noEPGList umbenannt.

Torsten/WarEagle hat da eine Erweiterung mit White und Blacklist eingebaut. Ob es schon eine Anpassung des Plugins dazu gibt weis ich nicht.

/Edit
hab mal unter obigem Link nachgesehen, und ja es gibt dort auch einen Patch fürs Plugin.

Grüße
horchi

Mein VDR beendet sich nach ein paar Sekunden kommentarlos. Ich hab die
1.4.7 und die 1.5.12 probier, auch mal ohne Plugins. Ein ungepatchter VDR
läuft durch. Ich hab noch nicht herausgefunden, an welchem es liegt.
Das Problem trat auch mit der Version 39 auf.

CHANNELSCAN = 1
CMDSUBMENU = 1
CUTTERLIMIT = 1
CUTTERQUEUE = 1
CUTTIME = 1
DDEPGENTRY = 1
DOLBYINREC = 1
DVBPLAYER = 1
DVBSETUP = 1
DVDARCHIVE = 1
DVDCHAPJUMP = 1
#DVLFRIENDLYFNAMES = 1
#DVLRECSCRIPTADDON = 1
#DVLVIDPREFER = 1
#GRAPHTFT = 1
HARDLINKCUTTER = 1
IPTV = 1
JUMPPLAY = 1
LIEMIKUUTIO = 1
#LIVEBUFFER = 1
#LNBSHARE = 1
MAINMENUHOOKS = 1
MENUORG = 1
#NOEPG = 1
OSDMAXITEMS = 1
#PINPLUGIN = 1
PLUGINAPI = 1
PLUGINMISSING = 1
#ROTOR = 1
SETTIME = 1
#SETUP = 1
#SOURCECAPS = 1
SORTRECORDS = 1
#SUBMENU = 1
SYNCEARLY = 1
TIMERCMD = 1
TIMERINFO = 1
TTXTSUBS = 1
VALIDINPUT = 1
VOLCTRL = 1
WAREAGLEICON = 1
#YAEPG = 1

Zitat

LIEMIKUUTIO: VDR crashes while selecting the OSD-Setup for conflictcheckonly, epgsearchonly or quickepgsearch

I haven't noticed these kind of crashes on my setup and just verified the functionality with vdr-1.5.2 + liemikuutio-1.16 + conflictcheckonly-0.9.24.beta13.

Ohne LIEMIKUUTIO crasht er aber auch.