日期:2014-05-17  浏览次数:20476 次

分享DEDECMS 验证码的一个细节问题。附提问一个疑惑,有关SESSION的
本帖最后由 default7 于 2012-12-01 15:54:51 编辑

问题:重复提交问题。表单提交成功了,发现页面单击右键点击返回上一页,浏览器提示“此页面不可用”,按下F5重新刷新,结果又提交成功了。导致“如果不停地刷新”就可以不停地提交成功!验证码的问题。
代码:

if($dopost == 'add'){
  $svali = GetCkVdValue();
  if(strtolower($vdcode)!=$svali || $svali=='')
  {
      ResetVdValue();
      ShowMsg('验证码错误!', '-1');
      exit();
  }
}

解决: 一开始以为是ResetVdValue();这个函数的问题,最后才发现是执行顺序的问题,把   ResetVdValue(); 放到判断“提交的验证码是否正确”的代码之前就可以了。

if($dopost == 'add'){
  $svali = GetCkVdValue();
  ResetVdValue(); //位置移前
  if(strtolower($vdcode)!=$svali || $svali=='')
  {
      ShowMsg('验证码错误!', '-1');
      exit();
  }
}




后来看了验证码方面的获取和销毁的函数,有一个疑问的地方:
1.未改动前的文件:/include/common.func.php,函数GetCkVdValue()和ResetVdValue()

/**
 *  获取验证码的session值
 *
 * @return    string
 */
function GetCkVdValue()
{
    @session_id($_COOKIE['PHPSESSID']);
    @session_start();
    return isset($_SESSION['securimage_code_value']) ? $_SESSION['securimage_code_value'] : '';
}

/**
 *  PHP某些版本有Bug,不能在同一作用域中同时读session并改注销它,因此调用后需执行本函数
 *
 * @return    void
 */
function ResetVdValue()
{
    @session_start();
    $_SESSION['securimage_code_value'] = '';
}


2.未改动前的验证码生成程序:/include/vdimgck.php 开头的地方有写

$sessSavePath = DEDEDATA."/sessions/";

// Session保存路径
if(is_writeable($sessSavePath) && is_readable($sessSavePath)){ session_save_path($sessSavePath); }
if(!empty($cfg_domain_cookie)) session_set_cookie_params(0,'/',$cfg_domain_cookie);

session_start();



个人将代码修改成如下,文件/include/common.func.php


/**
 *  获取验证码的session值
 *
 * @return    string
 */
function GetCkVdValue()
{
    $sessSavePath = DEDEDATA."/sessions/";
    if(is_writeable($sessSavePath) && is_readable($sessSavePath)){ session_save_path($sessSavePath); }
    if(!empty($GLOBALS['cfg_domain_cookie'])) session_set_cookie_params(0,'/',$GLOBALS['cfg_domain_cookie']);

    @session_id($_COOKIE['PHPSESSID']);
    @session_start();
    return isset($_SESSION['securimage_code_value']) ? $_SESSION['securimage_code_value'] : '';
}

/**
 *  PHP某些版本有Bug,不能在同一作用域中同时读session并改注销它,因此调用后需执行本函数
 *
 * @return    void
 */
function ResetVdValue()
{
    $sessSavePath = DEDEDATA."/sessions/";
    if(is_writeable($sessSavePa