[ANNOUNCE] VDR Version 2.6.2 freigegeben

Murry · December 7, 2022 at 9:53 PM

Jetzt ist alles da.

Danke

seahawk1986 · December 11, 2022 at 7:51 PM

Wie es aussieht, tested der VDR 2.6.2 den Zugriff auf das Video-Verzeichnis, bevor er den Wechsel von root auf den vdruser macht (http://git.tvdr.de/?p=vdr.git;a=b…c8;hb=HEAD#l656 ff.) - damit gibt es Probleme, wenn das VIDEODIR auf einer NFS-Freigabe liegt, für die die root_squash-Option nicht gesetzt ist (und man merkt nicht direkt beim Start, wenn der vdruser keine Zugriffsrechte auf das Verzeichnis hat) - kann man da eventuell die Reihenfolge wieder so gerade ziehen, dass der Zugriff aufs Terminal vor dem setuid passiert, aber der Zugriff auf das Videoverzeichnis erst danach erfolgt?

kls · December 12, 2022 at 11:25 AM

seahawk1986 Klar kann man das:

Diff

--- vdr.c       2022/11/28 10:44:01     5.10
+++ vdr.c       2022/12/12 10:18:06
@@ -651,13 +651,6 @@
   if (SysLogLevel > 0)
      openlog("vdr", LOG_CONS, SysLogTarget); // LOG_PID doesn't work as expected under NPTL

-  // Check the video directory:
-
-  if (!DirectoryOk(VideoDirectory, true)) {
-     fprintf(stderr, "vdr: can't access video directory %s\n", VideoDirectory);
-     return 2;
-     }
-
   // Daemon mode:

   if (DaemonMode) {
@@ -692,6 +685,13 @@
         }
      }

+  // Check the video directory:
+
+  if (!DirectoryOk(VideoDirectory, true)) {
+     fprintf(stderr, "vdr: can't access video directory %s\n", VideoDirectory);
+     return 2;
+     }
+
   isyslog("VDR version %s started", VDRVERSION);
   if (StartedAsRoot && VdrUser)
      isyslog("switched to user '%s'", VdrUser);

Display More

seahawk1986 · December 12, 2022 at 1:46 PM

Danke, ich habe das mal in meine Pakete für den VDR 2.6.2 übernommen.

Taipan · January 1, 2024 at 3:30 PM

Quote from HagenS

Im Wesentlichen wäre das der Zapcockpit Patch, der hier für einen deutlich höheren WAF verantwortlich zeigt

seahawk1986

Aus gegebenem Anlass möchte ich mal freundlichst nachfragen ob dieser Patch nicht per Default in Deine VDR-Versionen wandern könnte weil er doch ungenutzt keinerlei Auswirkung auf die Funktion hat. In Sachen Übersichtlichkeit bei Bouquets und Haptik finde ich ihn unschlagbar! Vielleicht könnte man ihn "separieren" und/oder über vdrctl aktivierbar machen wie den "Hide first Recording Level - Patch"?

Meine bescheidene persönliche Meinung ist aber diese Funktion fest für den VDR aufzunehmen... [kopfeinzieh]

Oder ist der Patch gar so enorm tiefgreifend/problematisch das er irgendwo "zwischenhauen" kann?

Zabrimus · January 1, 2024 at 6:34 PM

Falls Interesse besteht, dann findet man hier eine Version des Patches, bei dem man zapcockpit komplett per Setup abschalten kann. Dann wird auch keinerlei Funktionalität des Patches verwendet und es gibt keine Seiteneffekte.

seahawk1986 · January 1, 2024 at 7:00 PM

Danke, das schaue ich mir mal an.

Taipan · January 23, 2024 at 1:46 PM

seahawk1986

Gibt es schon irgendwelche Erkenntnisse über den Zapcockpit-Patch?

Darf er zeitnah 'DEFAULT' werden in Deinem PPA?

kls · January 23, 2024 at 1:56 PM

Nur zur Info: es wird in den nächsten Tagen eine VDR Version 2.66 geben. Ich warte nur noch, was hier rauskommt.

Participate now!