llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Jan Svoboda (jansvoboda11) <details> <summary>Changes</summary> There's some logic in Linux distro detection that relies on the VFS being the real FS. This check is implemented such that it violates the IO sandbox. This PR implements narrow sandbox disablement for this specific check. --- Full diff: https://github.com/llvm/llvm-project/pull/175201.diff 1 Files Affected: - (modified) clang/lib/Driver/Distro.cpp (+5-1) ``````````diff diff --git a/clang/lib/Driver/Distro.cpp b/clang/lib/Driver/Distro.cpp index df10458d092d6..3c335891f9971 100644 --- a/clang/lib/Driver/Distro.cpp +++ b/clang/lib/Driver/Distro.cpp @@ -11,6 +11,7 @@ #include "llvm/ADT/StringRef.h" #include "llvm/ADT/StringSwitch.h" #include "llvm/Support/ErrorOr.h" +#include "llvm/Support/IOSandbox.h" #include "llvm/Support/MemoryBuffer.h" #include "llvm/Support/Threading.h" #include "llvm/TargetParser/Host.h" @@ -211,7 +212,10 @@ static Distro::DistroType GetDistro(llvm::vfs::FileSystem &VFS, return Distro::UnknownDistro; // True if we're backed by a real file system. - const bool onRealFS = (llvm::vfs::getRealFileSystem() == &VFS); + const bool onRealFS = [&] { + auto BypassSandbox = llvm::sys::sandbox::scopedDisable(); + return llvm::vfs::getRealFileSystem() == &VFS; + }(); // If the host is not running Linux, and we're backed by a real file // system, no need to check the distro. This is the case where someone `````````` </details> https://github.com/llvm/llvm-project/pull/175201 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
