From cd56bf9f60e89c6303b0d2d3faf43406f83bf03f Mon Sep 17 00:00:00 2001
From: Alena Prokharchyk <Alena.Prokharchyk@citrix.com>
Date: Mon, 30 Jan 2012 15:26:43 -0800
Subject: [PATCH] Only Virtual Router can be specified as a provider for the
 Firewall service

---
 .../com/cloud/configuration/ConfigurationManagerImpl.java   | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/server/src/com/cloud/configuration/ConfigurationManagerImpl.java b/server/src/com/cloud/configuration/ConfigurationManagerImpl.java
index a2211b5ecb8..2a911c7cfb8 100755
--- a/server/src/com/cloud/configuration/ConfigurationManagerImpl.java
+++ b/server/src/com/cloud/configuration/ConfigurationManagerImpl.java
@@ -3014,6 +3014,12 @@ public class ConfigurationManagerImpl implements ConfigurationManager, Configura
                         if (provider == null) {
                             throw new InvalidParameterValueException("Invalid service provider: " + prvNameStr);
                         }
+                        
+                        //Only VirtualRouter can be specified as a firewall provider
+                        if (service == Service.Firewall && provider != Provider.VirtualRouter) {
+                        	throw new InvalidParameterValueException("Only Virtual router can be specified as a provider for the Firewall service");
+                        }
+                        
                         providers.add(provider);
                         
                         Set<Service> serviceSet = null;