Merge pull request #281 from ioito/hotfix/qx-azure-div-zero

fix(region): avoid div zero
yunionio · May 15, 2023 · 3a7171b · 3a7171b
2 parents 77babf2 + 5362e16
commit 3a7171b
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/pkg/multicloud/azure/securitygroup.go b/pkg/multicloud/azure/securitygroup.go
@@ -306,9 +306,9 @@ func (region *SRegion) CreateSecurityGroup(opts *cloudprovider.SecurityGroupCrea
 	}
 
 	sortRules := func(rules []cloudprovider.SecurityRule) []SecurityRules {
-		names := []string{}
 		securityRules := []SecurityRules{}
-		offset := (4096-100)/len(rules) - 1
+		names := []string{}
+		offset := (4096 - 100) / (len(rules) + 1) //avoid div zero
 		for i := 0; i < len(rules); i++ {
 			rules[i].Priority = 4096 - offset*i
 			rule := convertSecurityGroupRule(rules[i])