[Lldb-commits] [lldb] r200049 - Fix crash on Linux if sem_wait() is interrupted.

Todd Fiala tfiala at google.com
Fri Jan 24 14:59:22 PST 2014


Author: tfiala
Date: Fri Jan 24 16:59:22 2014
New Revision: 200049

URL: http://llvm.org/viewvc/llvm-project?rev=200049&view=rev
Log:
Fix crash on Linux if sem_wait() is interrupted.

Patch by Andrew MacPherson.

Modified:
    lldb/trunk/source/Plugins/Process/Linux/ProcessMonitor.cpp

Modified: lldb/trunk/source/Plugins/Process/Linux/ProcessMonitor.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/Process/Linux/ProcessMonitor.cpp?rev=200049&r1=200048&r2=200049&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/Process/Linux/ProcessMonitor.cpp (original)
+++ lldb/trunk/source/Plugins/Process/Linux/ProcessMonitor.cpp Fri Jan 24 16:59:22 2014
@@ -2062,7 +2062,12 @@ ProcessMonitor::ServeOperation(Operation
     for(;;)
     {
         // wait for next pending operation
-        sem_wait(&monitor->m_operation_pending);
+        if (sem_wait(&monitor->m_operation_pending))
+        {
+            if (errno == EINTR)
+                continue;
+            assert(false && "Unexpected errno from sem_wait");
+        }
 
         monitor->m_operation->Execute(monitor);
 
@@ -2082,7 +2087,12 @@ ProcessMonitor::DoOperation(Operation *o
     sem_post(&m_operation_pending);
 
     // wait for operation to complete
-    sem_wait(&m_operation_done);
+    while (sem_wait(&m_operation_done))
+    {
+        if (errno == EINTR)
+            continue;
+        assert(false && "Unexpected errno from sem_wait");
+    }
 }
 
 size_t





More information about the lldb-commits mailing list