Ticket #4860: mythtv-4860-scaled_preview_filesystem_permissions-makeFileAccessible-20080927.patch

libs/libmythtv/previewgenerator.cpp

@@ -466 +466 @@
     QByteArray fname = filename.toAscii();
     if (small_img.save(fname.constData(), "PNG"))
     {
-        chmod(fname.constData(), 0666); // Let anybody update it
+        makeFileAccessible(fname.constData()); // Let anybody update it
 
         VERBOSE(VB_PLAYBACK, LOC +
                 QString("Saved preview '%0' %1x%2")
@@ -481 +481 @@
     if (QFileInfo(fname.constData()).exists() &&
         small_img.save(newfilea.constData(), "PNG"))
     {
-        chmod(newfilea.constData(), 0666);
+        makeFileAccessible(newfilea.constData());
         rename(newfilea.constData(), fname.constData());
 
         VERBOSE(VB_PLAYBACK, LOC +

libs/libmythtv/datadirect.cpp

@@ -1644 +1644 @@
 
     VERBOSE(VB_GENERAL, "SaveLineupToCache("<<lineupid<<") -- success");
 
-    chmod(fna.constData(), 0666); // Let anybody update it
+    makeFileAccessible(fna.constData()); // Let anybody update it
 
     return true;
 }

libs/libmyth/util.h

@@ -58 +58 @@
 MPUBLIC long long copy(QFile &dst, QFile &src, uint block_size = 0);
 MPUBLIC QString createTempFile(QString name_template = "/tmp/mythtv_XXXXXX",
                                bool dir = false);
+MPUBLIC void makeFileAccessible(QString filename);
 
 MPUBLIC double MythGetPixelAspectRatio(void);
 

libs/libmyth/util.cpp

@@ -802 +802 @@
     return tmpFileName;
 }
 
+/** \fn makeFileAccessible(QString)
+ *  \brief Makes a file accessible to all frontends/backends.
+ *
+ *   This function abstracts the functionality of making a file accessible to
+ *   all frontends and backends.  Currently it contains a permissions hack that
+ *   makes a file accessible even on a system with an improperly configured
+ *   environment (umask/group) where the frontend and backend are being run as
+ *   different users or where a NFS share is used but UID's/GID's differ on
+ *   different hosts.
+ *
+ *   Though the function currently only changes the file mode to 0666, by
+ *   abstracting the functionality, it will be easier to make changes in the
+ *   future if a better approach is chosen.  Similarly, using this function
+ *   allows the hack to be applied only when required if code is written to
+ *   detect or allow the user to specify their system is misconfigured.
+ *
+ *  \param filename   Path of file to make accessible
+ */
+void makeFileAccessible(QString filename)
+{
+    QByteArray fname = filename.toAscii();
+    chmod(fname.constData(), 0666);
+}
+
 double MythGetPixelAspectRatio(void)
 {
     float pixelAspect = 1.0;

programs/mythbackend/mythxml.cpp

@@ -1196 +1196 @@
     QByteArray fname = pRequest->m_sFileName.toAscii();
     img.save( fname.constData(), "PNG" );
 
+    makeFileAccessible(fname.constData());
+
     delete pImage;
 }