Change in vdsm[master]: Explicitly shutdown m2crypto socket

juan.hernandez at redhat.com juan.hernandez at redhat.com
Wed Jan 16 19:19:45 UTC 2013


Juan Hernandez has posted comments on this change.

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


Patch Set 1: Verified

(1 inline comment)

Verified in RHEL6.

....................................................
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
Sorry, the last comment was wrong. The situation is that the RHEL6 machine with the with the original patch applied *doesn't* work, vdsClient hangs as Andrey described.

With the original pach plus this patch it vdsClient works correctly.

The communication from the engine does work correctly in both cases.

So I can verify this change in RHEL6.
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 <juan.hernandez at redhat.com>
Gerrit-Reviewer: Andrey Gordeev <dreyou at gmail.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list