• 复制黏贴的程序员要注意了!
  • 发布于 1个月前
  • 95 热度
    0 评论
  • 小宇宙
  • 0 粉丝 17 篇博客
  •   
张大胖最近接了一个C++的私活,在编程的时候需要生成一个字母和数字混合的随机字符串,张大胖不知道该怎么办,于是打开Google,输入关键字搜索,迅速在Stack Overflow找到了答案,还附有相关代码,他心中窃喜,直接复制粘贴过来,稍加修改,问题搞定,爽!

这可能是很多程序员都干过的事情,但是一个研究显示这么做是有风险的。 

1
最近一个计算机科学研究小组研究了Stack Overflow上过去十年的72483个C++代码片段,发现了69个有安全缺陷的代码片段,按照CWE (Common Weakness Enumeration) 的规范,被分成29类。 

从数量上来看,69个非常少,总数量的千分之一都不到,但是让人震惊的是这69个有安全缺陷的代码片段又出现在了2859个GitHub的开源项目中! 

很明显,这些代码都是程序员从Stack Overflow 复制粘贴过去的。 

拿文章开头张大胖的例子来讲, 他需要用C++生成一个字母数字混合的随机字符串,但是不知道怎么办,于是就到Stack Overflow上去搜索,他找到了一个ID为440240的答案,这个答案有265个赞同,被浏览了17万多次,应该是个好答案了吧! 

于是,张大胖“动心”了,他认为这是一段能解决问题的好代码,Copy , Paste 吧! 

然而这段代码是存在着大坑的。 

首先是字符串长度的问题,C/C++字符串以0结尾,带有'len'参数的函数应该把这一点考虑进来,但是这段代码只是简单的设置 s[len]=0 ,这就会导致不可预料的行为。 

其次rand()函数已经是一个过时的函数了,不应该被调用了,即使是被使用,也应该先调用srand()设置一个随机数种子。 

第三,使用rand() % N 试图来产生一个随机数字也不是最佳实践,因为很多随机数生成器生成数字的低字节并不是真正随机的。 

看起来很简单的代码却有很多安全的漏洞,如果贸然复制粘贴到自己的项目中,就是埋了一个地雷。 


2
其实,如果张大胖稍微细心一点,他就会看到这个答案有这么几个评论: 

实际上已经有人指出代码的错误了,Velkan还明确地说,Stack Overflow应该提供一种机制,让这些过时的、废弃的答案“沉”下去,不能在这里误导大家了。 

但是有多少Copy&Paste程序员会细心地看这些评论呢?

这个研究小组的科学家们非常贴心,专门开发了一个Chrome插件,当程序员去查看这些有安全缺陷的代码时,插件就会提示:这段代码有安全漏洞! 试图阻止复制粘贴,并且提供一个更好的解决方案的链接。

3
研究小组的科学家们就这些安全漏洞联系了GitHub项目的开发人员, 从回复来看,只有13.3%的人说他们已经Fix了,有40%的人承认这些漏洞的可能性,但是他们认为输入数据不是动态的,没有安全风险。还有13.3%的人根本不愿意修改。可见安全问题任重而道远啊! 

希望能给Copy & Paste程序员敲响一次警钟吧!

参考资料:
https://arxiv.org/pdf/1910.01321.pdf?
https://stackoverflow.com/questions/440133/how-do-i-create-a-random-alpha-numeric-string-in-c/440240#440240
http://c-faq.com/lib/randrange.html
用户评论