Redian新闻
>
嵌入式 C 语言史上最愚蠢的一个 BUG

嵌入式 C 语言史上最愚蠢的一个 BUG

公众号新闻

来自:酷 壳 - CoolShell   链接:https://coolshell.cn/articles/5388.html

原文:https://www.elpauer.org/2011/08/the-most-stupid-c-bug-ever/

我相信这样的bug,就算你是高手你也会犯的,你来看看作者犯的这个Bug吧。。
首先,作者想用一段程序来创建一个文件,如果有文件名的话,就创建真正的文件,如果没有的话,就调用tmpfile()创建临时文件。
他这段程序就是HTTP下载的C程序。code==200就是HTTP的返回码。
else if (code == 200) {     // Downloading whole file
    /* Write new file (plus allow reading once we finish) */
    g = fname ? fopen(fname, "w+") : tmpfile();
}
但是这个程序,只能在Unix/Linux下工作,因为 Microsoft 的tmpfile()的实现居然选择了 C:\ 作为临时文件的存放目录,这对于那些没有管理员权限的人来说就出大问题了。在Windows 7下,就算你有管理员权限也会有问题。
所以,上面的程序在Windows平台下需要用不同的方式来处理,不能直接使用Windows的tmpfile()函数。
于是作者就先把这个问题记下来,在注释中写下了FIXME:
else if (code == 200) {     // Downloading whole file
    /* Write new file (plus allow reading once we finish) */

    // FIXME Win32 native version fails here because
    //   Microsoft's version of tmpfile() creates the file in C:\
    g = fname ? fopen(fname, "w+") : tmpfile();
}
然后,作者觉得需要写一个跨平台的编译:
FILE * tmpfile void ) {
#ifndef _WIN32
    return tmpfile();
#else
    //code for Windows;
#endif
}
然后,作者觉得这样实现很不好,会发现名字冲突,因为这样一来这个函数太难看了。
于是他重构了一下他的代码——写一个自己实现的tmpfile() – w32_tmpfile,然后,在Windows 下用宏定义来重命名这个函数为tmpfile()。(注:这种用法是比较标准的跨平台代码的写法)
#ifdef _WIN32
  #define tmpfile w32_tmpfile
#endif

FILE * w32_tmpfile void ) {
    //code for Windows;
}
搞定!编译程序,运行。
靠!居然没有调用到我的w32_tmpfile(),什么问题?调试,单步跟踪,果然没有调用到!
难道是问号表达式有问题?改成if – else 语句,好了!
if(NULL != fname) {
    g = fopen(fname, "w+");
else {
    g = tmpfile();
}
问号表达式不应该有问题吧,难道我们的宏对问号表达式不起作用,这难道是编译器的预编译的一个bug?作者怀疑到。
现在我们把所有的代码连在一起看,并比较一下:
能正常工作的代码
#ifdef _WIN32
#  define tmpfile w32_tmpfile
#endif

FILE * w32_tmpfile void ) {
    code for Windows;
}

else if (code == 200) {     // Downloading whole file
    /* Write new file (plus allow reading once we finish) */
    // FIXME Win32 native version fails here because
    //     Microsoft's version of tmpfile() creates the file in C:\
    //g = fname ? fopen(fname, "w+") : tmpfile();
    if(NULL != fname) {
        g = fopen(fname, "w+");
    } else {
        g = tmpfile();
    }
}
不能正常工作的代码
#ifdef _WIN32
#  define tmpfile w32_tmpfile
#endif

FILE * w32_tmpfile void ) {
    code for Windows;
}

