wbtiger
/
foundationdb
mirror of https://github.com/apple/foundationdb.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

atomicReplace to preserve owner & permission info 

atomicReplace currently does not preserve the
ownership and permission info, this fix will
preserves the ownership and permission info - it
fails the call if it is not able to do so.
This commit is contained in:
sramamoorthy 2019-03-18 00:18:05 -07:00
parent 178ebfec30
commit 51a655d9c7
1 changed files with 21 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
flow/Platform.cpp
View File

@ -1666,6 +1666,27 @@ void atomicReplace( std::string const& path, std::string const& content, bool te
f = textmode ? fopen( tempfilename.c_str(), "wt" ) : fopen(tempfilename.c_str(), "wb");
if(!f)
throw io_error();
#ifdef _WIN32
#elif defined(__unixish__)
// get the uid/gid/mode bits of old file and set it on new file, else fail
struct stat info;
if (stat(path.c_str(), &info) < 0) {
TraceEvent("StatFailed").detail("path", path);
throw io_error();
}
if (chown(tempfilename.c_str(), info.st_uid, info.st_gid) < 0) {
deleteFile(tempfilename);
TraceEvent("ChownFailed").detail("tempfilename", tempfilename).detail("uid", info.st_uid).detail("gid", info.st_gid);
throw io_error();
}
if (chmod(tempfilename.c_str(), info.st_mode) < 0) {
deleteFile(tempfilename);
TraceEvent("ChmodFailed").detail("tempfilename", tempfilename).detail("mode", info.st_mode);
throw io_error();
}
#else
#error Port me!
#endif
if( textmode && fprintf( f, "%s", content.c_str() ) < 0)
throw io_error();