commit c65046d4ce6f4f0663750273f340cc14e06f1b71
author Johnny Chen <johnny.chen@apple.com> Thu Mar 08 20:53:04 2012 +0000
committer Johnny Chen <johnny.chen@apple.com> Thu Mar 08 20:53:04 2012 +0000
tree 09c083a83c89e340533be7230c7723b04ef3d446
parent c227cd962515a765638a901c14d5cea4ca1b2e9d

Fixed a crasher when Xcode calls into ScriptInterpreterPython::ResetOutputFileHandle().
The Locker should only perform acquire/free lock operation, but no enter/leave session
at all.  Also added sanity checks for items passed to the PyDict_SetItemString() calls.


git-svn-id: https://llvm.org/svn/llvm-project/lldb/trunk@152337 91177308-0d34-0410-b5e6-96231b3b80d8

source/Interpreter/ScriptInterpreterPython.cpp

diff --git a/source/Interpreter/ScriptInterpreterPython.cpp b/source/Interpreter/ScriptInterpreterPython.cpp
index 1653e92..7a9140f 100644
--- a/source/Interpreter/ScriptInterpreterPython.cpp
+++ b/source/Interpreter/ScriptInterpreterPython.cpp
@@ -240,6 +240,8 @@
     m_embedded_thread_input_reader_sp (),
     m_dbg_stdout (interpreter.GetDebugger().GetOutputFile().GetStream()),
     m_new_sysout (NULL),
+    m_old_sysout (NULL),
+    m_old_syserr (NULL),
     m_run_one_line (NULL),
     m_dictionary_name (interpreter.GetDebugger().GetInstanceName().AsCString()),
     m_terminal_state (),
@@ -332,8 +334,10 @@
         
     m_dbg_stdout = fh;
 
-    Locker py_lock(this);
-    
+    Locker locker(this,
+                  ScriptInterpreterPython::Locker::AcquireLock,
+                  ScriptInterpreterPython::Locker::FreeAcquiredLock);
+
     m_new_sysout = PyFile_FromFile (m_dbg_stdout, (char *) "", (char *) "w", _check_and_flush);
 }
 
@@ -365,8 +369,10 @@
 
     if (m_new_sysout && sysmod && sysdict)
     {
-        PyDict_SetItemString (sysdict, "stdout", (PyObject*)m_old_sysout);
-        PyDict_SetItemString (sysdict, "stderr", (PyObject*)m_old_sysout);
+        if (m_old_sysout)
+            PyDict_SetItemString (sysdict, "stdout", (PyObject*)m_old_sysout);
+        if (m_old_syserr)
+            PyDict_SetItemString (sysdict, "stderr", (PyObject*)m_old_syserr);
     }
 
     m_session_is_active = false;
@@ -410,8 +416,11 @@
     {
         m_old_sysout = PyDict_GetItemString(sysdict, "stdout");
         m_old_syserr = PyDict_GetItemString(sysdict, "stderr");
-        PyDict_SetItemString (sysdict, "stdout", (PyObject*)m_new_sysout);
-        PyDict_SetItemString (sysdict, "stderr", (PyObject*)m_new_sysout);
+        if (m_new_sysout)
+        {
+            PyDict_SetItemString (sysdict, "stdout", (PyObject*)m_new_sysout);
+            PyDict_SetItemString (sysdict, "stderr", (PyObject*)m_new_sysout);
+        }
     }
 
     if (PyErr_Occurred())