搜索
系统检测到您的用户名不符合规范:

orderField方法存在sql注入漏洞

浏览:691 发布日期:2021年06月05日 分类:ThinkPHP6专区 关键字: sql注入
我发布在 github 上暂无人回复(https://github.com/top-think/think-orm/issues/266)

--------------------------------------

文件代码 https://github.com/top-think/think-orm/blob/0b01a19654c3b0376043960a30e74c9ddb55d574/src/db/Query.php#L54-L62 /**
* 指定Field排序 orderField('id',[1,2,3],'desc')
* @access public
* @param string $field 排序字段
* @param array $values 排序值
* @param string $order 排序 desc/asc
* @return $this
*/
public function orderField(string $field, array $values, string $order = '')
文件代码 https://github.com/top-think/think-orm/blob/0b01a19654c3b0376043960a30e74c9ddb55d574/src/db/Builder.php#L951-L977 /**
* orderField分析
* @access protected
* @param Query $query 查询对象
* @param string $key
* @param array $val
* @return string
*/
protected function parseOrderField(Query $query, string $key, array $val): string
{
if (isset($val['sort'])) {
$sort = $val['sort'];
unset($val['sort']);
} else {
$sort = '';
}

$sort = strtoupper($sort);
$sort = in_array($sort, ['ASC', 'DESC'], true) ? ' ' . $sort : '';
$bind = $query->getFieldsBindType();

foreach ($val as $item) {
$val[] = $this->parseDataBind($query, $key, $item, $bind);
}

return 'field(' . $this->parseKey($query, $key, true) . ',' . implode(',', $val) . ')' . $sort;
}
orderField 的底层解析 parseOrderField 代码中: foreach ($val as $item) {
$val[] = $this->parseDataBind($query, $key, $item, $bind);
}

return 'field(' . $this->parseKey($query, $key, true) . ',' . implode(',', $val) . ')' . $sort;
并没有把输入参数替换成值绑定(用的 $val[] 追加),导致后面的 implode(',', $val) 直接把原始值带入sql。

解决方法:把 foreach 改成: foreach ($val as $k => $item) {
$val[$k] = $this->parseDataBind($query, $key, $item, $bind);
}
最佳答案
评论() 相关
后面还有条评论,
评论支持使用[code][/code]标签添加代码
您需要登录后才可以评论 登录 | 立即注册
收藏
yangdx
积分:125 等级:LV1
热点推荐
(追記) (追記ここまで)
最新更新

我们

合作

网站

信息

ThinkPHP 是一个免费开源的,快速、简单的面向对象的 轻量级PHP开发框架 ,创立于2006年初,遵循Apache2开源协议发布,是为了敏捷WEB应用开发和简化企业应用开发而诞生的。ThinkPHP从诞生以来一直秉承简洁实用的设计原则,在保持出色的性能和至简的代码的同时,也注重易用性。并且拥有众多的原创功能和特性,在社区团队的积极参与下,在易用性、扩展性和性能方面不断优化和改进,已经成长为国内最领先和最具影响力的WEB应用开发框架,众多的典型案例确保可以稳定用于商业以及门户级的开发。

AltStyle によって変換されたページ (->オリジナル) /