[Patch] fix segfault in audioplayer.cpp / pictureviewer.cpp

Das Original Benutzerinterface Neutrino-SD incl. zapit, sectionsd, yWeb etc...
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

[Patch] fix segfault in audioplayer.cpp / pictureviewer.cpp

Beitrag von GetAway »

fix segfault by left button if selected item on first side > 1
fix: now also left button "page up" if more than 2 sides
fix: now left button change to last side if "page up" on first side
fix: now also right button chnage to first page if selected "last item" and "page down"
fix turning over last page in playlist when last page has few entries
fix some whitespaces

Edit: One more fix:
if last page has full entries, there was no change to 1. page
TODO: Check this Patch for same bug.
http://cvs.tuxbox-cvs.sourceforge.net/l ... 00002.html

Patch aktualisiert:
fix_segfault_audioplayer.diff
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: [Patch] fix segfault in audioplayer.cpp / pictureviewer.cpp

Beitrag von GetAway »

fix segfault when playlist is empty and num 0 is pressed.
fix turning over last page in playlist when last page has few entries.
replace pagebreak in helpscreen, because of the "nice look" on last page.
add some code that ensures that code will exec only when the playlist is filled.
fix some whitespaces and unnecessary braces

fix_segfault_pictureviewer.diff
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: [Patch] fix segfault in audioplayer.cpp / pictureviewer.cpp

Beitrag von rhabarber1848 »

GetAway hat geschrieben:fix segfault by left button if selected item on first side > 1
Ich kann diesen segfault bestätigen, darüberhinaus passiert folgendes:
zapit ist im standby, Neutrino wird nach dem segfault neugestartet,
zapit bleibt dabei im standby:
Segmentation fault
Neutrino exited with nonzero exit status, restarting...
starting /bin/neutrino
[neutrino] frameBuffer Instance created
812k video mem
[...]
[zapit] cmd 24 refused in standby mode
[CBasicClient] receive failed: /tmp/zapit.sock
[neutrino] Loading of scan settings failed. Using defaults.
Vorschlag: neutrino_restart.diff

EDIT: Hier geht es zum Thema weiter:
http://forum.tuxbox-cvs.sourceforge.net ... 51&start=0
Zuletzt geändert von rhabarber1848 am Donnerstag 25. Juni 2009, 15:09, insgesamt 1-mal geändert.
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: [Patch] fix segfault in audioplayer.cpp / pictureviewer.cpp

Beitrag von GetAway »

Sectionsd wird nicht restartet, sondern stürtzt ab und wird neu gestartet.
Dadurch gibt es kein start mit "pause".

Mit dem zapit ist glaube ich keine gute Idee. Es wäre überhaupt kein Standby mehr möglich.
Edit: mit dem Standby ist natürlich quatsch.
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: [Patch] fix segfault in audioplayer.cpp / pictureviewer.cpp

Beitrag von rhabarber1848 »

GetAway hat geschrieben:Sectionsd wird nicht restartet, sondern stürtzt ab und wird neu gestartet.
Hier nicht. Mein Patch funktioniert, obwohl die Aktivierung
von zapit und sectionsd natürlich auch in Neutrino selbst
stattfinden kann. Ich brauchte aber erstmal eine schnelle
lokale Lösung, um die von Dir berichteten segfaults zu
verifizieren.
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: [Patch] fix segfault in audioplayer.cpp / pictureviewer.cpp

Beitrag von rhabarber1848 »

GetAway hat geschrieben:Patch aktualisiert:
fix_segfault_audioplayer.diff
Patch committed: http://article.gmane.org/gmane.comp.vid ... ox.scm/723

Danke für die tolle Arbeit, den segfault hatte ich in der
Vergangenheit auch schon, vom merkwürdigen Verhalten
der Dateiliste ganz zu schweigen. Das war übrigens ein
Grund, warum ich den Esound-Support eingebaut habe ;)
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: [Patch] fix segfault in audioplayer.cpp / pictureviewer.cpp

Beitrag von rhabarber1848 »

GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: [Patch] fix segfault in audioplayer.cpp / pictureviewer.cpp

Beitrag von GetAway »

GetAway hat geschrieben: if last page has full entries, there was no change to 1. page
TODO: Check this Patch for same bug.
http://cvs.tuxbox-cvs.sourceforge.net/l ... 00002.html
Fixt das in channellist.cpp, eventlist.cpp und bouquetlist.cpp. Bitte Testen.
fix_pageup_lastpage_full_entries.diff
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: [Patch] fix segfault in audioplayer.cpp / pictureviewer.cpp

Beitrag von rhabarber1848 »

GetAway hat geschrieben:Fixt das in channellist.cpp, eventlist.cpp und bouquetlist.cpp. Bitte Testen.
[url=http://ulc.tuxbox-cvs.sourceforge.netin ... tory=Diffs]fix_pageup_lastpage_full_entries.diff[/url
Positiv getestet und committed