[PATCH virt-manager] tunnels: do not close unowned fd

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The fd is handed to spice-gtk and gtk-vnc. They will close it when no
longer needed. Double closing leads to various race issues, since the
same fd may be opened for a different usage.

Fixes:
https://bugzilla.redhat.com/show_bug.cgi?id=1081227
---
 virtManager/sshtunnels.py | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/virtManager/sshtunnels.py b/virtManager/sshtunnels.py
index 8443694..2827295 100644
--- a/virtManager/sshtunnels.py
+++ b/virtManager/sshtunnels.py
@@ -153,10 +153,7 @@ class _Tunnel(object):
                       self.outfd and self.outfd.fileno() or self._outfds,
                       self.errfd and self.errfd.fileno() or self._errfds)
 
-        if self.outfd:
-            self.outfd.close()
-        elif self._outfds:
-            self._outfds[0].close()
+        if self._outfds:
             self._outfds[1].close()
         self.outfd = None
         self._outfds = None
-- 
1.9.3

_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list




[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux