From 5dbd33d8ab6aa345fa603329a92990db53ad754c Mon Sep 17 00:00:00 2001
From: Sheng Yang <sheng.yang@citrix.com>
Date: Tue, 12 Feb 2013 20:21:20 -0800
Subject: [PATCH] Add missing break to applyRules in FirewallManager

Otherwise it would falling through PortForwarding rule.

Seems like a typo after network refactor.
---
 server/src/com/cloud/network/firewall/FirewallManagerImpl.java | 1 +
 1 file changed, 1 insertion(+)

diff --git a/server/src/com/cloud/network/firewall/FirewallManagerImpl.java b/server/src/com/cloud/network/firewall/FirewallManagerImpl.java
index 0d17ba720a9..080f7b0edf6 100644
--- a/server/src/com/cloud/network/firewall/FirewallManagerImpl.java
+++ b/server/src/com/cloud/network/firewall/FirewallManagerImpl.java
@@ -545,6 +545,7 @@ public class FirewallManagerImpl extends ManagerBase implements FirewallService,
                 if (handled)
                     break;
             }
+            break;
         case PortForwarding:
             for (PortForwardingServiceProvider element: _pfElements) {
                 Network.Provider provider = element.getProvider();