fixed bug 189 206

data/model/user_bonus.model.php

@@ -141,6 +141,7 @@ class user_bonusModel extends Model
                     }
 
                     $ret = $this->where(array('type_id' => $type_id,'bonus_id' => $bonus['bonus_id']))->update($datas);
+                    $affect_rows = $this->affected_rows();
                     if($ret) {
                         $bonus = array_merge($bonus,$datas);
                     }
@@ -178,6 +179,7 @@ class user_bonusModel extends Model
             }
 
             $ret = $this->where($condition)->update($datas);
+            $affect_rows = $this->affected_rows();
             $this->commit();
 
             return $ret;

helper/bonus/generator.php

@@ -113,7 +113,7 @@ class GeneralGenerator extends IGenerator
         $base_amount = $min_amount;
 
         $num = intval($paramer['total_num']);
-        $total_amount = floatval($paramer['total_amount']) * 100 - $num * $base_amount;
+        $total_amount = intval(floatval($paramer['total_amount']) * 100) - $num * $base_amount;
 
         $ret = array();
         for ($i = 0; $i < $num; $i++) {
@@ -124,7 +124,7 @@ class GeneralGenerator extends IGenerator
         $max_val = 0;
         for ($i = 0; $i < $num && $total_amount > 0;)
         {
-            $amount = mt_rand(0, $max_amount);
+            $amount = intval(mt_rand(0, $max_amount));
 
             $ret[$i] += $amount;
             $total_amount -= $amount;

helper/sms_helper.php

@@ -49,6 +49,7 @@ class sms_helper
             $_SESSION['sms_code'][$itype]['count'] = $count + 1;
             $_SESSION['sms_code'][$itype]['code'] = $code;
             $_SESSION['sms_code'][$itype]['last_time'] = time();
+            $_SESSION['sms_code'][$itype]['mobile'] = $mobile;
 
             return array('code' => 200);
         } else {
@@ -57,10 +58,14 @@ class sms_helper
         }
     }
 
-    static public function check_code($itype,$code)
+    static public function check_code($itype,$code,$mobile)
     {
         if(isset($_SESSION['sms_code'][$itype]))
         {
+            if($_SESSION['sms_code'][$itype]['mobile'] != $mobile) {
+                return array('code' => errcode::ErrSmscode, 'msg' => '验证手机号和发送验证码手机号不一致.');
+            }
+
             $tmout = time() - $_SESSION['sms_code'][$itype]['last_time'];
             if($tmout <= self::code_expire)
             {

mobile/control/login.php

@@ -186,7 +186,7 @@ class loginControl extends mobileHomeControl
         if ($err != '') {
             return self::outerr(errcode::ErrParamter, $err);
         }
-        $ret = sms_helper::check_code(Sms::register_code,$code);
+        $ret = sms_helper::check_code(Sms::register_code,$code,$mobile);
         if(is_array($ret)) {
             return self::outerr($ret['code'], $ret['msg']);
         }
@@ -229,7 +229,7 @@ class loginControl extends mobileHomeControl
             return self::outerr(errcode::ErrParamter, $err);
         }
 
-        $ret = sms_helper::check_code(Sms::resetpass_code,$code);
+        $ret = sms_helper::check_code(Sms::resetpass_code,$code,$mobile);
         if(is_array($ret)) {
             return self::outerr($ret['code'], $ret['msg']);
         }
@@ -270,7 +270,7 @@ class loginControl extends mobileHomeControl
         }
 
         // 校验验证码
-        $ret = sms_helper::check_code(Sms::register_code,$code);
+        $ret = sms_helper::check_code(Sms::register_code,$code,$mobile);
         if(is_array($ret)) {
             return self::outerr($ret['code'], $ret['msg']);
         }