http://reviews.llvm.org/D8009
Files:
include/lldb/Host/android/HostInfoAndroid.h
source/Host/android/HostInfoAndroid.cpp
source/Host/android/ProcessLauncherAndroid.cpp
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
Index: include/lldb/Host/android/HostInfoAndroid.h
===================================================================
--- include/lldb/Host/android/HostInfoAndroid.h
+++ include/lldb/Host/android/HostInfoAndroid.h
@@ -19,10 +19,8 @@
{
friend class HostInfoBase;
- private:
- // Static class, unconstructable.
- HostInfoAndroid();
- ~HostInfoAndroid();
+ public:
+ static FileSpec GetDefaultShell();
protected:
static void ComputeHostArchitectureSupport(ArchSpec &arch_32, ArchSpec &arch_64);
Index: source/Host/android/HostInfoAndroid.cpp
===================================================================
--- source/Host/android/HostInfoAndroid.cpp
+++ source/Host/android/HostInfoAndroid.cpp
@@ -33,3 +33,9 @@
file_spec.GetDirectory() = HostInfoLinux::GetProgramFileSpec().GetDirectory();
return (bool)file_spec.GetDirectory();
}
+
+FileSpec
+HostInfoAndroid::GetDefaultShell()
+{
+ return FileSpec("/system/bin/sh", false);
+}
Index: source/Host/android/ProcessLauncherAndroid.cpp
===================================================================
--- source/Host/android/ProcessLauncherAndroid.cpp
+++ source/Host/android/ProcessLauncherAndroid.cpp
@@ -18,31 +18,79 @@
using namespace lldb;
using namespace lldb_private;
+static bool
+DupDescriptor(const char *path, int fd, int flags)
+{
+ int target_fd = ::open(path, flags, 0666);
+
+ if (target_fd == -1)
+ return false;
+
+ if (::dup2(target_fd, fd) == -1)
+ return false;
+
+ return (::close(target_fd) == -1) ? false : true;
+}
+
+// If there is no PATH variable specified inside the environment then set the path to /system/bin.
+// It is required because the default path used by execve() is wrong on android.
+static void
+FixupEnvironment(Args& env)
+{
+ static const char* path = "PATH=";
+ static const int path_len = ::strlen(path);
+ for (const char** args = env.GetConstArgumentVector(); *args; ++args)
+ if (::strncmp(path, *args, path_len) == 0)
+ return;
+ env.AppendArgument("PATH=/system/bin");
+}
+
HostProcess
ProcessLauncherAndroid::LaunchProcess(const ProcessLaunchInfo &launch_info, Error &error)
{
// TODO: Handle other launch parameters specified in launc_info
char exe_path[PATH_MAX];
launch_info.GetExecutableFile().GetPath(exe_path, sizeof(exe_path));
- const size_t err_len = 1024;
- char err_str[err_len];
-
- lldb::pid_t pid = ::fork ();
- if (pid < 0)
+ lldb::pid_t pid = ::fork();
+ if (pid == static_cast<lldb::pid_t>(-1))
{
// Fork failed
error.SetErrorStringWithFormat("Fork failed with error message: %s", strerror(errno));
return HostProcess(LLDB_INVALID_PROCESS_ID);
}
else if (pid == 0)
{
+ if (const lldb_private::FileAction *file_action = launch_info.GetFileActionForFD(STDIN_FILENO)) {
+ const char* path = file_action->GetPath();
+ if (path && ::strlen(path))
+ if (!DupDescriptor(path, STDIN_FILENO, O_RDONLY))
+ exit(-1);
+ }
+
+ if (const lldb_private::FileAction *file_action = launch_info.GetFileActionForFD(STDOUT_FILENO)) {
+ const char* path = file_action->GetPath();
+ if (path && ::strlen(path))
+ if (!DupDescriptor(path, STDOUT_FILENO, O_WRONLY | O_CREAT | O_TRUNC))
+ exit(-1);
+ }
+
+ if (const lldb_private::FileAction *file_action = launch_info.GetFileActionForFD(STDERR_FILENO)) {
+ const char* path = file_action->GetPath();
+ if (path && ::strlen(path))
+ if (!DupDescriptor(path, STDERR_FILENO, O_WRONLY | O_CREAT | O_TRUNC))
+ exit(-1);
+ }
+
// Child process
const char **argv = launch_info.GetArguments().GetConstArgumentVector();
- const char **envp = launch_info.GetEnvironmentEntries().GetConstArgumentVector();
+
+ Args env = launch_info.GetEnvironmentEntries();
+ FixupEnvironment(env);
+ const char **envp = env.GetConstArgumentVector();
+
const char *working_dir = launch_info.GetWorkingDirectory();
-
if (working_dir != nullptr && working_dir[0])
{
if (::chdir(working_dir) != 0)
_______________________________________________
lldb-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
