Change in vdsm[ovirt-3.3]: Avoid M2Crypto races

danken at redhat.com danken at redhat.com
Thu Nov 7 09:25:48 UTC 2013


Dan Kenigsberg has submitted this change and it was merged.

Change subject: Avoid M2Crypto races
......................................................................


Avoid M2Crypto races

As documented in
http://www.heikkitoivonen.net/m2crypto/api/M2Crypto.threading-module.html
and painfully recognized long ago in https://bugzilla.redhat.com/482420,
using M2Crypto in a multi-threaded server such as Vdsm requires explicit
initialization.

Bug-Url: https://bugzilla.redhat.com/1026556
Change-Id: I83a3ab41a7001758bd37b0124904fbcd0387ffbf
Signed-off-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/20813
Reviewed-by: Juan Hernandez <juan.hernandez at redhat.com>
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/20936
---
M lib/vdsm/SecureXMLRPCServer.py
1 file changed, 3 insertions(+), 0 deletions(-)

Approvals:
  Dan Kenigsberg: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I83a3ab41a7001758bd37b0124904fbcd0387ffbf
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list