X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=sfa%2Fopenstack%2Fsecurity_group.py;fp=sfa%2Fopenstack%2Fsecurity_group.py;h=99f07f03d1d8876226e529c29b7d85b7660c6cd8;hb=e0579aef839ee7bdd4cf13134eb468163cbc8437;hp=33c627433f4f582086382585bfdf37e2ffe70cf7;hpb=3d73576ce211a3f4fcf6bf58bcb30e10e12a01e0;p=sfa.git

diff --git a/sfa/openstack/security_group.py b/sfa/openstack/security_group.py
index 33c62743..99f07f03 100644
--- a/sfa/openstack/security_group.py
+++ b/sfa/openstack/security_group.py
@@ -30,8 +30,6 @@ class SecurityGroup:
                 to_port = int(ports[1])
             else:
                 from_port = to_port = int(ports[0])
-        else:
-            from_port = to_port = None
         return (from_port, to_port)
 
     def _validate_icmp_type_code(self, icmp_type_code):
@@ -53,7 +51,7 @@ class SecurityGroup:
 
         from_port, to_port = self._validate_port_range(port_range)
         icmp_type = self._validate_icmp_type_code(icmp_type_code)
-        if icmp_type:
+        if icmp_type and icmp_type[0] and icmp_type[1]:
             from_port, to_port = icmp_type[0], icmp_type[1]
 
         if group_name: