Segfault mit graphlcd-0.1.2-pre4 und vdr > 1.3.18

Zitat

Original von wofritz

ich habe Probleme mit graphlcd-0.1.2-pre4 und vdr >1.3.18. Wenn man das Abspielen einer Aufnahme oder die MP3-Wiedergabe mit der "Exit"-Taste bendet, kommt es ab und zu zu einem Segmentation fault.

Näher untersucht habe ich den Crash im Zusammenhang mit dem Beenden einer MP3-Wiedergabe. In vdr-Versionen ab 1.3.19 (alle getestet bis 1.3.22) gibt es einen Segmentation fault, mit folgendem Backtrace:

Ist das anderen hier auch schon aufgefallen?
Hat jemand einen Vorschlag, wie könnte man dem Problem noch beikommen könnte?

Gruss,

Wolfgang

Alles anzeigen

Dieser Absturz im zusammenhang mit mp3 lässt sich hier reproduzieren.
Allerdings normales Playback, Muggle, Image verhalten sind normal...

Scheinbar sind nochmehr Stellen durch die Umstellung des VDR auf Verwendung von Rückgabewerte die Threadsafe sind betroffen.

Siehe auch http://www.vdr-portal.de/board/thread.php?threadid=30870

Cu,
Andreas

Hi,
hmm, ueberraschenderweise hab ich das Problem mut dem GLCD-Plugin 0.1.1 nicht.
Wie sind denn die Einstellungen beim mp3-plugin, d.h. laeuft z.B. der Hintergrundscan, Pegelabgleich, ... ?
Es gab auch einen Bug (ebenfalls nicht threadsafe) beim mp3-plugin, wenn es z.B. die Message "connecting Streamserver ..." anzeigte (ist inzwischen behoben).
Gruss
Burkhardt

Hi,

ich habe die History des VDR als Stichwortgeber genommen :

2005-01-09: Version 1.3.18


- Several formerly non-threadsafe functions now have a return type of cString:
  cChannel::ToText()
  tChannelID::ToString()
  cEvent::GetDateString()
  cEvent::GetTimeString()
  cEvent::GetEndTimeString()
  cEvent::GetVpsString()
  cMark::ToText()
  cTimer::ToText()
  cSource::ToString()
  cTimer::PrintDay()
  cTimer::PrintFirstDay()
  PrefixVideoFileName()
  IndexToHMSF()
  ChannelString()
  strescape()
  AddDirectory()
  itoa()
  WeekDayName()
  DayDateTime()
  When using these functions in a 'const char *' context there is nothing special
  to consider, except that you can no longer have a pointer to the return value,
  as in
  const char *date = DayDateTime();
  Although this will compile without error message, the resulting 'date' will not
  be valid after this line. Use this instead:
  cString date = DayDateTime();
  In a 'const void *' context (as in printf() etc.) the result needs to be
  dereferenced with a '*', as in
  printf("%s", *DayDateTime());
  to make it a 'const char *'.

und dabei noch ein weitere Stelle idenifiziert :

--- display.c.orig	2005-01-06 16:11:32.000000000 +0100
+++ display.c	2005-03-06 08:52:33.000000000 +0100
@@ -507,7 +507,8 @@
 				break;
 		}
 #if VDRVERSNUM >= 10300
-		char * strId = strstr(ch->GetChannelID().ToString(), "-") + 1;
+		cString sid = ch->GetChannelID().ToString();
+		char * strId = strstr(sid, "-") + 1;
 		logo = logoList->GetLogo(strId, picType);
 #else
 		char strId[16];
@@ -743,7 +744,8 @@
 
 		strncpy(month, (char *)(tr("JanFebMarAprMayJunJulAugSepOctNovDec") + 3 * tm->tm_mon), 3);
 		month[3] = 0;
-		snprintf(buffer, sizeof(buffer), "%s %2d.%s  %d:%02d", WeekDayName(tm->tm_wday), tm->tm_mday, month, tm->tm_hour, tm->tm_min);
+		cString wdn = WeekDayName(tm->tm_wday);
+		snprintf(buffer, sizeof(buffer), "%s %2d.%s  %d:%02d", *wdn, tm->tm_mday, month, tm->tm_hour, tm->tm_min);
 		TextLen = normalFont.Width(buffer);
 
 		if (TextLen > std::max(FrameWidth - 2 * TEXT_OFFSET_X, 1))

Das Problem was ich habe, ist das dies ein hochgradiges Laufzeitproblem ist.
Mit dem Patch wird mir wenn der VDR im debugger läuft keine Segfault mehr geworfen, ohne schon. Allerdings wenn der VDR solo läuft kommt es trotzdem zum Segfault. Der per coredump angezeigte backtrace zeigt mir nur totalen Blödsinn an, das der Fehler vermutlich durch überschreiben des Stack zustande kam, allerdings ist Ankerpunkt immer cGraphLCDState::GetReplayState()

