You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by ed...@apache.org on 2013/02/14 02:11:27 UTC
[1/50] [abbrv] git commit: refs/heads/storage_refactor - Add missing
break to applyRules in FirewallManager
Add missing break to applyRules in FirewallManager
Otherwise it would falling through PortForwarding rule.
Seems like a typo after network refactor.
Project: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/commit/15e466c5
Tree: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/tree/15e466c5
Diff: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/diff/15e466c5
Branch: refs/heads/storage_refactor
Commit: 15e466c56dd195d3a4a5d9d63c320fdca4b8a836
Parents: 2776ad1
Author: Sheng Yang <sh...@citrix.com>
Authored: Tue Feb 12 20:21:20 2013 -0800
Committer: Sheng Yang <sh...@citrix.com>
Committed: Tue Feb 12 20:27:31 2013 -0800
----------------------------------------------------------------------
.../network/firewall/FirewallManagerImpl.java | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/blob/15e466c5/server/src/com/cloud/network/firewall/FirewallManagerImpl.java
----------------------------------------------------------------------
diff --git a/server/src/com/cloud/network/firewall/FirewallManagerImpl.java b/server/src/com/cloud/network/firewall/FirewallManagerImpl.java
index 0d17ba7..080f7b0 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();