ThinkSNS 2.8任意文件上传漏洞及修复

admin

  微博上传图片时只在前端进行验证, 服务器端没有进行安全过滤。
  v* K& O3 u  L  V
' X3 s3 J% v, r8 E1 e
\api\StatusesApi.class.php

# {2 _; X. l% E* s6 `! ifunction uploadpic(){
      if( $_FILES['pic'] ){
     //执行上传操作
+ G5 Z# ~% y. k     $savePath =  $this->_getSaveTempPath();
     $filename = md5( time().'teste' ).'.'.substr($_FILES['pic']['name'],strpos($_FILES['pic']['name'],'.')+1);
    if(@copy($_FILES['pic']['tmp_name'], $savePath.'/'.$filename) || @move_uploaded_file($_FILES['pic']['tmp_name'], $savePath.'/'.$filename))
  \. }) [# J8 s( w3 q  \8 q       {
* ?- B  \6 {, K& w$ V        $result['boolen']    = 1;
9 s9 ~, W. n8 Q3 u        $result['type_data'] = 'temp/'.$filename;
% L. ]% g/ b4 i8 Q. ~; `6 m, O        $result['picurl']    = SITE_PATH.'/uploads/temp/'.$filename;
       } else {
8 q9 x. ?7 D! P3 g        $result['boolen']    = 0;
        $result['message']   = '上传失败';
       }
     }else{
' w/ P) A1 z; M$ G         $result['boolen']    = 0;
         $result['message']   = '上传失败';
7 [; U3 S3 P; i* j. Y     }
return $result;
    }
( T7 `; ^4 W& n, C, U4 _) J2 wunloadpic()方法没有对文件类型进行验证

可以构建表单, 选择任意文件, 提交到
$ F% L/ G7 q# i5 D. H/index.php?app=w3g&mod=Index&act=doPost
  e9 k% u( d1 X
! {- H6 n+ t& k1 [8 {1 a在新提交的微博上可以找到上传的文件地址(去掉small_、middle_ 前缀)


在登录thinksns官方微博后,
构建以下表单:

<form action="http://t.thinksns.com/index.php?app=w3g&mod=Index&act=doPost" method="post" enctype="multipart/form-data" />
$ q3 s7 b( ^* ]: N<textarea name="content">test</textarea>
file: <input id="file" type="file" name="pic" />
2 k* }7 d. Z* @8 R3 X, M+ ^! K<input type="submit" value="Post" />
</form>
去掉缩略图的前缀(small_ )
$ E# }; Q) ]; J9 Y# m​修复方案：


\api\StatusesApi.class.php
9 |  a/ u! q% M7 v! q4 i7 k6 j
( o3 p/ X! E" D; Q3 ~/ `7 l- T1 f" ~function uploadpic(){
     /**
% x4 f1 V4 p6 d6 Y2 |+ R$ z      * 20121018 @yelo
      * 增加上传类型验证
      */
& d5 X0 K+ L# _; c) C     $pathinfo = pathinfo($_FILES['pic']['name']);
     $ext = $pathinfo['extension'];
& b0 f" P# F; m; n. k: Z# K $allowExts = array('jpg', 'png', 'gif', 'jpeg');

& P: o; V7 u! D4 t     $uploadCondition = $_FILES['pic'] && in_array(strtolower($ext),$allowExts,true);
0 U; u* I& U# ^
     if( $uploadCondition ){
5 c) a; u; [# b  @0 N# l7 L9 U     //执行上传操作
     $savePath =  $this->_getSaveTempPath();
  V0 R9 V# `0 H& R     $filename = md5( time().'teste' ).'.'.substr($_FILES['pic']['name'],strpos($_FILES['pic']['name'],'.')+1);
    if(@copy($_FILES['pic']['tmp_name'], $savePath.'/'.$filename) || @move_uploaded_file($_FILES['pic']['tmp_name'], $savePath.'/'.$filename))
- v3 ?2 O6 Q. F( P0 ?3 }       {
# u, p1 G. i& A* B4 U        $result['boolen']    = 1;
- P* v! W  o8 h. n# c7 _        $result['type_data'] = 'temp/'.$filename;
5 a( q2 U1 }1 ^8 ]9 V        $result['picurl']    = SITE_PATH.'/uploads/temp/'.$filename;
       } else {
        $result['boolen']    = 0;
        $result['message']   = '上传失败';
9 e9 a5 D2 i6 \& H  z       }
     }else{
5 [/ c3 Y1 U! a. a         $result['boolen']    = 0;
         $result['message']   = '上传失败';
     }
return $result;
    }
8 {* R: |/ |' b5 Z" v​

1 U" ?# L, L  t$ E" V) F0 |9 v