Juan Hernandez has uploaded a new change for review.

Change subject: Explicitly shutdown  m2crypto socket
......................................................................

Explicitly shutdown  m2crypto socket

Aparently some versions of the m2crypto library don't shutdown correctly
underlying sockets when a SSL connection is closed. This patch changes
the SSLSocket class so that it explicitly shuts down and closes the
underlying socket when  when the connection is closed.

Change-Id: Ie1a471aaccb32554b94340ebfb92b9d7ba14407a
Signed-off-by: Juan Hernandez <[email protected]>
---
M vdsm/SecureXMLRPCServer.py
1 file changed, 4 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/72/10972/1

diff --git a/vdsm/SecureXMLRPCServer.py b/vdsm/SecureXMLRPCServer.py
index c674acb..78c4b72 100644
--- a/vdsm/SecureXMLRPCServer.py
+++ b/vdsm/SecureXMLRPCServer.py
@@ -57,6 +57,10 @@
     def gettimeout(self):
         return self.connection.socket.gettimeout()
 
+    def close(self):
+        self.connection.shutdown(socket.SHUT_RDWR)
+        self.connection.close()
+
     def __getattr__(self, name):
         # This is how we delegate all the rest of the methods to the
         # underlying SSL connection:


--
To view, visit http://gerrit.ovirt.org/10972
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie1a471aaccb32554b94340ebfb92b9d7ba14407a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to