[Lldb-commits] [lldb] r230212 - Set error status when failed to catch stop after launch

Tamas Berghammer tberghammer at google.com
Mon Feb 23 02:59:54 PST 2015


Author: tberghammer
Date: Mon Feb 23 04:59:54 2015
New Revision: 230212

URL: http://llvm.org/viewvc/llvm-project?rev=230212&view=rev
Log:
Set error status when failed to catch stop after launch

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

Modified:
    lldb/trunk/source/Target/Process.cpp

Modified: lldb/trunk/source/Target/Process.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Target/Process.cpp?rev=230212&r1=230211&r2=230212&view=diff
==============================================================================
--- lldb/trunk/source/Target/Process.cpp (original)
+++ lldb/trunk/source/Target/Process.cpp Mon Feb 23 04:59:54 2015
@@ -3101,6 +3101,7 @@ Process::Launch (ProcessLaunchInfo &laun
                     {
                         // 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();
                     }





More information about the lldb-commits mailing list