diff --git a/webapp/controller/HeroSkinController.class.php b/webapp/controller/HeroSkinController.class.php
index dad92bb4..9e6f592c 100644
--- a/webapp/controller/HeroSkinController.class.php
+++ b/webapp/controller/HeroSkinController.class.php
@@ -115,14 +115,14 @@ class HeroSkinController extends BaseAuthedController {
     }
 
     public function synSkin(){
-        $skinId = trim(getReqVal('skin_id', 0));
         $itemId = getReqVal('item_id',0);
         $itemNum = getReqVal('item_num',0);
-        $itemMeta = \mt\Item::get($skinId);
-        if ($itemMeta['type'] != \mt\Item::HERO_SKIN_TYPE){
-            $this->_rspErr(1, 'skin_id error');
+        $itemMeta = \mt\Item::get($itemId);
+        if (!$itemMeta || $itemMeta['type'] != \mt\Item::FRAGMENT_TYPE){
+            $this->_rspErr(1, 'param error');
             return;
         }
+        $skinId = $itemMeta['relationship'];
         $dressMeta = mt\DressAttribute::get($skinId);
         if (!$dressMeta){
             $this->_rspErr(1, 'skin_id error');
@@ -148,8 +148,9 @@ class HeroSkinController extends BaseAuthedController {
         $this->_decItems($costItems);
         $propertyChgService = new services\PropertyChgService();
         $propertyChgService->addBagChg();
+        $skinMeta = \mt\Item::get($skinId);
         for ($i=0; $i<$number; $i++){
-            HeroSkin::addSkin($itemMeta);
+            HeroSkin::addSkin($skinMeta);
         }
         $propertyChgService->addHeroSkinChg();
         $event = array(
@@ -165,7 +166,7 @@ class HeroSkinController extends BaseAuthedController {
         );
         LogService::burialPointEvent($event);
         $this->_rspData(array(
-            'item_id' => $itemMeta['id'],
+            'item_id' => $skinId,
             'property_chg' => $propertyChgService->toDto(),
         ));
     }