代码审计之Finecms任意文件下载漏洞

Posted 2020-09-14 可我浪费着我寒冷的年华

PS：该漏洞已被公布，只是学习。故自己跟着大佬的步伐审计。

漏洞所在文件地址：\controllers\ApiController.php Line 57

    public function downAction() {
        $data = fn_authcode(base64_decode($this->get(‘file‘)), ‘DECODE‘);
        $file = isset($data[‘finecms‘]) && $data[‘finecms‘] ? $data[‘finecms‘] : ‘‘;
        if (empty($file)) {
            $this->msg(lang(‘a-mod-213‘));//该方法可以不管。
        }
        if (strpos($file, ‘:/‘)) {//查找:/第一次出现的位置，如果有就执行header否则...
            //远程
            header("Location: $file");
        } else {
            //本地
            $file = str_replace(‘..‘, ‘‘, $file);//将变量file里的..替换为空
            $file = strpos($file, ‘/‘) === 0 ? APP_ROOT.$file : $file;//找$file出现在第一位，则返回根路径+$file
            if (!is_file($file)) {
                $this->msg(lang(‘a-mod-214‘) . ‘(#‘ . $file . ‘)‘);
            };
            header(‘Pragma: public‘);
            header(‘Last-Modified: ‘ . gmdate(‘D, d M Y H:i:s‘) . ‘ GMT‘);
            header(‘Cache-Control: no-store, no-cache, must-revalidate‘);
            header(‘Cache-Control: pre-check=0, post-check=0, max-age=0‘);
            header(‘Content-Transfer-Encoding: binary‘);
            header(‘Content-Encoding: none‘);
            header(‘Content-type: ‘ . strtolower(trim(substr(strrchr($file, ‘.‘), 1, 10))));//strchr是从首个出现.的地方开始截断。strolower转换为小写。
            header(‘Content-Disposition: attachment; filename="‘ . basename($file) . ‘"‘);
            header(‘Content-length: ‘ . sprintf("%u", filesize($file)));//springtf：把%号替换成一个作为参数，进行传递的变量。
            readfile($file);
            exit;
        }
    }

从这个函数当中可以看出$file是可控的一个变量。