From 9adeccc552d6f28bdc503a8c1ffe13ea6ea6d73d Mon Sep 17 00:00:00 2001 From: John Criswell Date: Tue, 2 Sep 2003 20:30:16 +0000 Subject: [PATCH] Added a description of the algorithm. Return failure if the chmod() fails. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@8326 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Support/FileUtilities.cpp | 21 ++++++++++++++------- support/lib/Support/FileUtilities.cpp | 21 ++++++++++++++------- 2 files changed, 28 insertions(+), 14 deletions(-) diff --git a/lib/Support/FileUtilities.cpp b/lib/Support/FileUtilities.cpp index 01d7ca2126c..16c41bc7b9e 100644 --- a/lib/Support/FileUtilities.cpp +++ b/lib/Support/FileUtilities.cpp @@ -98,12 +98,14 @@ std::string getUniqueFilename(const std::string &FilenameBase) { /// /// Description: /// This method makes the specified filename executable by giving it -/// execute permission. +/// execute permission. It respects the umask value of the process, and it +/// does not enable any unnecessary access bits. /// -/// For the UNIX version of this method, we turn on all of the read and -/// execute bits and then turn off anything specified in the umask. This -/// should help ensure that access to the file remains at the level that -/// the user desires. +/// Algorithm: +/// o Get file's current permissions. +/// o Get the process's current umask. +/// o Take the set of all execute bits and disable those found in the umask. +/// o Add the remaining permissions to the file's permissions. /// bool MakeFileExecutable (const std::string & Filename) @@ -134,8 +136,13 @@ MakeFileExecutable (const std::string & Filename) return false; } - // Make the script executable... - chmod(Filename.c_str(), (fstat.st_mode | (0111 & ~mask))); + // + // Make the file executable... + // + if ((chmod(Filename.c_str(), (fstat.st_mode | (0111 & ~mask)))) == -1) + { + return false; + } return true; } diff --git a/support/lib/Support/FileUtilities.cpp b/support/lib/Support/FileUtilities.cpp index 01d7ca2126c..16c41bc7b9e 100644 --- a/support/lib/Support/FileUtilities.cpp +++ b/support/lib/Support/FileUtilities.cpp @@ -98,12 +98,14 @@ std::string getUniqueFilename(const std::string &FilenameBase) { /// /// Description: /// This method makes the specified filename executable by giving it -/// execute permission. +/// execute permission. It respects the umask value of the process, and it +/// does not enable any unnecessary access bits. /// -/// For the UNIX version of this method, we turn on all of the read and -/// execute bits and then turn off anything specified in the umask. This -/// should help ensure that access to the file remains at the level that -/// the user desires. +/// Algorithm: +/// o Get file's current permissions. +/// o Get the process's current umask. +/// o Take the set of all execute bits and disable those found in the umask. +/// o Add the remaining permissions to the file's permissions. /// bool MakeFileExecutable (const std::string & Filename) @@ -134,8 +136,13 @@ MakeFileExecutable (const std::string & Filename) return false; } - // Make the script executable... - chmod(Filename.c_str(), (fstat.st_mode | (0111 & ~mask))); + // + // Make the file executable... + // + if ((chmod(Filename.c_str(), (fstat.st_mode | (0111 & ~mask)))) == -1) + { + return false; + } return true; } -- 2.34.1