[mythtv] [mythtv-commits] Ticket #11195: Patch: Adds detail when mythfilldatabase cache isn't writable

Daniel Thor Kristjansson danielk at cuymedia.net
Sat Nov 3 00:01:20 UTC 2012


On 11/02/2012 06:51 PM, Gary Buhrmaster wrote:
>       This problem can be avoided by setting the permissioins on the file
>       to allow anyone to write it (the 1st user to run the program does
>       this and any other user silently fails to change them.)
> Rather than trying to subvert the unix security model(*), may
> I suggest a better approach would be to create a temporary
> directory with a random name (using QTemporaryDir?) and
> use that as needed (with the other required code changes
> of course).

You know I was a bit puzzled by this because I distinctly
remembered creating a temporary directory and putting temp
files in it when I last touched that code. I looked just
now and the temp directory code is still there, see 
DataDirectProcessor::CreateTempDirectory().

DataDirectProcessor::DDPost() looks like it's an exception
and it doesn't event make any sense. We write out the data
to this file from a QByteArray and then immediately read it
back out of the file into a new QByteArray. It also doesn't
use m_tmpDir like the other DataDirect stuff that touches
the file system.

I'll assign that ticket to myself and take a look at it soon.

-- Daniel



More information about the mythtv-dev mailing list