[Lldb-commits] [lldb] r163163 - /lldb/trunk/source/Target/Process.cpp

Greg Clayton gclayton at apple.com
Tue Sep 4 13:29:05 PDT 2012


Author: gclayton
Date: Tue Sep  4 15:29:05 2012
New Revision: 163163

URL: http://llvm.org/viewvc/llvm-project?rev=163163&view=rev
Log:
Modified patch from Matt Kopec that fixes a process launch issue on linux where we wouldn't acquire the process run lock.


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=163163&r1=163162&r2=163163&view=diff
==============================================================================
--- lldb/trunk/source/Target/Process.cpp (original)
+++ lldb/trunk/source/Target/Process.cpp Tue Sep  4 15:29:05 2012
@@ -2561,8 +2561,16 @@
                 SetPublicState (eStateLaunching);
                 m_should_detach = false;
 
-                // Now launch using these arguments.
-                error = DoLaunch (exe_module, launch_info);
+                if (m_run_lock.WriteTryLock())
+                {
+                    // Now launch using these arguments.
+                    error = DoLaunch (exe_module, launch_info);
+                }
+                else
+                {
+                    // This shouldn't happen
+                    error.SetErrorString("failed to acquire process run lock");
+                }
 
                 if (error.Fail())
                 {





More information about the lldb-commits mailing list