[Patch] timers.c, duplikate

Zabrimus · November 9, 2025 at 7:26 PM New

Es passiert immer wieder, daß nicht nur doppelte, sondern eine größere Anzahl von identischen (Such-)Timern angelegt werden. Die Ursachenforschung steht noch aus, allerdings denke ich, daß der VDR selbst schon echte Duplikate ablehnen bzw. gar nicht erst erzeugen sollte. Mir fällt kein vernünftiger Use-Case ein, in dem doppelte Timer irgendwie sinnvoll sein sollten.

Das Problem lässt sich durch Hausmittel schnell nachstellen

Code

svdrpsend newt "1:1:2025-11-10:0441:0455:50:99:Tagesschau"
svdrpsend newt "1:1:2025-11-10:0441:0455:50:99:Tagesschau"
svdrpsend newt "1:1:2025-11-10:0441:0455:50:99:Tagesschau"

Aktuell werden tatsächlich 3 Timer erstellt und mit dem Patch exakt nur einer.

timer.c.patch.txt

jsffm · November 10, 2025 at 8:44 PM New

Das Problem hatte ich auch schon, spätestens, wenn das wieder passiert, werde ich das Testen.

SHF · November 14, 2025 at 12:30 AM New

Ich weiß nicht ob das hilft, aber ich bin noch mit einer ziemlich alten Version vom EPGsearch unterwegs. Die hat auch diverse Macken, aber mehrfache Timer habe das noch nie bemerkt. Der Fehler müsste also eigentlich irgendwann in den letzten 10 Jahren oder so rein gekommen sein.

Btw. blöde Frage:
Erzeugen die mehrfachen Timer auch mehrfach die gleiche Aufnahme oder wird das da dann irgendwo unterbunden?

Zabrimus · November 14, 2025 at 4:55 AM New

Die mehrfachen Timer werden durch epgd erzeugt. Durch mein letztes Plugin konnte ich schön sehen, daß jede Stunde neue Timer erzeugt wurden. Und wenn ich mal eine Zeitlang die Timer nicht im Auge hatte, gab es extrem viele und die einzige (bequeme) Lösung war das Stoppen des VDR und das manuelle Löschen der Timer aus der timers.conf.

Da dies alles remote timer sind, war der Start der VDR Client auch eher anspruchsvoll. Bis das OSD bedienbar wurde, dauerte es Minuten. Ich bin mir aber sicher, daß es irgendwo noch eine Bremse gibt,

Mehrfache Aufnahmen wären mir nicht aufgefallen.

Mit dem Patch habe ich zumindest mit den Timern Ruhe. Es gibt nicht mehr, als notwendig.

kls · November 14, 2025 at 9:54 PM New

Zabrimus Hast du das auch mit Remote-Timern getestet?

Zabrimus · November 14, 2025 at 10:37 PM New

Den Fall mit den Remote-Timern hatte ich bisher nicht, aber ich habe es mal probiert:
- Timer auf dem Server angelegt
- Client zeigt den RemoteTimer an
- Timer auf dem Client angelegt (wird angelegt, da RemoteTimer wohl nicht berücksichtigt werden)
- Versucht, den Timer vom Client auf den Server zu schieben. Dabei kam die Fehlermeldung sinngemäß "Fehler beim Ansprechen des entfernten Timers).

Also habe ich 2 identische Timer: Einen jeweils auf dem Server und auf dem Client. Jetzt weiß ich nicht, ob das richtig oder falsch ist. Für beides gibt es Argumente.

kls · November 14, 2025 at 11:02 PM New

Ich denke mal, es ist nicht "verboten", lokal einen Timer anzulegen, den es remote bereits gibt. Hauptsache, es passiert nichts "schlimmes" ;-).

Über SVDRP können sowieso nur lokale timer angelegt werden.

kls · November 14, 2025 at 11:17 PM New

Es gibt doch etwas, was gegen diesen Patch spricht: an der Aufrufstelle von cTimers::Add() wird davon ausgegangen, dass der Timer in den "Besitz" der Liste übergeht. Wird er das durch diesen Patch im Falle eines bereits vorhandenen solchen Timers nicht, dann gibt es niemanden, der diesen später löscht, also haben wir ein Memory-Leak.

Falls das Problem nur bei über SVDRP neu angelegten Timern besteht, könnte man das in cSVDRPServer::CmdNEWT() entsprechend einbauen. Da dein Testcase NEWT macht, gehe ich mal davon aus und werde das dort ändern.

kls · November 15, 2025 at 11:49 AM New

Hier ein Vorschlag für einen Fix in NEWT.

Zabrimus · November 15, 2025 at 1:46 PM New

Der Patch ist elegant und funktioniert ziemlich gut. Besonders gefällt mir, daß man den schon vorhandenen Timer zurückbekommt

Code

# svdrpsend newt "1:1:2025-11-15:1958:2025:50:99:Tagesschau:<epgd><timerid>28405</timerid></epgd>"
220 client1 SVDRP VideoDiskRecorder 2.7.4; Sat Nov 15 13:42:41 2025; UTF-8
550 1 1:C-1-1051-10301:2025-11-15:1958:2025:50:99:Tagesschau:<epgd><timerid>28405</timerid></epgd>
221 client1 closing connection


vdr: [2699107] SVDRP client1 < 127.0.0.1:50960 attempted to add timer 0 (1 1958-2025 'Tagesschau')
vdr: [2699107] SVDRP client1 < 127.0.0.1:50960 timer already exists as 1 (1 1958-2025 'Tagesschau')

[Patch] timers.c, duplikate

Participate now!

Users Viewing This Thread