欢迎您访问程序员文章站本站旨在为大家提供分享程序员计算机编程知识!
您现在的位置是: 首页  >  IT编程

浅析PHP框架Laravel最新SQL注入漏洞

程序员文章站 2022-03-20 16:54:46
PHP知名开发框架Laravel,之前在官方博客通报了一个高危SQL注入漏洞,这里简单分析下。 首先,这个漏洞属于网站coding写法不规范,官方给了提示: 但官方还是做了修补,升级最新版本V5.8.7可修复。 我们先定位下这里: 官方推荐的写法是: 如果网站coding没有预先对$id的值做处理时 ......

php知名开发框架laravel,之前在官方博客通报了一个高危sql注入漏洞,这里简单分析下。

浅析PHP框架Laravel最新SQL注入漏洞

首先,这个漏洞属于网站coding写法不规范,官方给了提示:

浅析PHP框架Laravel最新SQL注入漏洞

但官方还是做了修补,升级最新版本v5.8.7可修复。

我们先定位下这里:

illuminate\validation\rule

 

   

官方推荐的写法是:

rule::unique('users')->ignore($id),

 

   

如果网站coding没有预先对$id的值做处理时,用户可以直接传递恶意数据给ignore函数,就会导致sql注入。

我们来跟一下函数:

\illuminate\validation\rules\unique.php class unique {

... public function ignore($id, $idcolumn = null) { if ($id instanceof model) { return $this->ignoremodel($id, $idcolumn);

        } $this->ignore = $id; $this->idcolumn = $idcolumn ?? 'id'; return $this;

    }

 

   

这里我们不考虑把$id写成实例的情况,$id是用户可控的话,$idcolumn直接写为空即可,最后赋值情况如下:

$this->ignore = $id; $this->idcolumn = 'id';

 

   

如果网站代码类似这样构造的话,黑客输入的值就属于可控状态:

$id = $request->input('id');

 

   

最后我们会走到这儿:

illuminate\validation\rules\unique.php public function __tostring() {

        ...

        ...

    }

 

   

我们看下关键的代码变更:

illuminate\validation\rules\unique.php

v5.8.7【最新版】 public function __tostring() { $this->ignore ? '"'.addslashes($this->ignore).'"' : 'null',

    }

illuminate\validation\rules\unique.php

v5.8.4 public function __tostring() { $this->ignore ? '"'.$this->ignore.'"' : 'null',

    }

 

   

这里最新的代码v5.8.7,把$this->ignore直接给addslashes了,以前这里是没有防护的。

有趣的是,笔者对比了下diff,期间官方还试图对其他引用的地方进行过滤。最后还是在__tostring处,进行了统一的过滤。

最后提一句,后面的代码会进入databaserule,进行后续sql规则匹配。

illuminate\validation\rules\databaserule.php

 

   

这之后就没有再进一步处理,接着形成了sql注入。