ThinkSNS又一个任意上传文件漏洞

admin

  某模块未对上传文件类型进行验证。可上传任意文件
; f- C+ U9 b8 o4 B- B/ s* @
: N- _% {# _4 z- I. m9 u# ^+ o​

代码产生位置
apps\wap\Lib\Action\IndexAction.class.php
263行
if(!empty($_FILES['pic']['name'])) { // 自动发一条图片微博
4 S5 w' a% h- e+ E9 b: `$data['pic']      = $_FILES['pic'];
3 @9 L$ l0 y3 `. C$data['content']  = '图片分享';
$data['from']     = $this->_type_wap;
2 r8 h- f: R; G  Y  e$ n& b$res = api('Statuses')->data($data)->upload();
}

未对文件类型过滤
6 I/ N& V- {" O% ~, ?9 u( y
, n1 B8 n! q, o) Y访问wap 模块


2 @, v  B/ A( c. ]发一条微博并传图
9 u. P  V% T( J1 C/ [$ x  I
5 \, T- u4 z" xfirebug 地址
$ P6 ^3 Y8 x; ?" N  l- p
​
  h4 G; J5 s+ l3 e% s" c
) M5 b: I' `" j8 d+ h( F$ E
去掉small_然后访问

6 J4 Y6 Z) ?0 ]4 fhttp://www.myhack58.com/data/upl ... 7/50865d481c217.php
  A2 s9 |' f" B, ?
2 v6 L% t& `5 `& o! Q/ X

修复方案：

对上传类型要进行检查

/ N7 M, B# Q2 ?/ \# J+ T