Dan Kenigsberg has posted comments on this change. Change subject: Explicitly shutdown m2crypto socket ......................................................................
Patch Set 1: I would prefer that you didn't submit this (1 inline comment) .................................................... Commit Message Line 5: CommitDate: 2013-01-14 10:33:04 +0100 Line 6: Line 7: Explicitly shutdown m2crypto socket Line 8: Line 9: Aparently some versions of the m2crypto library don't shutdown correctly any idea which versions are these? has it been fixed upstream? maybe we could just backport the fix to el6's m2crypto? could you provide some references to the issues you describe here? Line 10: underlying sockets when a SSL connection is closed. This patch changes Line 11: the SSLSocket class so that it explicitly shuts down and closes the Line 12: underlying socket when when the connection is closed. Line 13: -- To view, visit http://gerrit.ovirt.org/10972 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ie1a471aaccb32554b94340ebfb92b9d7ba14407a Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <[email protected]> Gerrit-Reviewer: Andrey Gordeev <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
