[PATCH rhel7-branch] network: fix filtering of libvirt devices (check for None values) (#1298444)

[Radek Vykydal]

Related: rhbz#1298444
---
 pyanaconda/ui/gui/spokes/network.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pyanaconda/ui/gui/spokes/network.py b/pyanaconda/ui/gui/spokes/network.py
index 80f9a55..052a375 100644
--- a/pyanaconda/ui/gui/spokes/network.py
+++ b/pyanaconda/ui/gui/spokes/network.py
@@ -437,7 +437,7 @@ class NetworkControlBox(GObject.GObject):
             log.debug("network: GUI, not adding connection %s, already in list", uuid)
             return False
         dev_cfg = DeviceConfiguration(con_uuid=uuid)
-        if network.is_libvirt_device(dev_cfg.get_iface()):
+        if network.is_libvirt_device(dev_cfg.get_iface() or ""):
             log.debug("network: GUI, not adding %s", dev_cfg.get_iface())
             return False
         if dev_cfg.setting_value("connection", "read-only"):

[Vratislav Podzimek]

On Mon, 2016-05-30 at 17:08 +0200, Radek Vykydal wrote:
> Related: rhbz#1298444
>
> pyanaconda/ui/gui/spokes/network.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/pyanaconda/ui/gui/spokes/network.py b/pyanaconda/ui/gui/spokes/network.py
> index 80f9a55..052a375 100644
> --- a/pyanaconda/ui/gui/spokes/network.py
> +++ b/pyanaconda/ui/gui/spokes/network.py
> @@ -437,7 +437,7 @@ class NetworkControlBox(GObject.GObject):
>              log.debug("network: GUI, not adding connection %s, already in list", uuid)
>              return False
>          dev_cfg = DeviceConfiguration(con_uuid=uuid)
> -        if network.is_libvirt_device(dev_cfg.get_iface()):
> +        if network.is_libvirt_device(dev_cfg.get_iface() or ""):
>              log.debug("network: GUI, not adding %s", dev_cfg.get_iface())
>              return False
>          if dev_cfg.setting_value("connection", "read-only"):

ACK.