修复整理ecshop漏洞

abcen 2022-08-22 发布于山东

展开全文

1、ECShop存在一个盲注漏洞，问题存在于/api/client/api.php文件中，提交特制的恶意POST请求可进行SQL注入攻击，可获得敏感信息或操作数据库。
路径：/api/client/includes/lib_api.php
参照以下修改：

推荐（免费）：eschop

function API_UserLogin($post)

{

/* SQL注入过滤 */

if (get_magic_quotes_gpc())

{

$post['UserId'] = $post['UserId'];

}

else

{

$post['UserId'] = addslashes($post['UserId']);

}

/* end */

$post['username'] = isset($post['UserId']) ? trim($post['UserId']) : '';

2、ecshop的后台编辑文件/admin/shopinfo.php中，对输入参数id未进行正确类型转义，导致整型注入的发生。
路径： /admin/shopinfo.php
参照以下修改（53-71-105-123行）：

源代码：

admin_priv('shopinfo_manage');

改为：

admin_priv('shopinfo_manage');

$_REQUEST['id'] = intval($_REQUEST['id']);

3、文件/admin/affiliate_ck.php中，对输入参数auid未进行正确类型转义，导致整型注入的发生。
参照以下修改（31行和51行）：

源代码：

$logdb = get_affiliate_ck();

改成：

$_GET['auid'] = intval($_GET['auid']);

$logdb = get_affiliate_ck();

注：好像按上面改了阿里云还是会提示这个漏洞，然后我在文件顶部处添加以下代码就可以了：

1	`$_GET['auid'] =` `intval($_GET['auid']);`

4、ecshop的/admin/comment_manage.php中，对输入参数sort_by、sort_order未进行严格过滤，导致SQL注入。
参照以下修改：

1 2	`$filter['sort_by'] =` `empty($_REQUEST['sort_by']) ?` `'add_time'` `: trim(htmlspecialchars($_REQUEST['sort_by']));` `$filter['sort_order'] =` `empty($_REQUEST['sort_order']) ?` `'DESC'` `: trim(htmlspecialchars($_REQUEST['sort_order']));`

5、ecshop没有对会员注册处的username过滤,保存重的用户信息时,可以直接写入shell。
路径：/admin/integrate.php
大概109行，参照以下修改：

1	`$code` `=` `empty($_GET['code']) ?` `''` `: trim(addslashes($_GET['code']));`

大概601行，参照以下修改：

源代码：

@file_put_contents(ROOT_PATH . 'data/repeat_user.php', $json->encode($repeat_user));

修改成：

@file_put_contents(ROOT_PATH.'data/repeat_user.php','<?php die();?>'.$json->encode($repeat_user));

注：好像按上面改了阿里云还是会提示这个漏洞，然后我在文件顶部处添加以下代码就可以了：

1	`$_GET['code'] =` `empty($_GET['code']) ?` `''` `: trim(addslashes($_GET['code']));`

6、ecshop后台模版编译导致黑客可插入任意恶意代码。
路径：/admin/edit_languages.php
大概在第120行

1	`$dst_items[$i] =` `$_POST['item_id'][$i] .' = '.` `'"'` `.$_POST['item_content'][$i].` `'";';`

修改为

1	`$dst_items[$i] =` `$_POST['item_id'][$i] .' = '.` `'\''` `.$_POST['item_content'][$i].` `'\';';`

7、ecshop过滤不严导致SQL注入漏洞。
路径：/category.php 、 /ecsapi/category.php
修改方法：https://www.cnblogs.com/LittleHann/p/4524161.html

8、ecshop的/includes/lib_insert.php文件中，对输入参数未进行正确类型转义，导致整型注入的发生。
有$arr['num'] 、$arr['id']、$arr['type']这些参数的，在函数开头加上：

大概289行加上：

$arr['num'] = intval($arr['num']);

$arr['id'] = intval($arr['id']);

大概454行加上：

$arr['id'] = intval($arr['id']);

$arr['type'] = addslashes($arr['type']);

大概495行加上：

$arr['id'] = intval($arr['id']);

9、ECSHOP支付插件存在SQL注入漏洞，此漏洞存在于/includes/modules/payment/alipay.php文件中，该文件是ECshop的支付宝插件。由于ECShop使用了str_replace函数做字符串替换，黑客可绕过单引号限制构造SQL注入语句。只要开启支付宝支付插件就能利用该漏洞获取网站数据，且不需要注册登入。

搜索代码：

$order_sn = str_replace($_GET['subject'], '', $_GET['out_trade_no']);

将下面一句改为：

$order_sn = trim(addslashes($order_sn));

Ecshop后台模版编辑漏洞
详情描述参考：https://www.cnblogs.com/LittleHann/p/4272255.html

漏洞文件：/admin/mail_template.php 和 /includes/cls_template.php
/admin/mail_template.php文件添加以下代码（过滤了部分php关键词）

/*------------------------------------------------------ */

//-- 保存模板内容

/*------------------------------------------------------ */

elseif ($_REQUEST['act'] == 'save_template')

{

if (empty($_POST['subject']))

{

sys_msg($_LANG['subject_empty'], 1, array(), false);

}

else

{

$subject = trim($_POST['subject']);

}

if (empty($_POST['content']))

{

sys_msg($_LANG['content_empty'], 1, array(), false);

}

else

{

$content = trim($_POST['content']);

}

$type = intval($_POST['is_html']);

$tpl_id = intval($_POST['tpl']);

/*过滤了部分php关键词*/

$temp_check = preg_replace("/([^a-zA-Z0-9_]{1,1})+(extract|parse_str|str_replace|unserialize|ob_start|require|include|array_map|preg_replace|copy|fputs|fopen|file_put_contents|file_get_contents|fwrite|eval|phpinfo|assert|base64_decode|create_function|call_user_func)+( |\()/is", "", $content);

$temp_check = preg_replace("/<\?[^><]+(\?>){0,1}|<\%[^><]+(\%>){0,1}|<\%=[^><]+(\%>){0,1}|<script[^>]+language[^>]*=[^>]*php[^>]*>[^><]*(<\/script\s*>){0,1}/iU", "", $temp_check);

$content = $temp_check;

/*过滤了部分php关键词*/

$sql = "UPDATE " .$ecs->table('mail_templates'). " SET ".

"template_subject = '" .str_replace('\\\'\\\'', '\\\'', $subject). "', ".

"template_content = '" .str_replace('\\\'\\\'', '\\\'', $content). "', ".

"is_html = '$type', ".

"last_modify = '" .gmtime(). "' ".

"WHERE template_id='$tpl_id'";

if ($db->query($sql, "SILENT"))

{

$link[0]=array('href' => 'mail_template.php?act=list', 'text' => $_LANG['update_success']);

sys_msg($_LANG['update_success'], 0, $link);