- m_strName = strName;
-
- // we want to create the file in the same directory as strName because
- // otherwise rename() in Commit() might not work (if the files are on
- // different partitions for example). Unfortunately, the only standard
- // (POSIX) temp file creation function tmpnam() can't do it.
- #ifdef __UNIX__
- static const char *szMktempSuffix = "XXXXXX";
- m_strTemp << strName << szMktempSuffix;
- mktemp((char *)m_strTemp.c_str()); // @@@ even if the length doesn't change
- //m_strTemp.UngetWriteBuf();
- #else // Windows
- m_strTemp = tmpnam(NULL);
- #endif // Windows/Unix
-
- return m_file.Open(m_strTemp, wxFile::write);
+ // we must have an absolute filename because otherwise CreateTempFileName()
+ // would create the temp file in $TMP (i.e. the system standard location
+ // for the temp files) which might be on another volume/drive/mount and
+ // wxRename()ing it later to m_strName from Commit() would then fail
+ //
+ // with the absolute filename, the temp file is created in the same
+ // directory as this one which ensures that wxRename() may work later
+ wxFileName fn(strName);
+ if ( !fn.IsAbsolute() )
+ {
+ fn.Normalize(wxPATH_NORM_ABSOLUTE);
+ }
+
+ m_strName = fn.GetFullPath();
+
+ m_strTemp = wxFileName::CreateTempFileName(m_strName, &m_file);
+
+ if ( m_strTemp.empty() )
+ {
+ // CreateTempFileName() failed
+ return FALSE;
+ }
+
+#ifdef __UNIX__
+ // the temp file should have the same permissions as the original one
+ mode_t mode;
+
+ wxStructStat st;
+ if ( stat(m_strName.fn_str(), &st) == 0 )
+ {
+ mode = st.st_mode;
+ }
+ else
+ {
+ // file probably didn't exist, just give it the default mode _using_
+ // user's umask (new files creation should respect umask)
+ mode_t mask = umask(0777);
+ mode = 0666 & ~mask;
+ umask(mask);
+ }
+
+ if ( chmod(m_strTemp.mb_str(), mode) == -1 )
+ {
+ wxLogSysError(_("Failed to set temporary file permissions"));
+ }
+#endif // Unix
+
+ return TRUE;