Set error status when failed to catch stop after launch

Tamas Berghammer · Tamas Berghammer · commit 04e6314cf4da · 2015-02-23T10:59:54.000Z
Process::Launch try to catch a stop signal after launching a process. If it is unsuccessful it destroy the process but previously still reported that the process launched successfully. This behavior caused a deadlock. With thic change the process launch error reported correctly. Differential revision: http://reviews.llvm.org/D7784 llvm-svn: 230212
diff --git a/lldb/source/Target/Process.cpp b/lldb/source/Target/Process.cpp
@@ -3101,6 +3101,7 @@ Process::Launch (ProcessLaunchInfo &launch_info)
                     {
                         // We were able to launch the process, but we failed to
                         // catch the initial stop.
+                        error.SetErrorString ("failed to catch stop after launch");
                         SetExitStatus (0, "failed to catch stop after launch");
                         Destroy();
                     }

Original file line number	Diff line number	Diff line change
`@@ -3101,6 +3101,7 @@ Process::Launch (ProcessLaunchInfo &launch_info)`
`3101`	`3101`	`{`
`3102`	`3102`	`// We were able to launch the process, but we failed to`
`3103`	`3103`	`// catch the initial stop.`
	`3104`	`+ error.SetErrorString ("failed to catch stop after launch");`
`3104`	`3105`	`SetExitStatus (0, "failed to catch stop after launch");`
`3105`	`3106`	`Destroy();`
`3106`	`3107`	`}`