Skip to content

Commit

Permalink
Fixed gcc linker error 58
Browse files Browse the repository at this point in the history
  • Loading branch information
tmiguelf committed Feb 23, 2024
1 parent 9d43b27 commit 4c676b9
Showing 1 changed file with 20 additions and 10 deletions.
30 changes: 20 additions & 10 deletions googletest/src/gtest-death-test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -630,13 +630,23 @@ bool DeathTestImpl::Passed(bool status_ok) {
#ifndef GTEST_OS_WINDOWS
// Note: The return value points into args, so the return value's lifetime is
// bound to that of args.
static std::unique_ptr<char*[]> CreateArgvFromArgs(
std::vector<std::string>& args) {
auto result = std::make_unique<char*[]>(args.size() + 1);
for (size_t i = 0; i < args.size(); ++i) {
result[i] = &args[i][0];
static std::vector<char*> CreateArgvFromArgs(
std::vector<std::string>& args){
std::vector<char*> result;

size_t arg_size = args.size();
//if(arg_size >= std::numeric_limits<ssize_t>::max()){
// result.resize(1);
// result[0] = nullptr;
// return result;
//}

result.resize(arg_size + 1);
for (size_t i = 0; i < arg_size; ++i) {
result[i] = args[i].data();
}
result[args.size()] = nullptr; // extra null terminator

result[arg_size] = nullptr; // extra null terminator
return result;
}
#endif
Expand Down Expand Up @@ -1036,8 +1046,8 @@ DeathTest::TestRole FuchsiaDeathTest::AssumeRole() {
// "Fuchsia Test Component" which contains a "Fuchsia Component Manifest")
// Launching processes is a privileged operation in Fuchsia, and the
// declaration indicates that the ability is required for the component.
std::unique_ptr<char*[]> argv = CreateArgvFromArgs(args);
status = fdio_spawn_etc(child_job, FDIO_SPAWN_CLONE_ALL, argv[0], argv.get(),
std::vector<char*> argv = CreateArgvFromArgs(args);
status = fdio_spawn_etc(child_job, FDIO_SPAWN_CLONE_ALL, argv[0], argv.data(),
nullptr, 2, spawn_actions,
child_process_.reset_and_get_address(), nullptr);
GTEST_DEATH_TEST_CHECK_(status == ZX_OK);
Expand Down Expand Up @@ -1388,8 +1398,8 @@ DeathTest::TestRole ExecDeathTest::AssumeRole() {
// is necessary.
FlushInfoLog();

std::unique_ptr<char*[]> argv = CreateArgvFromArgs(args);
const pid_t child_pid = ExecDeathTestSpawnChild(argv.get(), pipe_fd[0]);
std::vector<char*> argv = CreateArgvFromArgs(args);
const pid_t child_pid = ExecDeathTestSpawnChild(argv.data(), pipe_fd[0]);
GTEST_DEATH_TEST_CHECK_SYSCALL_(close(pipe_fd[1]));
set_child_pid(child_pid);
set_read_fd(pipe_fd[0]);
Expand Down

0 comments on commit 4c676b9

Please sign in to comment.