ACL list can be empty, thereby response being empty, not even the "ne…

…tworkacl" entry
MissionCriticalCloud · Aug 16, 2016 · 54b9f07 · 54b9f07
1 parent 7d8202c
commit 54b9f07
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/lib/vagrant-cloudstack/action/run_instance.rb b/lib/vagrant-cloudstack/action/run_instance.rb
@@ -679,7 +679,9 @@ def create_firewall_rule(rule)
               aclid:  network.details['aclid']
             })
             number = 0
-            resp["listnetworkaclsresponse"]["networkacl"].each{ |ace| number = [number, ace["number"]].max }
+            if resp["listnetworkaclsresponse"]["count"] > 0
+              resp["listnetworkaclsresponse"]["networkacl"].each{ |ace| number = [number, ace["number"]].max }
+            end
             number = number+1
 
             command_string  = 'createNetworkACL'