验证码的一般思路,就是每次登陆的地方访问一个脚本文件,该文件生成含验证码的图片并将值写入到Session里,提交的时候验证登陆的脚本就会判断提交的验证码是否与Session里的一致。
问题出现了,在登陆密码错误之后,我们不去访问生成验证图片的文件,那么如果Session中的验证码没有被清空,此时验证码就是跟上次的一样,辛辛苦苦构建的验证码机制就形同虚设了。
下面我们先来看一段有问题的代码:
登陆部分:
| 以下是代码片段: <tr> <td>管理员姓名:</td> <td><input type="text" name="username" /></td> </tr> <tr> <td>管理员密码:</td> <td><input type="password" name="password" /></td> </tr> <tr> <td>验证码:</td> <td><input type="text" name="captcha" onkeyup="pressCaptcha(this)" /></td> </tr> <tr> <td colspan="2" align="right"> <img src="index.php?act=captcha&1628020115" width="145" height="20" alt="CAPTCHA" border="1" onclick= this.src="index.php?act=captcha&"+Math.random() style="cursor: pointer;" title="看不清?点击更换另一个验证码。" /> </td> </tr> 这里没什么问题,来看登陆验证的代码(我想这样的验证思路,也是大多数人都在用的吧): <?php /*------------------------------------------------------ */ //-- 验证登陆信息 /*------------------------------------------------------ */ if ($_REQUEST == 'signin') { include('../includes/cls_captcha.php'); /* 检查验证码是否正确 */ $validator = new captcha(); if (!$validator->check_word($_POST)) { sys_msg($_LANG, 1); } /* 检查密码是否正确 */ $sql = "SELECT user_id, user_name, password, action_list FROM " .$ecs->table('admin_user'). " WHERE user_name='$_POST' AND password='" .md5($_POST). "'"; $row = $db->GetRow($sql); if ($row) { // 登录成功 set_admin_session($row, $row, $row); // 更新最后登录时间和IP $db->Execute("UPDATE " .$ecs->table('admin_user'). " SET last_time='" .date('Y-m-d H:i:s', time()). "', last_ip='" .real_ip(). "'". " WHERE user_id=$_SESSION") OR die($db->ErrorMsg()); if (isset($_POST)) { setcookie('ECSCP', $row, time() + 3600 * 24 * 360); setcookie('ECSCP', md5($row . $_CFG), time() + 3600 * 24 * 360); } header('location:./'); } else { sys_msg($_LANG, 1); } } ?> |
大家可以看下面的图片,增强点直观的认识。

解决方法:我们需要在检查密码错误后更新验证码,对于留言等类型的,还要在提交成功后更新验证码。
安全就是这样,我们总是想让自己的程序更安全,但是一般情况下,我们又总是走在常规思维里跳不出来,于是导致我们的程序出现了很多"非常规漏洞",或者叫做"缺陷",总之就是不完美。我写这篇文章除了指出上面这个问题之外,还希望大家都能行动起来,用"非常规"眼光,重新检查下自己的程序,把更多以前自己没有发现的小问题写出来,让大家共同提高!
