[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