From 14faff534d1b8a7ef5000d8553f30be10e1c1314 Mon Sep 17 00:00:00 2001
From: Kelven Yang <kelven@cloud.com>
Date: Tue, 22 Mar 2011 15:37:00 -0700
Subject: [PATCH] validate CPU speed in system service offering to auto-correct
 settings from old data set

---
 .../src/com/cloud/service/dao/ServiceOfferingDaoImpl.java   | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/server/src/com/cloud/service/dao/ServiceOfferingDaoImpl.java b/server/src/com/cloud/service/dao/ServiceOfferingDaoImpl.java
index 710c2c18520..6ae9650adbe 100644
--- a/server/src/com/cloud/service/dao/ServiceOfferingDaoImpl.java
+++ b/server/src/com/cloud/service/dao/ServiceOfferingDaoImpl.java
@@ -25,7 +25,6 @@ import javax.persistence.EntityExistsException;
 
 import org.apache.log4j.Logger;
 
-import com.cloud.dc.DataCenterVO;
 import com.cloud.service.ServiceOfferingVO;
 import com.cloud.utils.db.DB;
 import com.cloud.utils.db.GenericDaoBase;
@@ -82,6 +81,11 @@ public class ServiceOfferingDaoImpl extends GenericDaoBase<ServiceOfferingVO, Lo
         assert offering.getUniqueName() != null : "how are you going to find this later if you don't set it?";
         ServiceOfferingVO vo = findByName(offering.getUniqueName());
         if (vo != null) {
+        	// check invalid CPU speed in system service offering, set it to default value of 500 Mhz if 0 CPU speed is found
+        	if(vo.getSpeed() <= 0) {
+        		vo.setSpeed(500);
+        		update(vo.getId(), vo);
+        	}
             return vo;
         }
         try {