
This reverts commita0260a95ec
, reapplying7c5f5f3ef8
, with a fix that makes *both* pipe handles inheritable. The original commit description was: This is a follow-up to https://github.com/llvm/llvm-project/pull/126935, which enables passing handles to a child process on windows systems. Unlike on unix-like systems, the handles need to be created with the "inheritable" flag because there's to way to change the flag value after it has been created. This is why I don't respect the child_process_inherit flag but rather always set the flag to true. (My next step is to delete the flag entirely.) This does mean that pipe may be created as inheritable even if its not necessary, but I think this is offset by the fact that windows (unlike unixes, which pass all ~O_CLOEXEC descriptors through execve and *all* descriptors through fork) has a way to specify the precise set of handles to pass to a specific child process. If this turns out to be insufficient, instead of a constructor flag, I'd rather go with creating a separate api to create an inheritable copy of a handle (as typically, you only want to inherit one end of the pipe).
227 lines
8.2 KiB
C++
227 lines
8.2 KiB
C++
//===-- ProcessLauncherWindows.cpp ----------------------------------------===//
|
|
//
|
|
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
|
|
// See https://llvm.org/LICENSE.txt for license information.
|
|
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
|
|
//
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
#include "lldb/Host/windows/ProcessLauncherWindows.h"
|
|
#include "lldb/Host/HostProcess.h"
|
|
#include "lldb/Host/ProcessLaunchInfo.h"
|
|
|
|
#include "llvm/ADT/ScopeExit.h"
|
|
#include "llvm/ADT/SmallVector.h"
|
|
#include "llvm/Support/ConvertUTF.h"
|
|
#include "llvm/Support/Program.h"
|
|
|
|
#include <string>
|
|
#include <vector>
|
|
|
|
using namespace lldb;
|
|
using namespace lldb_private;
|
|
|
|
static void CreateEnvironmentBuffer(const Environment &env,
|
|
std::vector<char> &buffer) {
|
|
// The buffer is a list of null-terminated UTF-16 strings, followed by an
|
|
// extra L'\0' (two bytes of 0). An empty environment must have one
|
|
// empty string, followed by an extra L'\0'.
|
|
for (const auto &KV : env) {
|
|
std::wstring warg;
|
|
if (llvm::ConvertUTF8toWide(Environment::compose(KV), warg)) {
|
|
buffer.insert(
|
|
buffer.end(), reinterpret_cast<const char *>(warg.c_str()),
|
|
reinterpret_cast<const char *>(warg.c_str() + warg.size() + 1));
|
|
}
|
|
}
|
|
// One null wchar_t (to end the block) is two null bytes
|
|
buffer.push_back(0);
|
|
buffer.push_back(0);
|
|
// Insert extra two bytes, just in case the environment was empty.
|
|
buffer.push_back(0);
|
|
buffer.push_back(0);
|
|
}
|
|
|
|
static bool GetFlattenedWindowsCommandString(Args args, std::wstring &command) {
|
|
if (args.empty())
|
|
return false;
|
|
|
|
std::vector<llvm::StringRef> args_ref;
|
|
for (auto &entry : args.entries())
|
|
args_ref.push_back(entry.ref());
|
|
|
|
llvm::ErrorOr<std::wstring> result =
|
|
llvm::sys::flattenWindowsCommandLine(args_ref);
|
|
if (result.getError())
|
|
return false;
|
|
|
|
command = *result;
|
|
return true;
|
|
}
|
|
|
|
HostProcess
|
|
ProcessLauncherWindows::LaunchProcess(const ProcessLaunchInfo &launch_info,
|
|
Status &error) {
|
|
error.Clear();
|
|
|
|
std::string executable;
|
|
std::vector<char> environment;
|
|
STARTUPINFOEX startupinfoex = {};
|
|
STARTUPINFO &startupinfo = startupinfoex.StartupInfo;
|
|
PROCESS_INFORMATION pi = {};
|
|
|
|
HANDLE stdin_handle = GetStdioHandle(launch_info, STDIN_FILENO);
|
|
HANDLE stdout_handle = GetStdioHandle(launch_info, STDOUT_FILENO);
|
|
HANDLE stderr_handle = GetStdioHandle(launch_info, STDERR_FILENO);
|
|
auto close_handles = llvm::make_scope_exit([&] {
|
|
if (stdin_handle)
|
|
::CloseHandle(stdin_handle);
|
|
if (stdout_handle)
|
|
::CloseHandle(stdout_handle);
|
|
if (stderr_handle)
|
|
::CloseHandle(stderr_handle);
|
|
});
|
|
|
|
startupinfo.cb = sizeof(startupinfoex);
|
|
startupinfo.dwFlags |= STARTF_USESTDHANDLES;
|
|
startupinfo.hStdError =
|
|
stderr_handle ? stderr_handle : ::GetStdHandle(STD_ERROR_HANDLE);
|
|
startupinfo.hStdInput =
|
|
stdin_handle ? stdin_handle : ::GetStdHandle(STD_INPUT_HANDLE);
|
|
startupinfo.hStdOutput =
|
|
stdout_handle ? stdout_handle : ::GetStdHandle(STD_OUTPUT_HANDLE);
|
|
|
|
std::vector<HANDLE> inherited_handles;
|
|
if (startupinfo.hStdError)
|
|
inherited_handles.push_back(startupinfo.hStdError);
|
|
if (startupinfo.hStdInput)
|
|
inherited_handles.push_back(startupinfo.hStdInput);
|
|
if (startupinfo.hStdOutput)
|
|
inherited_handles.push_back(startupinfo.hStdOutput);
|
|
|
|
size_t attributelist_size = 0;
|
|
InitializeProcThreadAttributeList(/*lpAttributeList=*/nullptr,
|
|
/*dwAttributeCount=*/1, /*dwFlags=*/0,
|
|
&attributelist_size);
|
|
|
|
startupinfoex.lpAttributeList =
|
|
static_cast<LPPROC_THREAD_ATTRIBUTE_LIST>(malloc(attributelist_size));
|
|
auto free_attributelist =
|
|
llvm::make_scope_exit([&] { free(startupinfoex.lpAttributeList); });
|
|
if (!InitializeProcThreadAttributeList(startupinfoex.lpAttributeList,
|
|
/*dwAttributeCount=*/1, /*dwFlags=*/0,
|
|
&attributelist_size)) {
|
|
error = Status(::GetLastError(), eErrorTypeWin32);
|
|
return HostProcess();
|
|
}
|
|
auto delete_attributelist = llvm::make_scope_exit(
|
|
[&] { DeleteProcThreadAttributeList(startupinfoex.lpAttributeList); });
|
|
for (size_t i = 0; i < launch_info.GetNumFileActions(); ++i) {
|
|
const FileAction *act = launch_info.GetFileActionAtIndex(i);
|
|
if (act->GetAction() == FileAction::eFileActionDuplicate &&
|
|
act->GetFD() == act->GetActionArgument())
|
|
inherited_handles.push_back(reinterpret_cast<HANDLE>(act->GetFD()));
|
|
}
|
|
if (!inherited_handles.empty()) {
|
|
if (!UpdateProcThreadAttribute(
|
|
startupinfoex.lpAttributeList, /*dwFlags=*/0,
|
|
PROC_THREAD_ATTRIBUTE_HANDLE_LIST, inherited_handles.data(),
|
|
inherited_handles.size() * sizeof(HANDLE),
|
|
/*lpPreviousValue=*/nullptr, /*lpReturnSize=*/nullptr)) {
|
|
error = Status(::GetLastError(), eErrorTypeWin32);
|
|
return HostProcess();
|
|
}
|
|
}
|
|
|
|
const char *hide_console_var =
|
|
getenv("LLDB_LAUNCH_INFERIORS_WITHOUT_CONSOLE");
|
|
if (hide_console_var &&
|
|
llvm::StringRef(hide_console_var).equals_insensitive("true")) {
|
|
startupinfo.dwFlags |= STARTF_USESHOWWINDOW;
|
|
startupinfo.wShowWindow = SW_HIDE;
|
|
}
|
|
|
|
DWORD flags = CREATE_NEW_CONSOLE | CREATE_UNICODE_ENVIRONMENT |
|
|
EXTENDED_STARTUPINFO_PRESENT;
|
|
if (launch_info.GetFlags().Test(eLaunchFlagDebug))
|
|
flags |= DEBUG_ONLY_THIS_PROCESS;
|
|
|
|
if (launch_info.GetFlags().Test(eLaunchFlagDisableSTDIO))
|
|
flags &= ~CREATE_NEW_CONSOLE;
|
|
|
|
LPVOID env_block = nullptr;
|
|
::CreateEnvironmentBuffer(launch_info.GetEnvironment(), environment);
|
|
env_block = environment.data();
|
|
|
|
executable = launch_info.GetExecutableFile().GetPath();
|
|
std::wstring wcommandLine;
|
|
GetFlattenedWindowsCommandString(launch_info.GetArguments(), wcommandLine);
|
|
|
|
std::wstring wexecutable, wworkingDirectory;
|
|
llvm::ConvertUTF8toWide(executable, wexecutable);
|
|
llvm::ConvertUTF8toWide(launch_info.GetWorkingDirectory().GetPath(),
|
|
wworkingDirectory);
|
|
// If the command line is empty, it's best to pass a null pointer to tell
|
|
// CreateProcessW to use the executable name as the command line. If the
|
|
// command line is not empty, its contents may be modified by CreateProcessW.
|
|
WCHAR *pwcommandLine = wcommandLine.empty() ? nullptr : &wcommandLine[0];
|
|
|
|
BOOL result = ::CreateProcessW(
|
|
wexecutable.c_str(), pwcommandLine, NULL, NULL,
|
|
/*bInheritHandles=*/!inherited_handles.empty(), flags, env_block,
|
|
wworkingDirectory.size() == 0 ? NULL : wworkingDirectory.c_str(),
|
|
reinterpret_cast<STARTUPINFO *>(&startupinfoex), &pi);
|
|
|
|
if (!result) {
|
|
// Call GetLastError before we make any other system calls.
|
|
error = Status(::GetLastError(), eErrorTypeWin32);
|
|
// Note that error 50 ("The request is not supported") will occur if you
|
|
// try debug a 64-bit inferior from a 32-bit LLDB.
|
|
}
|
|
|
|
if (result) {
|
|
// Do not call CloseHandle on pi.hProcess, since we want to pass that back
|
|
// through the HostProcess.
|
|
::CloseHandle(pi.hThread);
|
|
}
|
|
|
|
if (!result)
|
|
return HostProcess();
|
|
|
|
return HostProcess(pi.hProcess);
|
|
}
|
|
|
|
HANDLE
|
|
ProcessLauncherWindows::GetStdioHandle(const ProcessLaunchInfo &launch_info,
|
|
int fd) {
|
|
const FileAction *action = launch_info.GetFileActionForFD(fd);
|
|
if (action == nullptr)
|
|
return NULL;
|
|
SECURITY_ATTRIBUTES secattr = {};
|
|
secattr.nLength = sizeof(SECURITY_ATTRIBUTES);
|
|
secattr.bInheritHandle = TRUE;
|
|
|
|
llvm::StringRef path = action->GetPath();
|
|
DWORD access = 0;
|
|
DWORD share = FILE_SHARE_READ | FILE_SHARE_WRITE | FILE_SHARE_DELETE;
|
|
DWORD create = 0;
|
|
DWORD flags = 0;
|
|
if (fd == STDIN_FILENO) {
|
|
access = GENERIC_READ;
|
|
create = OPEN_EXISTING;
|
|
flags = FILE_ATTRIBUTE_READONLY;
|
|
}
|
|
if (fd == STDOUT_FILENO || fd == STDERR_FILENO) {
|
|
access = GENERIC_WRITE;
|
|
create = CREATE_ALWAYS;
|
|
if (fd == STDERR_FILENO)
|
|
flags = FILE_FLAG_WRITE_THROUGH;
|
|
}
|
|
|
|
std::wstring wpath;
|
|
llvm::ConvertUTF8toWide(path, wpath);
|
|
HANDLE result = ::CreateFileW(wpath.c_str(), access, share, &secattr, create,
|
|
flags, NULL);
|
|
return (result == INVALID_HANDLE_VALUE) ? NULL : result;
|
|
}
|