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

admin

  某模块未对上传文件类型进行验证。可上传任意文件

! M) z. `, `$ B' m+ w$ f  P​
3 I8 P, H  ~/ \7 J7 @- z
  Z6 ~/ v2 K: L. F; \7 {+ Q5 S代码产生位置
apps\wap\Lib\Action\IndexAction.class.php
& U% K, p/ J$ u6 s7 ]263行
if(!empty($_FILES['pic']['name'])) { // 自动发一条图片微博
" }3 ~* J+ R! N* M8 u$data['pic']      = $_FILES['pic'];
$data['content']  = '图片分享';
$data['from']     = $this->_type_wap;
$res = api('Statuses')->data($data)->upload();
}
& R7 }! ^0 x' L! K7 y" s4 ?
" v: g8 _9 A, @2 @未对文件类型过滤
* n) N% Q7 j! b8 X  S9 b
/ |+ U) O3 _: ~1 {访问wap 模块
( N3 C7 Y& p- Y7 A7 s

发一条微博并传图

0 R4 {) ]- E+ B3 b0 x2 vfirebug 地址
1 A) q/ y0 V. o- G
3 y/ b( V) |6 R8 x8 i8 A​
+ f+ p6 e) ]) D9 L& b' `9 |

' g4 U8 _1 _( c: p' b3 f, {' U4 Z去掉small_然后访问

http://www.myhack58.com/data/upl ... 7/50865d481c217.php



修复方案：
' ^/ y$ S7 S- ~8 c, }
对上传类型要进行检查
  x+ t$ M) \2 X5 F5 B5 E( b