移植dns-proxy的一段代码，字符转换，偶尔会死掉，就是不知道哪里出错了，请大家帮忙

killdog1982

void dns_decode_name(char *name, char **buf)
{
     int i, k, len, j;

     i = k = 0;
     while( **buf )
     {
         len = *(*buf)++;
         if (len <=0)
             break;
   
         for( j = 0; j<len && len<NAME_SIZE; j++)
              name[k++] = *(*buf)++;
         name[k++] = '.';
     }
     //name[k-1] = *(*buf)++;
     (*buf)++;
     if( k > 0 )
     {
           name[k-1] = 0x00;
     }
     else
     {
          name[0] = 0x00;
     }
}

这段代码是dns-proxy里面一段解析DNS域名的代码，一般不会出错，但是偶尔程序会在这里死掉，
就是不知道这段代码哪里写错了，请大侠们赐教

buf为输入: 如 输入字符串为  (3)www(5)baidu(3)com(0), 那么解析后的输出到name里为www.baidu.com,
注意输入里的3表示后面有3个字符，然后就是5，后面有5个字符.

bruceteen

代码很烂
len = *(*buf)++; 从这一句上是用一个char来保存长度，但char是有符号数还是无符号数是实现定义的
for( j = 0; j<len && len<NAME_SIZE; j++) 一来没必要每次循环都判断，二来达到NAME_SIZE时会增加个点号，怎么看都是个算法bug
(*buf)++; 这个属于设计错误，你这个函数就不应该干这事，接口也可以简化一下

回正题^_^，偶尔死掉，可能是 name的空间不足

killdog1982

忘了这句, NAME_SIZE 是256.
多谢楼上回复，不过dns域名一般不会超过256长度的。

linux_c_py_php

好烂的代码。