"Segmentation Fault", wenn VDR in recording.c auf "esyslog("ERROR: can't read from file '%s'", *fileName);" (Zeile 2605) kommt.

MarkusE · Jan 21st 2022

Etwa so:

Ein Programm ruft cMarks::Load .

Das ruft cMarks::Update(void)

Das ruft Align();

Das ruft

cIndexFile IndexFile(recordingFileName, false, isPesRecording);

Code

Zeile 2597: last = int((buf.st_size + delta) / sizeof(tIndexTs) - 1);
(das ist größer als 0)
Zeile 2604: if (safe_read(f, index, size_t(buf.st_size)) != buf.st_size)
das Lesen geht schief, warum ist mir unklar, möglicherweise ein Netzwerkfehler)
Zeile 2607 index = NULL
Im Objekt "IndexFile" ist also index = NULL und last > 0.

dann IndexFile.GetClosestIFrame(m->Position());

Zeile 2822: if (index[Index].independent)

Hier der "Segmentation Fault", da index == NULL .

~ Markus

SHF · Jan 21st 2022

Ich habe Abstürze bei leerer index-Datei beobachtet.

Aber nur, wenn die Datei 0 Byte hat und vorhanden ist. Wenn sie ganz fehlt, wird sie neu erzeugt.

Erreichen kann man den Zustand, in dem man während der Index-Generierung "exit" drückt und die Wiedergabe beendet.

Könnte das, dein Problem sein?

Sämtliche genannten Zeilennummern stammen cIndexFile::...

Die Abfrage access(fileName, R_OK)(Zeile 2577) scheint wohl nicht zu reichen.

Da müsste man wohl mindestens noch auf Grösse >0 prüfen.

Was bei unvollständigen index-Dateien passiert ist dann auch noch eine Frage. (ist bei mir aber noch nicht aufgetreten)

Da mein Produktiv-System inzwischen etwas (hust) veraltet ist, wollte ich nicht sofort einen Bugreport aufmachen.

MarkusE · Jan 22nd 2022

Teste doch mal den attachten Patch

kls · Jan 24th 2022

MarkusE : beim zweiten hunk deines Patchs fehlen die Klammern.

Im Fehlerfall 'last = -1' zu setzen ist sicher sinnvoll. Zusätzlich würde ich noch 'size = 0' setzen, damit alles zusammenpasst.

Anbei ein Patch, der auch noch in cIndexFile::GetClosestIFrame() 'index' abfragt.

SHF · Jan 24th 2022

Danke für die Arbeit. Ich werde es die Tage mal versuchen.

So wie ich das verstehe sollte der Patch den Absturz beheben, die leere Index-Datei sollte aber nicht regeneriert werden. Mal schauen, was passiert ....

kls · Jan 25th 2022

SHF : Ja, das sollte erstmal nur den Absturz verhindern.

Um eine leere Indexdatei zu regenerieren sollte in cIndexFile::cIndexFile() die Zeile

Code

if (!Record && access(fileName, R_OK) != 0) {

nach

Code

if (!Record && (access(fileName, R_OK) != 0 || FileSize(fileName) == 0)) {

geändert werden. Magst du das mal probieren?

SHF · Jan 27th 2022

Wenn ich es schaffe den Fehler zu reproduzieren, mache ich das natürlich.

Dauert aber ein bisschen, ich bin gerade an was anderem dran.

SHF · Feb 9th 2022

Es hat jetzt etwas länger gedauert, als ich wollte, aber das Chaos, was ich beim letzten Update angerichtet hatte, war hartnäckig er als gedacht. Nun spricht aber endlich der Compiler wieder mit mir .

Den Absturz mit leerer Index-Datei konnte ich nicht reproduzieren. Ausschliessen würde ich Abstürze aber aufgrund des Verhaltens aber nicht.

Die Aufnahmen werden abgespielt (immer von vorne), aber ausser play/pause geht nichts. "ok" liefert auch keine OSD-Einblendung! Fehlermeldungen gibt es aber keine.

Patch 1 hat demzufolge (logischer weise) bei mir keine Wirkung.

Das fehlen jeglicher Fehlermeldungen finde ich etwas merkwürdig, irgendwo müsste doch auffallen, dass kein Index-Daten kommen?

Patch 2 wirkt wie erwartet und startet die Index-Generierung auch, wenn eine 0 Byte Index-File vorhanden ist.

Share