这里是普通文章模块栏目内容页
源码级剖析PHP 7.2.x GD拒绝服务漏洞

php 7.2.x,开启gd库。只需要三行代码即可完成!

我在本地调试php的时候发现某个老代码能够直接把php给crash掉,因此成文。

1.PNG

php没有报错,直接死掉了,应该是内部逻辑有问题。再传到服务器上试试:

1-2.PNG

啊哈,一样的结果。触发这个问题的代码如下:

$im=imagecreate(100,100); imageantialias($im,true); imageline($im,0,0,10,10,0xffffff);

话不多说,上vs调试。先看调用堆栈吧。

2.PNG

从这里可以看出是在GD库的画像素点的地方出了错,被调试器断在了gdImageSetAAPixelColor这个函数里。 

3.PNG

再看对应代码,访问了gdImagePtr结构体中的一个成员,导致访问违例。我们再从即时窗口检查一下:

4.PNG

没错,tpixels是个空指针!

那tpixels是干啥的呢?在gd.h里面有如下说明:

/* Truecolor flag and pixels. New 2.0 fields appear here at the end to minimize breakage of existing object code. */ int trueColor; int ** tpixels;

看来是和真彩色相关的东西,我们再沿着调用堆栈往前看。

5.PNG

这里是gdImageAALine函数,一个个点地画线,干的是苦力活。从gdImageLine里调用了它:

6.PNG

这里的条件判断是是否开启了防锯齿功能。如果我们调用imageantialias函数打开这个功能,那么就会走这里来。

7.PNG

上面图里就是我们从php调用的imageline函数的实现啦,非常简单。可以看出图片是真彩色的时候它会默认开启防锯齿功能。

这里问题就在于,我们创建(imagecreate)的图片不是真彩色的图,而后我们手动开启了防锯齿(imageantialias),调用进去想当然地把它当作一张真彩色图,从而导致了错误。

最后我们来看看两个函数的不同:

8.PNG

跟进去,可以看到imagecreate函数调用的gdImageCreate里直接把真彩色相关的成员设为了null。

与之对比,imagecreatetruecolor函数调用的gdImageCreateTrueColor函数里为每个像素点都分配了对应内存并初始化为0了:

 

10-gdImageCreateTrueColor.PNG

 

总结一下,从上面分析可以看出,触发这个问题的条件有3个:

1.php版本为7.2.x且开启了gd库

2.创建了非真彩色图且开启抗锯齿

3.在创建的图句柄上进行像素点写入

导致这个问题的原因还是代码修改考虑不周全,引入了新的漏洞;没有对所有可能条件进行测试,所以从php 7.2.0一直到php 7.2.4都还存在问题。

收藏
0
有帮助
0
没帮助
0