diff options
author | Argyrios Kyrtzidis <akyrtzi@gmail.com> | 2013-02-28 00:38:19 +0000 |
---|---|---|
committer | Argyrios Kyrtzidis <akyrtzi@gmail.com> | 2013-02-28 00:38:19 +0000 |
commit | 2b9a946f981963c226eaafe1dd8fb8801b87d411 (patch) | |
tree | dca429f43813d4c4f5ed19017562a1e349add761 /lib/Support | |
parent | d2eaf45af446ae0a4dd1e606ad6e16f0822f4405 (diff) | |
download | external_llvm-2b9a946f981963c226eaafe1dd8fb8801b87d411.zip external_llvm-2b9a946f981963c226eaafe1dd8fb8801b87d411.tar.gz external_llvm-2b9a946f981963c226eaafe1dd8fb8801b87d411.tar.bz2 |
[PathV2] In llvm::sys::fs::unique_file, make sure it doesn't fall into an infinite loop by constantly trying
to create the parent path.
This can happen if the path is a relative filename and the current directory was removed.
Thanks to Daniel D. for the hint in fixing it.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@176226 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Support')
-rw-r--r-- | lib/Support/Unix/PathV2.inc | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/lib/Support/Unix/PathV2.inc b/lib/Support/Unix/PathV2.inc index 741f44a..44b31b3 100644 --- a/lib/Support/Unix/PathV2.inc +++ b/lib/Support/Unix/PathV2.inc @@ -417,6 +417,10 @@ retry_random_path: RandomPath[i] = "0123456789abcdef"[sys::Process::GetRandomNumber() & 15]; } + // Make sure we don't fall into an infinite loop by constantly trying + // to create the parent path. + bool TriedToCreateParent = false; + // Try to open + create the file. rety_open_create: int RandomFD = ::open(RandomPath.c_str(), O_RDWR | O_CREAT | O_EXCL, mode); @@ -427,7 +431,9 @@ rety_open_create: goto retry_random_path; // If path prefix doesn't exist, try to create it. if (SavedErrno == errc::no_such_file_or_directory && - !exists(path::parent_path(RandomPath))) { + !exists(path::parent_path(RandomPath)) && + !TriedToCreateParent) { + TriedToCreateParent = true; StringRef p(RandomPath); SmallString<64> dir_to_create; for (path::const_iterator i = path::begin(p), |