这个函数里面的if() 是不是写的太罗嗦了？

2005227042

asdf2110 发表于 2013-07-19 15:07
1.  static 变量只初始化一次，所以第2，3，。。。，n次进入函数时，p的指向就是 buffer + n 了
2.  参考1 ...

5楼的说法正确，补充1:  static char buffer[1024];//是比malloc分配内存更理想的方式，申请释放由编译器和系统处理;也比全局变量分配方式更能限定作用域。这种应用场景在C语言环境下，还是比较理想。
              补充2: if (p + size >= buffer + sizeof (buffer))   abort ();//用于防止no_prefix（..)函数被超出次数调用,也防止strcpy内存访问出错。因为在函数注释中有"allocated off static storage at Wget startup."  ,调用函数上下文对函数调用环境做了限定。同时这个函数不是线程安全的, 当然注释中显示在wget startup使用，应该是单线程环境。

djsxut

按六楼说的“ 标准要求指针的加减法得到的结果必须仍指向数组元素或数组最后一个元素之后的第一个位置，否则是未定义行为”：
7 楼的用法蛮好:
buffer + sizeof(buffer) - p > size

yulihua49

pmerofc 发表于 2013-07-22 21:51
武断
你怎么知道“从没问题”？
本版就有人因为这个错误在产品升级时出现过问题

7楼那个很好，但是相对原式只是代数变换，并无实质不同。
你怎么证明原来错误？
如果原来错误，你又有什么高招避免该错误？
wget已经是实用的了，除了线程不安全，有别的错吗？


再重复一次，指针，只要不访问其内容，指向任何位置都没问题。
最简单的证明：
if（point == NULL） 。。。。最常用的了吧，一旦point==NULL，该语句有错吗？
更极端的用法：
point=（char *）-1L；
if(-1 == (long)point) {//千万别用*point
。。。。
没有问题。请放心使用。
它的潜在风险是,后边的代码如果只是进行这个判决：
if(point != NULL) {
        // 使用了*point
就出轨了。
但凡这种用法，一定会有一个使用说明：
返回值：
point==NULL，非致命错误；
point==-1，致命错误。

你说出的那个问题，一定是在某处用到那个内容才出的错。

关键是你不懂汇编，不知道C语句会怎样操作指令，哪条指令会引起异常。

最后，给你开开眼：
一段连接池代码：

1.         if(rs->Conn.Socket<0 || rs->cli.Errno<0) {
   
2.                 ret=sc_connect(pl,rs);
   
3.                 if(ret) {
   
4.                         ShowLog(1,"%s:scpool[%d].%d 连接%s/%s错:err=%d,%s",
   
5.                                 __FUNCTION__,n,i,pl->log.HOST,pl->log.PORT,
   
6.                                 rs->cli.Errno, rs->cli.ErrMsg);
   
7.                         rs->TCBno=-1;
   
8.                         rs->cli.Errno=-1;
   
9.                         pthread_mutex_lock(&pl->mut);
   
10.                         add_lnk(pl,i);
    
11.                         pthread_mutex_unlock(&pl->mut);
    
12.                         return (T_Connect *)-1;
    
13.                 }
    
14.         }
    
15. 
    
16. 后边的：
    
17. 
    
18. *connp=get_SC_connect(TCBno,poolno,1);
    
19.         if(*connp == (T_Connect *)-1) {
    
20.                 *connp=NULL;
    
21.                 return -1;
    
22.         }
    
23. 
    

复制代码

都是经过充分测试的代码，生产项目使用。

yulihua49

pmerofc 发表于 2013-07-23 15:54
回复 23# yulihua49

好吧，你对了。
7楼的程序没问题。
我赶紧检查了所有的程序，有类似需求，还真没有这种用法。
幸亏。
目前那些程序安全。