From: Tony Mack <tmack@cs.princeton.edu>
Date: Tue, 11 May 2010 17:34:09 +0000 (+0000)
Subject: merging bugfix from trunk
X-Git-Tag: sfa-0.9-11~5
X-Git-Url: http://git.onelab.eu/?p=sfa.git;a=commitdiff_plain;h=388315b1b7e94e09a1b717d38693a03be5e23058

merging bugfix from trunk
---

diff --git a/sfa/server/interface.py b/sfa/server/interface.py
index 05034795..4ae8e9fc 100644
--- a/sfa/server/interface.py
+++ b/sfa/server/interface.py
@@ -61,8 +61,17 @@ class Interfaces(dict):
         if not isinstance(interfaces, list):
             interfaces = [self.interfaces]
         self.interfaces = {}
+        required_fields = self.default_fields.keys()
         for interface in interfaces:
-            self.interfaces[interface['hrn']] = interface
+            valid = True
+            # skp any interface definition that has a null hrn, 
+            # address or port
+            for field in required_fields:
+                if field not in interface or not interface[field]:
+                    valid = False
+                    break
+            if valid:     
+                self.interfaces[interface['hrn']] = interface
 
 
     def sync_interfaces(self):