用户
 找回密码
 入住 CI 中国社区
搜索
查看: 5150|回复: 10
收起左侧

[已解决] CI写的代码,检测出漏洞

[复制链接]
发表于 2011-11-15 15:00:31 | 显示全部楼层 |阅读模式
QQ截图20111115145904.png
这是一个增加评论的功能代码如下:

PHP复制代码
 
//提交评论
    function add_comments() {
        $submit = $this->input->post("submit");
        $r_id = $this->input->post("resource_id",TRUE);
        $login_name = $this->input->post("login_name",TRUE);
        if (empty($login_name)) {
            $login_name = "无名氏";
        }
        $content = $this->input->post("content",TRUE);
        if ($submit) {
            $datestring = "%Y-%m-%d %H:%i:%s";
            $time = time();
            $create_date = mdate($datestring, $time);
            if (empty($content)) {
                redirect("single?id=" . $r_id);
            } else {
                $query = $this->m_single->do_add_comments($r_id, $login_name, $content, $create_date);
                if ($query) {
                    redirect("single?id=" . $r_id);
                } else {
                    redirect("single?id=" . $r_id);
                }
            }
        }
    }
 
 
复制代码




发表于 2011-11-16 02:45:37 | 显示全部楼层
CI一上来就直接吧$_GET销毁了,如果要CRSF只能通过uri。在配置文件里面把CSRF过滤打开应该就没问题了吧
 楼主| 发表于 2011-11-16 11:43:53 | 显示全部楼层
打开以后会报错
发表于 2011-11-18 09:11:50 | 显示全部楼层
顶楼主一个!好东西,又学习了.
发表于 2011-11-18 10:46:05 | 显示全部楼层
看不出来哪有问题,呵呵,刚刚接触
发表于 2011-11-22 23:26:28 | 显示全部楼层
其实xss过滤做好了的话csrf基本不需要担心吧
发表于 2011-11-23 15:00:21 | 显示全部楼层
学习了过会试一下
发表于 2011-11-26 00:44:56 | 显示全部楼层
CI新手,学习了!!!!!!{:soso_e185:}顶起来!!!
发表于 2012-1-17 15:11:53 | 显示全部楼层
没看懂,求解释。
发表于 2012-1-18 21:39:21 | 显示全部楼层
CSRF攻击需要你在表单里增加一个token,这个 CI 2.0 里提供了这方面的验证,仔细看一下手册,呵呵。

本版积分规则