0×01 前沿: u9 Y+ k) `4 q5 G2 s
; Z1 w, a1 n4 V+ X
Phpcms2008 是一款基于 PHP+Mysql 架构的网站内容管理系统,也是一个开源的 PHP 开发平台。Phpcms 采用模块化方式开发,功能易用便于扩展,可面向大中型站点提供重量级网站建设解决方案。3年来,凭借 Phpcms 团队长期积累的丰富的Web开发及数据库经验和勇于创新追求完美的设计理念,使得 Phpcms 得到了近10万网站的认可,并且越来越多地被应用到大中型商业网站。2 c, ?, f9 m: P! u! B8 n
9 n) n/ }1 Y9 s3 B0×02 写在前面的话
; ^2 q& @: v8 T' `% S* ]4 n6 o* Q5 p
" d h, a+ v" K6 W! H$ o/ u; [ phpcms 2008 这是我看第二次代码了,之前已经发现了一些问题,只是没放出来,这次稍微仔细看了看,又发现了一些问题4 [# @" J9 Q F. Y" Z) U) h, z4 `
! F, n: Q, W: D: b4 j
这次就放2个吧,其中啥啥的getshell暂时就不会放了,比起v9来说,2008的安全性能确实差很多,模块化以及代码严谨程度也没有v9强( Q9 k) u6 I+ b! ~* o- s/ k# K2 Y9 ~
2 O/ T( J$ E7 ]' b0 [这次还没把代码看完,只看完几个页面,就先放2个有问题的地方,如果有更好的方式,到时候一起讨论1 Q3 |- R2 d5 B) [
2 m* T, d5 C- S5 Z( n0×03 路径? ? ?* A: d2 W. I8 y# [
& P$ G# A! {5 p t5 e 在include/common.inc.php中 ,这是phpcms的全局要加载的配置文件 b) ^0 A* v- B h0 v/ {/ @/ S
1 O+ Y$ J# l1 O* J k
$dbclass = 'db_'.DB_DATABASE; require $dbclass.'.class.php'; $db = new $dbclass; $db->connect(DB_HOST, DB_USER, DB_PW, DB_NAME, DB_PCONNECT, DB_CHARSET); require 'session_'.SESSION_STORAGE.'.class.php'; $session = new session(); session_set_cookie_params(0, COOKIE_PATH, COOKIE_DOMAIN); if($_REQUEST) { if(MAGIC_QUOTES_GPC) { $_REQUEST = new_stripslashes($_REQUEST); if($_COOKIE) $_COOKIE = new_stripslashes($_COOKIE); extract($db->escape($_REQUEST), EXTR_SKIP); } else { $_POST = $db->escape($_POST); $_GET = $db->escape($_GET); $_COOKIE = $db->escape($_COOKIE); @extract($_POST,EXTR_SKIP); @extract($_GET,EXTR_SKIP); @extract($_COOKIE,EXTR_SKIP); } if(!defined('IN_ADMIN')) $_REQUEST = filter_xss($_REQUEST, ALLOWED_HTMLTAGS); if($_COOKIE) $db->escape($_COOKIE); } if(QUERY_STRING && strpos(QUERY_STRING, '=') === false && preg_match("/^(.*)\.(htm|html|shtm|shtml)$/", QUERY_STRING, $urlvar)) { parse_str(str_replace(array('/', '-', ' '), array('&', '=', ''), $urlvar[1])); } % h2 P1 c. \& C% P
- N1 N* P* b& d" U- b这里的话首先实例化了这个数据库,产生了一个$db资源句柄,他是用来操作数据库的. ]) f+ a2 f6 n1 J* D8 M
, Z3 K3 U+ v) N3 B然后就是将我们传进来的参数进行变量化0 f+ r0 r9 u1 t( Q- Q
! A U! {) T5 e4 E' n这里有一些小过滤,自己可以看,所以这里传进来的参数就作为了变量: `2 Y% x: c2 w1 n. S+ o) ~
- [5 r+ W# }/ a) {# z但是接下来这行呢?3 B: m' V- ^6 f% o6 S
L# ]7 G9 B' T/ _+ Z: ]
& o+ z1 j. S. W! x6 n! z
+ Q: i) w# O/ c) C& jif(QUERY_STRING && strpos(QUERY_STRING, '=') === false && preg_match("/^(.*)\.(htm|html|shtm|shtml)$/", QUERY_STRING, $urlvar)) { parse_str(str_replace(array('/', '-', ' '), array('&', '=', ''), $urlvar[1])); } 9 `3 \! U+ O. A v
9 K& x7 l6 K9 p K' X9 H7 I9 O+ x A
( H: ]- {% S7 `8 L* B7 Q; ]
2 Q1 M* V0 H6 A @$ B% |8 |- f
看看这里?
2 W8 _8 a$ p5 `8 E# C6 | J! O4 [# s% ~
这里的QUERY_STRING来自前面; _. K: [% O* D. {; w
* }3 C1 e& C2 }' Q( V3 ]
& C7 ~. B. @% w3 B2 A+ [6 L+ P& P3 z9 G/ L# q
define('IP', ip()); define('HTTP_REFERER', isset($_SERVER['HTTP_REFERER']) ? $_SERVER['HTTP_REFERER'] : ''); define('SCRIPT_NAME', isset($_SERVER['SCRIPT_NAME']) ? $_SERVER['SCRIPT_NAME'] : preg_replace("/(.*)\.php(.*)/i", "\\1.php", $_SERVER['PHP_SELF'])); define('QUERY_STRING', safe_replace($_SERVER['QUERY_STRING']));7 g) x9 r! F& Y8 u+ u4 U
这里有个过滤,但是不影响% K1 f6 s# E# v" q/ V( |. v
( w, u, c% N4 Q, d7 ^0 b1 D如果我们在这里进行覆盖这个db变量呢
1 u4 L& L; s7 A5 d/ u; Y
% d3 h* k0 V- F4 h, S$ \( z因为这里 parse_str(str_replace(array(‘/’, ’-', ’ ’), array(‘&’, ’=', ”), $urlvar[1]));5 k+ C7 d8 V( r5 t4 n
1 O: [ ^# F' A* e( s! `4 W6 x
可以将我们传进去的/ - 进行替换
& s' B3 \% r( c# v0 ?& y# }. f0 p* g* G
" ^& R6 z! F) _$ f( f- q, P: C所以我们如果提交如下字符
& f; }7 W- v& o, s6 l, ]8 b8 I3 r$ c+ G9 D
http://localhost/phpcms/index.php?db-5/gid-xd.html
- n" _) `, [+ D
R8 \2 t$ d: t# z1 a: h# ~他由于这个db被覆盖就会出错,所以物理路径就爆出来了 Y$ K# X; S: a+ F
* M0 E% U$ T) H# Z
0×04 SQL注入!!!0 }+ _7 J# h. n' r( U8 R" ]$ o8 R$ a
+ g4 ~& L- g5 I5 k& F4 s 在c.php中
7 J; l0 ?3 ], w
. ~6 h8 e! j. F' F. a
. _% Q% H0 x; X, p' n. k: d& x9 D9 _7 e: w# D
<?php require './ads/include/common.inc.php'; $id = intval($id); $ads = $c_ads->get_info($id); if($ads) { $db->query("UPDATE ".DB_PRE."ads SET `clicks`=clicks+1 WHERE adsid=".$ads['adsid']); $info['username'] = $_username; $info['clicktime'] = time(); $info['ip'] = IP; $info['adsid'] = $id; $info['referer'] = HTTP_REFERER; $year = date('ym',TIME); $table = DB_PRE.'ads_'.$year; $table_status = $db->table_status($table); if(!$table_status) { include MOD_ROOT.'include/create.table.php'; } $db->insert($table, $info); $url = strpos($ads['linkurl'], 'http://')===FALSE ? 'http://'.$ads['linkurl'] : $ads['linkurl']; } 5 o- O7 m- }: E
$ b, Z& M- O9 ]) G% i1 l) r / b: C2 f) N0 Q
" ?: A1 P- i$ q# a j% L+ b7 v注意这里的HTTP_REFERER这个常量$ y# E2 V1 u/ C$ b' {7 i3 W6 }) o+ c
/ ^" G# t4 Y& u) m
这里的常量是通过前面的common.inc.php定义好的) V3 b' M- Z# H3 h
, l- K8 \* u% v" S( L
define(‘HTTP_REFERER’, isset($_SERVER['HTTP_REFERER']) ? $_SERVER['HTTP_REFERER'] : ”);, ?0 I3 N1 ~% ^
& {8 p+ k1 Q6 X
没有经过任何过滤操作,所以你懂的,我估计很多同学已经发现了,只是没去公布了,所以俺就替你们xxoo了,哈哈…别骂我
4 b# C* d4 |; u9 k
0 d: L3 O1 Q7 g+ }, T然后
# j% t: @$ ~% H" C
3 F0 Z. c) B& u r% C8 t# M! m3 ?$db->insert($table, $info);0 X" U4 h2 W g, [) r4 G; |
我们来看一下它这里的操作% D- {& O; ?6 G
) {/ g0 |: m" O g$ T2 Cfunction insert($tablename, $array) { $this->check_fields($tablename, $array); return $this->query("INSERT INTO `$tablename`(`".implode('`,`', array_keys($array))."`) VALUES('".implode("','", $array)."')"); }
; M0 i( I9 |$ V5 _0 W. R4 u+ t/ e$ n6 K2 x i) ?9 X* r
所以你懂的
/ V `" x) x+ L+ @5 W. ~7 }6 K9 p* \
# Y! \/ s9 B: V) o4 b8 [
. J( ?1 k! O; N# i. v
附EXP:http://pan.baidu.com/share/link?shareid=468231&uk=40456377372 Z9 n! p3 z- {
0 u" {7 j* N& t8 c3 C4 U1 s* c. G2 M
' f7 \; H) p+ |$ ]
* m# ^# T. {' l& l _) s" x: M; ~ |