Merge pull request #252 from treviza153/bugfix/CheckPrefix

Checking prefix

networkapi/ambiente/models.py

@@ -1929,6 +1929,19 @@ def check_cidr(self, environment, network):
 
         return False
 
+    def check_prefix(self, network_address, subnet_mask):
+        """
+        check if subnet mask is correct, based on network mask.
+
+        :param network_address: environment cidr
+        :param subnet_mask: environment cidr subnet mask
+        :return: boolean
+        """
+
+        network = ipaddr.IPNetwork(network_address)
+
+        return True if int(network.prefixlen) <= int(subnet_mask) else False
+
     def check_duplicated_cidr(self, environment, network):
         """
         check if the network overlaps another cidr from another environment.
@@ -2020,6 +2033,9 @@ def post(self, env_cidr):
             self.ip_version = env_cidr.get('ip_version')
             self.subnet_mask = env_cidr.get('subnet_mask')
 
+            if not self.check_prefix(self.network, self.subnet_mask):
+                raise CIDRErrorV3("The prefix %s is not valid for the network %s" % (self.subnet_mask, self.network))
+
             objects = EnvCIDR.objects.filter(id_env=int(env_cidr.get('environment')))
             for obj in objects:
                 if ipaddr.IPNetwork(obj.network).overlaps(ipaddr.IPNetwork(self.network)):