VDR crasht für "CLRE" bei unbekannter Kanalnummer

M-Reimer · 1. Januar 2021

Habe gerade etwas mit SVDRP rumprobiert und mich gewundert warum ich für "CLRE 100" keine Antwort bekomme. Bis ich gesehen habe das der VDR-Server sich dann jedes Mal restartet:

Spoiler anzeigen

Tritt wohl auf wenn man CLRE auf eine Kanalnummer ausführen will, welche durch die "Kanalgruppen-Syntax" nicht exisitert. Also in einer durch die Channelgroups erzeugte Lücke fällt. Eventuell betrifft das auch andere SVDRP-Befehle.

kls · 1. Januar 2021

Quick fix (ungetestet):

Diff

--- svdrp.c     2020/12/26 15:49:01     5.1
+++ svdrp.c     2021/01/01 19:02:09
@@ -1301,8 +1301,10 @@
      tChannelID ChannelID = tChannelID::InvalidID;
      if (isnumber(Option)) {
         int o = strtol(Option, NULL, 10);
-        if (o >= 1 && o <= cChannels::MaxNumber())
-           ChannelID = Channels->GetByNumber(o)->GetChannelID();
+        if (o >= 1 && o <= cChannels::MaxNumber()) {
+           if (const cChannel *Channel = Channels->GetByNumber(o))
+              ChannelID = Channel->GetChannelID();
+           }
         }
      else {
         ChannelID = tChannelID::FromString(Option);

Alles anzeigen

davie2000 · 1. Januar 2021

Wow ... das nenn ich mal schnell reagiert

Da können sich professionelle Softwareschmieden ein paar Scheiben abschneiden!

M-Reimer · 1. Januar 2021

Bin auch überrascht! Danke für den schnellen Fix! Jetzt wo ich den Fix sehe ist mir auch klar wo das Problem ist

Komme aber erst morgen zum Testen. Ich baue das dann direkt ins PKGBUILD für Arch ein. Macht mir das Testen einfacher.

M-Reimer · 2. Januar 2021

Verhalten nach dem Patchen:

Code

$ svdrpsend CLRE 100
220 manuelsvdr SVDRP VideoDiskRecorder 2.4.6; Sat Jan  2 19:13:29 2021; UTF-8
501 Undefined channel "100"
221 manuelsvdr closing connection

Der Patch funktioniert also einwandfrei

VDR crasht für "CLRE" bei unbekannter Kanalnummer

Jetzt mitmachen!

Teilen