Cu,
Andreas

Hi,

jetzt wird es mystisch, ich habe noch eine Stelle geändert und ich kann keine "Standalone" Seqfaults mehr reproduzieren. Allerdings szCurrent&sztotal sind vom typ std::string, eigentlich Klassen mit eigener Speichermanagment, sprich sie sollen bei der Zuweisung den Inhalt herauskopieren, und damit eigentlich nicht ursache des Problems sein ...

@@ -1488,9 +1490,12 @@
 		if ((replay.total > 1 && IndexIsGreaterAsOneHour(replay.total)) ||
 		    IndexIsGreaterAsOneHour(replay.current)) // Check if any index bigger as one hour
 		{
-			szCurrent = IndexToHMSF(replay.current);
-			if (replay.total > 1) // Don't draw totaltime for endless streams  
-				szTotal = IndexToHMSF(replay.total);
+			cString s = IndexToHMSF(replay.current);
+			szCurrent = s;
+			if (replay.total > 1) { // Don't draw totaltime for endless streams  
+				cString s = IndexToHMSF(replay.total);
+				szTotal = s;
+			}
 		}
 		else
 		{

aber vielleicht ergibt sich auf einem anderen Rechner ein anderes Verhalten...
Andreas

Zitat

Original von wofritz

Die Patches haben bei mir nichts verändert. Gleich der erste Test gab wieder den bekannten segfault.

Versuche es mal mit dem letztem Patch für IndexToHMSF ...

Der Sammelpatch enthalt auch die bisherigen Modifikationen, die alten Versionen mit patch -R vorher entfernen.

Andreas

Hi,

das war nur eine Vermutung das der Downloadzähler noch auf Nullstand.

Ich muss erstmal passen, ich kann keine offensichtlichen Stellen entdecken,
die in das Problem hereinspielen, vielleicht gibt es diese Probleme nur in Verbindung mit anderen Plugins.
Was den bei einen Minimal-VDR, nur mit den zwei Plugins.

vdr -Pmp3 -Pglcd ...

Andreas

Bie mir crashed die Kiste auch. Ich bekomme mit dem gdb allerdings keinen backtrace.
Ausgabe:
#0 0x40580f61 in ?? ()
#1 0x31000074 in ?? ()
#2 0x402b6a52 in ?? ()
#3 0x40580f60 in ?? ()
#4 0x083e7d2c in ?? ()
#5 0x083e7d34 in ?? ()

Was muß ich machen, damit ich die Symbols bekomme um den Fehler im code zu finden?
Ich habe alles mit -g compiliert?

Zitat

Original von pompase
Was muß ich machen, damit ich die Symbols bekomme um den Fehler im code zu finden?
Ich habe alles mit -g compiliert?

Folgendes im VDR/Makefile (VDR/Make.config) sollte helfen

CXXFLAGS += -g -ggdb -O0
LDFLAGS += -g -ggdb -O0

Vollständiges Rebuild nicht vergessen.
(make plugins-clean ...)

Andreas

Ich habe eine interessante Entdeckung gemacht (Naja kommt darauf an, ob ein aderer die Entdeckung schon gemacht hat). Ich habe mal die Programmbilder, die im Graphlcd angezeigt werden entfernt. Daraufhin wird dann zwar kein Bildchen mehr angezeigt, aber die Kiste stürzt nicht mehr ab. Ich bin leider noch nicht wirklich zum Debuggen gekommen, aber immerhin könnte das zunächst mal ein Workarround um das Problem sein.

Hallo,

Zitat

Original von wofritz
anscheinend ist es tatsächlich so, dass graphlcd Methoden in mp3-Objekten aufruft, obwohl die Objekte bereits zerstört sind. Das liegt daran, dass das MP3-Plugin erst dann signalisiert, dass das Abspielen beendet ist, wenn die dazugehörenden Objekte schon zerstört sind. Abhilfe könnte folgender Patch im mp3-plugin bringen:

Diff

--- mp3.c       (Revision 228)
+++ mp3.c       (Arbeitskopie)
@@ -261,15 +261,18 @@


 cMP3Control::~cMP3Control()
 {
+  cStatus::MsgReplaying(this, NULL);
   delete lastMode;
   Hide();
   Stop();
-  cStatus::MsgReplaying(this, NULL);
 }

Alles anzeigen

Klingt Logisch,
am besten den Patch auch an den MP3 Maintainer weiterleiten, sofern es ehe noch nicht geschehen ist.
Andreas