picnoir/Nix
1
0
Fork 0
Code Issues Pull requests 1 Releases Wiki Activity

tarfile.cc: Don't change the cwd

Browse source 
Nix is multithreaded so it's not safe to change the cwd.
This commit is contained in:
Eelco Dolstra 2019-12-19 15:08:16 +01:00
parent be32da0ed0
commit 2550c11373
No known key found for this signature in database
GPG key ID: 8170B4726D7198DE
1 changed files with 4 additions and 23 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

27
src/libutil/tarfile.cc
View file

@ -80,34 +80,12 @@ private:
}
};
struct PushD {
char * oldDir;
PushD(const std::string &newDir) {
oldDir = getcwd(0, 0);
if (!oldDir) throw SysError("getcwd");
int r = chdir(newDir.c_str());
if (r != 0) throw SysError("changing directory to tar output path");
}
~PushD() {
int r = chdir(oldDir);
free(oldDir);
if (r != 0)
warn("failed to change directory back after tar extraction");
/* can't throw out of a destructor */
}
};
static void extract_archive(TarArchive & archive, const Path & destDir)
{
// need to chdir back *after* archive closing
PushD newDir(destDir);
int flags = ARCHIVE_EXTRACT_FFLAGS
| ARCHIVE_EXTRACT_PERM
| ARCHIVE_EXTRACT_SECURE_SYMLINKS
| ARCHIVE_EXTRACT_SECURE_NODOTDOT
| ARCHIVE_EXTRACT_SECURE_NOABSOLUTEPATHS;
| ARCHIVE_EXTRACT_SECURE_NODOTDOT;
for (;;) {
struct archive_entry * entry;
@ -118,6 +96,9 @@ static void extract_archive(TarArchive & archive, const Path & destDir)
else
archive.check(r);
archive_entry_set_pathname(entry,
(destDir + "/" + archive_entry_pathname(entry)).c_str());
archive.check(archive_read_extract(archive.archive, entry, flags));
}