Trigger von UPDR auf verbundenen VDRs bei Aufnahmestart auf Server

mrjoe · August 4, 2024 at 6:21 PM

kls : bitte die Datei mit permission 666 anlegen lassen, damit es keine Zugriffsprobleme gibt. Wäre auch gut bei der ".sort".

Hintergrund: z.B. bei softhdodroid muss der vdr als root laufen und in meinem Fall wurde dann die .sort mit 644 und somit nur für root schreibend angelegt.

kls · September 2, 2024 at 10:53 AM

mrjoe Sollte sich das nicht über "umask" steuern lassen?

Dazu gab es extra mal einen Patch: https://patchwork.linuxtv.org/project/vdr/pa…@wopr.deltab.de

Im GIT: http://git.tvdr.de/?p=vdr.git;a=c…0f4a052d1f9d511

mrjoe · September 7, 2024 at 6:04 PM

Man könnte es über umask machen, zur Vereinfachung wäre es super, wenn vdr die Datei bei Bedarf mit den richtigen permissions anlegt. Etwa so in der Art:

Diff

--- tools.c.orig        2024-09-07 17:54:32.728622969 +0200
+++ tools.c     2024-09-07 18:01:29.332622969 +0200
@@ -716,8 +716,18 @@

 void TouchFile(const char *FileName)
 {
-  if (utime(FileName, NULL) == -1 && errno != ENOENT)
-     LOG_ERROR_STR(FileName);
+  if (utime(FileName, NULL) == -1)
+     if (errno == ENOENT) {
+        int fd = creat(FileName);
+        if (fd != -1) {
+           fchmod(fd, 0666);
+           close(fd);
+           }
+        else
+           LOG_ERROR_STR(FileName);
+        }
+     else
+        LOG_ERROR_STR(FileName);
 }

 time_t LastModifiedTime(const char *FileName)

Display More

kls · September 8, 2024 at 4:15 PM

Quote from mrjoe

Man könnte es über umask machen, zur Vereinfachung wäre es super, wenn vdr die Datei bei Bedarf mit den richtigen permissions anlegt.

Die Funktion sieht jetzt so aus:

Code

void TouchFile(const char *FileName, bool Create)
{
   if (Create && access(FileName, F_OK) != 0) { // the file does not exist
      isyslog("creating file '%s'", FileName);
      int f = open(FileName, O_WRONLY | O_CREAT, DEFFILEMODE);
      if (f >= 0)
         close(f);
      else
         LOG_ERROR_STR(FileName);
      }
   if (utime(FileName, NULL) == -1 && errno != ENOENT)
     LOG_ERROR_STR(FileName);
}

Display More

Der neue Parameter "Create" (default = false) war nötig, um rückwärtskompatibel zu sein (falls ein Plugin die Funktion verwendet und sich darauf verlässt, dass die Datei nicht angelegt wird).

Mit umask meinte ich, dass in der aufrufenden Shell die gewünschten Zugriffsrechte als Maske gesetzt werden. VDR selber legt alle Dateien mit DEFFILEMODE an (sys/stat.h):

Code

#define DEFFILEMODE (S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH | S_IWOTH) /* 0666 */

Allerdings habe ich jetzt mal versuchsweise vor dem Aufruf von VDR ein "umask 0000" gemacht, die Dateien wurden aber dennoch mit -rw-r--r-- angelegt. Warum das so ist, weiß ich leider nicht. Auf alle Fälle macht es wohl keinen Sinn, wenn die .update bzw. .sort mit anderen Rechten angelegt werden als alle anderen Dateien.

Trigger von UPDR auf verbundenen VDRs bei Aufnahmestart auf Server

Participate now!

Users Viewing This Thread