else if (code == 200) {     // Downloading whole file
    /* Write new file (plus allow reading once we finish) */
    // FIXME Win32 native version fails here because
    //    Microsoft's version of tmpfile() creates the file in C:\
    g = fname ? fopen(fname, "w+") : tmpfile();
}
也许你在一开始就看到了这个bug,但是作者没有。所有的问题都出在注释上:
/* Write new file (plus allow reading once we finish) */
// FIXME Win32 native version fails here because
//     Microsoft's version of tmpfile() creates the file in C:\
你看到了最后那个C:\吗?在C中,“\” 代表此行没有结束,于是,后面的代码也成了注释。这就是这个bug的真正原因
而之所以改成if-else能工作的原因是因为作者注释了老的问号表达式的代码,所以,那段能工作的代码成了:
/* Write new file (plus allow reading once we finish) */
// FIXME Win32 native version fails here because Microsoft's version of tmpfile() creates the file in C:    //g = fname ? fopen(fname, "w+") : tmpfile();
if(NULL != fname) {
    g = fopen(fname, "w+");
else {
    g = tmpfile();
}
我相信,当作者找到这个问题的原因后,一定会骂一句“妈的”!我也相信,这个bug花费了作者很多时间!
最后,我也share一个我以前犯的一个错。
我有一个小函数,需要传入一个int* pInt的类型,然后我需要在我的代码里 把这个int* pInt作除数。于是我的代码成了下面的这个样子:
float result = num/*pInt; ….
/*  some comments */
-x<10 ? f(result):f(-result);
因为我在我当时用vi编写代码,所以没有语法高亮,而我的程序都编译通过了,但是却出现了很奇怪的事。
我也不知道,用gdb调式的时候,发现有些语句直接就过了。
这个问题让我花了很多时间,最后发现问题原来是没有空格导致的,TNND,下面我用代码高亮的插件来显示上面的代码,
float result = num/*pInt;
....

/*  some comments */


-x<10 ? f(result):f(-result); 
Holly Shit!  我的代码成了:
float result = num-x<10 ? f(result):f(-result);
妈的!我的这个错误在愚蠢程度上和上面那个作者出的错误有一拼。

END

官方站点:www.linuxprobe.com

Linux命令大全:www.linuxcool.com

刘遄老师QQ:5604215

Linux技术交流群:3861509

(新群,火热加群中……)

想要学习Linux系统的读者可以点击"阅读原文"按钮来了解书籍《Linux就该这么学》,同时也非常适合专业的运维人员阅读,成为辅助您工作的高价值工具书!


微信扫码关注该文公众号作者

戳这里提交新闻线索和高质量文章给我们。
相关阅读
谁知盘中餐:我们费心费力研究的真是“人类愚蠢的分子机理”吗?巴菲特:两年前减持苹果是愚蠢的决定;陆奇:中国大模型要先赶上 GPT-3.5;羊了个羊侵害用户权益被通报 | 极客早知道那一年,母亲走了认识bug,理解bug,爱上debug!大二科研小白,发论文啦!嵌入式BSP工程师到底是干嘛的"愚蠢的华人,我要杀了你"纽约华男半夜下班回家被人从背后狂揍,钱包,车钥匙统统被抢。好日子开始啦!这3种优雅的嵌入式软件架构,你值得拥有!"愚蠢的华人, 我要杀了你"!纽约华男半夜下班回家被人从背后狂揍, 钱包, 车钥匙统统被抢...2023 嵌入式现状酒店Bug价:凯悦难得出Bug,10欧元入住全新收购的酒店品牌最愚蠢的数学教育莫过于是......搞嵌入式,也有35岁危机?网友展示“嵌入式垃圾桶”,正准备挑毛病,结果…好可爱哈哈天,刚才在自己身上发现了 bed bug嵌入式:听说,嵌入式这个行业未来十年都不会卷?我要吐槽邮轮,常在河边走终于湿鞋了。两个小孩相继中了stomach bug,吐一天,拉一天,估计以后再也不坐邮轮了全场景生态走向深入,openEuler 首批嵌入式商业版正式发布好日子开始啦!嵌入式 BSP 工程师到底是干嘛的嵌入式生成AI将为游戏角色提供动力这3句话,聪明人烂在肚子里,愚蠢的人却到处说!本周开幕|embedded world中国首展暨上海国际嵌入式大会精彩内容抢先看!金朝最愚蠢的皇帝!国家危亡还四处树敌,险成亡国之君"愚蠢的华人,我要杀了你"纽约华男半夜下班回家被人从背后狂揍,钱包,车钥匙统统被抢拜登被记者问及腐败指控,直斥这是一个“愚蠢的问题”知乎高赞:Java和嵌入式,选哪个?老版孙猴子扮相的一个小 Bug取消终身教职成趋势?美媒:这些愚蠢的努力导致人才流失黄立平:用复制思维做园区是愚蠢的详细讲解MMU——为什么嵌入式 linux 没他不行?花粉病传播中国文化,推广中国诗歌“亚当斯定律”:为什么最聪明的人总是做出最愚蠢的决策?周三开幕!上海国际嵌入式大会精彩内容抢先看!其实吧,学嵌入式也不是很难。“愚蠢的问题”!拜登怒呛记者走嵌入式方向,一定要软硬件都懂吗?
logo
联系我们隐私协议©2024 redian.news
Redian新闻
Redian.news刊载任何文章,不代表同意其说法或描述,仅为提供更多信息,也不构成任何建议。文章信息的合法性及真实性由其作者负责,与Redian.news及其运营公司无关。欢迎投稿,如发现稿件侵权,或作者不愿在本网发表文章,请版权拥有者通知本网处理。