[mythtv-commits] Ticket #5186: mkstemp() should be wrapped by umask

[MythTV]

#5186: mkstemp() should be wrapped by umask
---------------------------------------------+------------------------------
 Reporter:  Erik Hovland <erik at hovland.org>  |       Owner:  ijr    
     Type:  defect                           |      Status:  new    
 Priority:  minor                            |   Milestone:  unknown
Component:  mythtv                           |     Version:  head   
 Severity:  low                              |     Mlocked:  0      
---------------------------------------------+------------------------------
 In programs/mythfilldatabase/filldata.cpp in the member function
 FillData::grabData() there is a call to mkstemp(). Which is the right
 function to use when making temporary files. But when using mkstemp() it
 is important to explicitly set the umask of the process before call. That
 way there is no doubt the the permissions on the temporary file are known
 to be safe.

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/5186>
MythTV <http://www.mythtv.org/>
MythTV