diff --git a/webapp/controller/HeroController.class.php b/webapp/controller/HeroController.class.php
index 604380b3..3d27d598 100644
--- a/webapp/controller/HeroController.class.php
+++ b/webapp/controller/HeroController.class.php
@@ -140,7 +140,7 @@ class HeroController extends BaseAuthedController {
             $this->_rspErr(1, "Not enough hero skill points");
             return;
         }
-        Hero::upgradeSkill($heroUniId,$skillIdx,$skill['skill_point']);
+        Hero::upgradeSkill($heroUniId,$skillIdx,$next_skill['skill_point']);
         $propertyChgService = new services\PropertyChgService();
         $propertyChgService->addHeroChg();
         $this->_rspData(array(
diff --git a/webapp/models/Hero.php b/webapp/models/Hero.php
index e33b7e05..80f901e2 100644
--- a/webapp/models/Hero.php
+++ b/webapp/models/Hero.php
@@ -407,7 +407,7 @@ class Hero extends BaseModel {
                     return "${fieldName} + 1";
                 },
                  'skill_points' => function() use ($skill_points){
-                     return "GREATEST(0, skill_points - ${$skill_points})";
+                     return "GREATEST(0, skill_points - ${skill_points})";
                  },
                 'modifytime' => myself()->_getNowTime()
         ));