找回密码
 立即注册
查看: 2716|回复: 0
打印 上一主题 下一主题

PHPCMS V9 member表内容随意修改漏洞及修复

[复制链接]
跳转到指定楼层
楼主
发表于 2012-12-4 11:27:48 | 只看该作者 回帖奖励 |倒序浏览 |阅读模式
某变量未初始化导致服务器设置register_global=On时会员可以随意修改自己的member信息。; Y, W4 ]) k' S2 s8 S

1 H# c7 o& Y/ Q/ O* X由于需要register_global=On支持,影响有限。。漏洞涉及到会员可以修改自己的余额导致网站可能产生严重后果因此漏洞等级设为高。3 m- N3 l( {0 r# Q% \9 B
详细说明:修改用户密码的account_manage_password函数未对$updateinfo初始化,导致可以提交9 h* A8 A* x" B
updateinfo[amount]这样的数组随意修改自己的余额、点数、会员组、VIP等一切存放在member表的信息( g) S% X! `" t" {
文件: \phpcms\modules\member\index.php, C; `* D; S& [8 ?- p

$ ?& d5 f$ {% Y约435行:
: R8 d# a0 X" l: f: {9 P1 x. ^$newpassword = password($_POST['info']['newpassword'], $this->memberinfo['encrypt']);
8 k$ {, o* G, `: y! J$updateinfo['password'] = $newpassword;3 H0 V) @: n  q3 K' X
$this->db->update($updateinfo, array('userid'=>$this->memberinfo['userid']));. g4 w5 f: |, c* _$ V, n$ O6 e3 s
漏洞证明:文件: \phpcms\modules\member\index.php* V5 V8 N0 q, h0 r

5 n5 M! g/ D8 d5 ]% P) y约435行:
# @8 B  S) T# N7 t; J0 }$newpassword = password($_POST['info']['newpassword'], $this->memberinfo['encrypt']);
5 ^" Z( V: Q; A$ m" G$updateinfo['password'] = $newpassword;
8 E$ C+ S5 L1 p9 V. _3 V$this->db->update($updateinfo, array('userid'=>$this->memberinfo['userid']));/ j5 b" l; n  @, s( Q

) g3 R' E; W, A修复方案:3 `) [: O+ n( S3 y' H

! v0 T3 ~* K+ F" c, L3 O$updateinfo = array();8 g2 j. Z3 T: c% E: j
" t' n! C- D! ~, y$ B. w
回复

使用道具 举报

您需要登录后才可以回帖 登录 | 立即注册

本版积分规则

快速回复 返回顶部 返回列表