diff options
author | Misha Brukman <brukman+llvm@gmail.com> | 2005-04-20 15:42:11 +0000 |
---|---|---|
committer | Misha Brukman <brukman+llvm@gmail.com> | 2005-04-20 15:42:11 +0000 |
commit | 5e7bac21d555068c4f7442cae892a781a39e1893 (patch) | |
tree | 97936c1c9bc81232cf2a8073f662357393a01432 /llvm/lib/System/Unix/Program.inc | |
parent | a9a8c1b65b1b95fd1d03daa4b03db44016dc2bc3 (diff) | |
download | bcm5719-llvm-5e7bac21d555068c4f7442cae892a781a39e1893.tar.gz bcm5719-llvm-5e7bac21d555068c4f7442cae892a781a39e1893.zip |
Add FIXME by Markus Oberhumer from bug 545: not checking for "." in $PATH may
result in returning executable files that won't be runnable.
llvm-svn: 21378
Diffstat (limited to 'llvm/lib/System/Unix/Program.inc')
-rw-r--r-- | llvm/lib/System/Unix/Program.inc | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/llvm/lib/System/Unix/Program.inc b/llvm/lib/System/Unix/Program.inc index 484ce0f8c89..94091e2ca33 100644 --- a/llvm/lib/System/Unix/Program.inc +++ b/llvm/lib/System/Unix/Program.inc @@ -44,6 +44,8 @@ Program::FindProgramByName(const std::string& progName) { Path temp; if (!temp.setFile(progName)) // invalid name return Path(); + // FIXME: have to check for absolute filename - we cannot assume anything + // about "." being in $PATH if (temp.executable()) // already executable as is return temp; |