简单说说如何编写安全的代码

前段时间看完了《代码整洁之道》,发现其中很多方法有助于去编写更加安全的代码。以前在阅读一些WEB App中,找到一些漏洞皆是在代码编写风格上出现的问题,比如,一个CMS有文章分类、文章显示、评论等等功能,在针对每一种功能的参数进行处理时,都单独做过参数过滤,也就是说,存在大量冗余的过滤代码,很可能在某一功能上可能是安全的,在另外一种功能上却存在漏洞。这种比较极端的例子还是很多的。因此:

在写函数时,我们尽可能保证函数代码的短小,代码越是长,出现的问题越可能变得很多,无论是功能上的bug,还是可能的安全隐患。那么如何把函数编写得更加短小呢?其实很简单,只让函数去处理一个功能,并且专心处理这个功能,其他与这个功能无关的,我们将它编写成另外的函数来调用。比如很多WEB APP这样实现的:

$id = $_GET['id];
//中间一长窜过滤的代码
$sql = 'select * from articles where id='.$id
$query = Mysql::query($sql)

然后在另外一处地方又出现了类似的不同功能的参数处理代码,因此对于这种情况,我们应该绝对保证:1、程序中绝对不能有重复功能的代码,将重复的全部写成一个函数或者放在类中;2、函数短小,将很多琐碎的分成其他函数;3、函数不要关心其他功能,只认真处理好本该完成的功能,这也是函数的本身具有的意义。也就是说,上方的例子中过滤代码可以单独写成一个函数,所有其他类似的地方都只应该调用这个函数,而不是每个地方都单独编写一长窜代码来过滤。

函数应该短小到多少呢,其实我也不清楚,五行到二十行就差不多了吧。目前我写的博客程序中,完全是面向对象写的,一个函数或者方法目前最多的行数目前是5行,普遍在一行和三行左右,或许以后还要稍微长一点点,但绝对不会太多。函数越是小,我们发现问题和处理问题的效率越是高——至少我这么觉得。顺便说句,需要给函数一个合理的、能够看函数名就知函数功能的命名,别弄得连自己都看不懂这个函数是干嘛的。

运用注释

记得在JAVA中有个“TODO”注释,好像在VS生成的代码中也有看到,大概意思表示打TODO注释的地方应该要完成什么什么样的功能。我写代码的时候,先把主要的精力投入在完成功能上,在可能涉及到安全问题的地方打一个TODO注释,最后根据注释来做处理。比如我博客的一个搜索功能:

public function searchKey($key)
{
    //TODO:过滤跨站和注入
    return self::getResultArray("select ar_title,ar_category,post_time,post_author from
    lx_articles where ar_title like '%$key%'");
}

最后再来解决安全问题。当然,这只是个人方法而已。

运用思维导图

思维导图是一种组织思维,将思维清晰表达出来的一种方式。看很多web app的时候,从代码风格上就可以看出他们的思维硬是混乱的,很极端的例子就如我上面说到的问题,太多冗余的安全处理代码。看这种程序会加大我找到漏洞的信心,我相信这里处理好了,另外一个地方多半都是有问题的。其实,那些处理代码很可能参考网上的,或者直接剽窃过来的。从这点就能知道该作者对安全方面的意识很薄弱。但是,我们还是可以装成很牛的样子,先对整体要完成的功能画一个思维导图,中间把可能出现的问题就顺路画出来了,类似于安全建模,但我们绘制的图更偏向于程序整体的构架。例如下面我针对搜索功能画的一个极其简单的图,该图可能随着我的新想法新点子随时改进:

1.jpg