Francesco Romani has submitted this change and it was merged.

Change subject: virt: Set cloexec flag on channel sockets
......................................................................


virt: Set cloexec flag on channel sockets

To counter act potential leakage of open handles, this patch is setting the
CLOEXEC flag on the sockets used for the guest agent channels.
It is known that this is not totally race free however it will still improve
the number of file handles leaked.

Change-Id: Ic4a452f7837267434bc836fced4c2efd92d745cf
Bug-Url: https://bugzilla.redhat.com/1297414
Signed-off-by: Vinzenz Feenstra <[email protected]>
Reviewed-on: https://gerrit.ovirt.org/42570
Reviewed-by: Francesco Romani <[email protected]>
Continuous-Integration: Jenkins CI
Reviewed-by: Nir Soffer <[email protected]>
Reviewed-on: https://gerrit.ovirt.org/51878
Continuous-Integration: Francesco Romani <[email protected]>
---
M vdsm/virt/guestagent.py
1 file changed, 10 insertions(+), 3 deletions(-)

Approvals:
  Nir Soffer: Looks good to me, but someone else must approve
  Vinzenz Feenstra: Verified
  Francesco Romani: Looks good to me, approved; Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/51878
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic4a452f7837267434bc836fced4c2efd92d745cf
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Vinzenz Feenstra <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Vinzenz Feenstra <[email protected]>
Gerrit-Reviewer: gerrit-hooks <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to