From b07239f8fe7a50939c52fff7fdb690861dfde916 Mon Sep 17 00:00:00 2001 From: Vasil Kolev Date: Sat, 10 Dec 2016 18:02:47 +0100 Subject: fix for #117 (voctomix leaking file descriptors) On connection close voctocore doesn't close the sockets, leading to a leak. --- voctocore/lib/tcpmulticonnection.py | 1 + voctocore/lib/tcpsingleconnection.py | 1 + 2 files changed, 2 insertions(+) diff --git a/voctocore/lib/tcpmulticonnection.py b/voctocore/lib/tcpmulticonnection.py index ac228a3..e7ab700 100644 --- a/voctocore/lib/tcpmulticonnection.py +++ b/voctocore/lib/tcpmulticonnection.py @@ -43,6 +43,7 @@ class TCPMultiConnection(object): def close_connection(self, conn): if conn in self.currentConnections: + conn.close() del(self.currentConnections[conn]) self.log.info('Now %u Receiver connected', len(self.currentConnections)) diff --git a/voctocore/lib/tcpsingleconnection.py b/voctocore/lib/tcpsingleconnection.py index 3f589a7..62a4b1f 100644 --- a/voctocore/lib/tcpsingleconnection.py +++ b/voctocore/lib/tcpsingleconnection.py @@ -41,5 +41,6 @@ class TCPSingleConnection(object): return True def close_connection(self): + self.currentConnection.close() self.currentConnection = None self.log.info('Connection closed') -- cgit v1.2.3