[Lldb-commits] [PATCH] Fix socket leak in lldb-server platform

Vince Harron vince at nethacker.com
Sun Mar 29 23:02:02 PDT 2015


Hi clayborg, flackr, tberghammer,

lldb-server server process was holding on to a connection to every
client that connected.  This continued until the process hit it's
file limit and exited.

http://reviews.llvm.org/D8697

Files:
  tools/lldb-server/lldb-platform.cpp

Index: tools/lldb-server/lldb-platform.cpp
===================================================================
--- tools/lldb-server/lldb-platform.cpp
+++ tools/lldb-server/lldb-platform.cpp
@@ -287,6 +287,10 @@
             while (waitpid(-1, nullptr, WNOHANG) > 0);
             if (fork())
             {
+                // Parent doesn't need a connection to the lldb client
+                delete socket;
+                socket = nullptr;
+
                 // Parent will continue to listen for new connections.
                 continue;
             }

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D8697.22868.patch
Type: text/x-patch
Size: 563 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20150330/819de159/attachment.bin>


More information about the lldb-commits mailing list