Add missing break to applyRules in FirewallManager

Sheng Yang · Sheng Yang · commit 15e466c56dd1 · 2013-02-12T20:27:31.000-08:00
Otherwise it would falling through PortForwarding rule.

Seems like a typo after network refactor.
diff --git a/server/src/com/cloud/network/firewall/FirewallManagerImpl.java b/server/src/com/cloud/network/firewall/FirewallManagerImpl.java
@@ -545,6 +545,7 @@ public  boolean applyRules(Network network, Purpose purpose, List<? extends Fire
                 if (handled)
                     break;
             }
+            break;
         case PortForwarding:
             for (PortForwardingServiceProvider element: _pfElements) {
                 Network.Provider provider = element.getProvider();

Original file line number	Diff line number	Diff line change
`@@ -545,6 +545,7 @@ public boolean applyRules(Network network, Purpose purpose, List<? extends Fire`
`545`	`545`	`if (handled)`
`546`	`546`	`break;`
`547`	`547`	`}`
	`548`	`+ break;`
`548`	`549`	`case PortForwarding:`
`549`	`550`	`for (PortForwardingServiceProvider element: _pfElements) {`
`550`	`551`	`Network.Provider provider = element.getProvider();`