diff --git a/api/src/org/apache/cloudstack/api/response/FirewallResponse.java b/api/src/org/apache/cloudstack/api/response/FirewallResponse.java
index 368f732b197..462bd1bd1b0 100644
--- a/api/src/org/apache/cloudstack/api/response/FirewallResponse.java
+++ b/api/src/org/apache/cloudstack/api/response/FirewallResponse.java
@@ -87,11 +87,11 @@ public class FirewallResponse extends BaseResponse {
         this.protocol = protocol;
     }
 
-    public void setStartPort(String startPort) {
+    public void setStartPort(Integer startPort) {
         this.startPort = startPort;
     }
 
-    public void setEndPort(String endPort) {
+    public void setEndPort(Integer endPort) {
         this.endPort = endPort;
     }
 
diff --git a/server/src/com/cloud/api/ApiResponseHelper.java b/server/src/com/cloud/api/ApiResponseHelper.java
index 06c68c4e1b2..6ee7c26d581 100644
--- a/server/src/com/cloud/api/ApiResponseHelper.java
+++ b/server/src/com/cloud/api/ApiResponseHelper.java
@@ -2124,11 +2124,11 @@ public class ApiResponseHelper implements ResponseGenerator {
         response.setId(fwRule.getUuid());
         response.setProtocol(fwRule.getProtocol());
         if (fwRule.getSourcePortStart() != null) {
-            response.setStartPort(Integer.toString(fwRule.getSourcePortStart()));
+            response.setStartPort(fwRule.getSourcePortStart());
         }
 
         if (fwRule.getSourcePortEnd() != null) {
-            response.setEndPort(Integer.toString(fwRule.getSourcePortEnd()));
+            response.setEndPort(fwRule.getSourcePortEnd());
         }
 
         List<String> cidrs = ApiDBUtils.findFirewallSourceCidrs(fwRule.getId());