ThinkCMF是一款基于ThinkPHP+MySQL开发的中文内容管理框架。 cmfx, 在 ThinkPHP 3.2.3上,它与 ThinkCMF ThinkCMFX based相同,并且从thinkphp3抽象出了四个base Controller ,HomebaseController、AdminbaseController、AppframebaseController、MemberbaseController。
官方给出的修复结果如下
Thinkphp3中模板渲染经常会使用到View层中的fetch、display、assign方法,之前thinkphp3曾出现过的安全问题正是发生在这模板引擎渲染过程中所导致,如tp3.x任意文件包含正是由于在模版渲染的情况下存在任意变量覆盖所导致。
了解了这个,看thinkcmfx从tp3.2.3抽象出来的控制类,通过调试发现传入的content进入到了HomebaseController.php的fetch中
F7跟进调用了thinkphp的controller类,然后最终调用think核心的view,
在View
类的fetch
方法中将可控的$content直接拼接到了eval里
从Hook::listen
一路跟下去,进入到listen方法中
在listen中可以看到将$centon传给了view_parse传入的参数$params然后进入到了exec
有插件时执行进入第二个if,直到跟踪到run
在run方法中调用think模板引擎if判断是否存在生成了模板缓存文件否则进入think->Template进行编译并加载模板文件,继续跟进fetch可控的参数名变成了$templateFile
:
将$templateFile
传入了loadTemplate方法,进入loadTemplate
可以追踪到该可完全控制的变量又进入到了编译模板内容方法中,在compiler中可以发现
可控的$tmplContent直接拼接到了php代码中。
调试后如下图
编译完成后返回编译后的文件。
我们看一下`Storage::load方法干了什么: 直接进行了文件包含,就这样我们的代码就被成功执行了。
漏洞发生在前台\application\Asset\Controller\UeditorController.class.php中在上传图片时会进入upload->uploadimage->调用UE上传方法_ueditor_upload如下图所示
跟进_ueditor_upload该方法先后进行了设置上传信息、获取上传后缀、文件大小定义允许的后缀名和config信息再交由think的upload.这里主要看think->upload方法
而在传入upload的$config仔细查看可以发现问题若上传的后缀不在定义规定的允许的上传后缀中时返回值为null如下图所示
think的upload.class.php中upload方法如下:
public function upload($files='') { if('' === $files){ $files = $_FILES; } if(empty($files)){ $this->error = '没有上传的文件!'; return false; } /* 检测上传根目录 */ if(!$this->uploader->checkRootPath($this->rootPath)){ $this->error = $this->uploader->getError(); return false; } /* 检查上传目录 */ if(!$this->uploader->checkSavePath($this->savePath)){ $this->error = $this->uploader->getError(); return false; } /* 逐个检测并上传文件 */ $info = array(); if(function_exists('finfo_open')){ $finfo = finfo_open ( FILEINFO_MIME_TYPE ); } // 对上传文件数组信息处理 $files = $this->dealFiles($files); foreach ($files as $key => $file) { $file['name'] = strip_tags($file['name']); if(!isset($file['key'])) $file['key'] = $key; /* 通过扩展获取文件类型,可解决FLASH上传$FILES数组返回文件类型错误的问题 */ if(isset($finfo)){ $file['type'] = finfo_file ( $finfo , $file['tmp_name'] ); } /* 获取上传文件后缀,允许上传无后缀文件 */ $file['ext'] = pathinfo($file['name'], PATHINFO_EXTENSION); /* 文件上传检测 */ if (!$this->check($file)){ continue; } /* 获取文件hash */ if($this->hash){ $file['md5'] = md5_file($file['tmp_name']); $file['sha1'] = sha1_file($file['tmp_name']); } /* 调用回调函数检测文件是否存在 */ $data = call_user_func($this->callback, $file); if( $this->callback && $data ){ if ( file_exists('.'.$data['path']) ) { $info[$key] = $data; continue; }elseif($this->removeTrash){ call_user_func($this->removeTrash,$data);//删除垃圾据 } } /* 生成保存文件名 */ $savename = $this->getSaveName($file); if(false == $savename){ continue; } else { $file['savename'] = $savename; } /* 检测并创建子目录 */ $subpath = $this->getSubPath($file['name']); if(false === $subpath){ continue; } else { $file['savepath'] = $this->savePath . $subpath; } /* 对图像文件进行严格检测 */ $ext = strtolower($file['ext']); if(in_array($ext, array('gif','jpg','jpeg','bmp','png','swf'))) { $imginfo = getimagesize($file['tmp_name']); if(empty($imginfo) /* || ($ext == 'gif' && empty($imginfo['bits'])) */){//ThinkCMF NOTE 限制太严格,以防单页gif文件无法上传 $this->error = '非法图像文件!'; continue; } } /* 保存文件 并记录保存成功的文件 */ if ($this->uploader->save($file,$this->replace)) { unset($file['error'], $file['tmp_name']); $info[$key] = $file; } else { $this->error = $this->uploader->getError(); } } if(isset($finfo)){ finfo_close($finfo); } return empty($info) ? false : $info; }
这里对文件依次进行了检查,在文件处理处跟进通过dealFiles获取到原本的上传文件信息将文件赋给files,遍历files开始上传
调用check()对文件进行检查
可以发现对文件后缀的检查checkExt存在问题,直接返回的是文件后缀并未检查。如下图所示:
这时的后缀仍然为php,往下调用getSaveName生成保存的文件名filename并拼接后缀php后返回赋给$savename
往下继续看,虽然发现又对文件ext判断一次但是显然并无影响最终执行save()
回到UeditorController.class.php中,最后将上传成功后的文件路径信息返回。