一个strtok的奇怪问题

mudga

有下面代码，功能是分割每个email地址

1. #include <stdio.h>;
   
2. #include <string.h>;
   
3. 
   
4. int main()
   
5. {
   
6.         char *s = "dd@163.net;xx@citiz.net;mudga@hotmail.com";
   
7.         char *delim = ";";
   
8.         char *content[5];
   
9.         int content_num = 0;
   
10.         int i;
    
11. 
    
12.         content[0] = strtok(s,delim);
    
13.         for(i = 1;i < 5; i++)
    
14.         {
    
15.                 content[i] = strtok(NULL,delim);
    
16.                 content_num++;
    
17.                 if (!content[i] )
    
18.                         break;
    
19.         }
    
20.         for (i=0; i<content_num; i++)
    
21.         {
    
22.                 printf("content[%d]:%s",i,content[i]);
    
23.         }
    
24.         return 0;
    
25. }

复制代码

编译后出现segment fault错误，环境是rh+gcc
但是奇怪的是，我如果在gdb下面调试这个程序时，不设断点，还是出现上面的错误，说是strtok出错；如果在strtok处设置断点的话，程序完全正常！这是什么原因呢？

superdoctor

给你一个正确的例子，取第一个邮件地址

1. 
   
2. #include <stdio.h>;
   
3. #include <string.h>;
   
4. 
   
5. int main()
   
6. {
   
7.    char s[] = "dd@163.net;xx@citiz.net;mudga@hotmail.com";
   
8.    char *delim = ";";
   
9. 
   
10.    strtok(s,delim);
    
11.    printf("%s\n", s);
    
12.    return 0;
    
13. }
    

复制代码

运行结果为dd@163.net
希望对你有启发

gangcai

char *content[5];

你定义了这个二维数组，但是没有分配内存啊。当然会出core了
content=(char *)malloc( size*5);
size 自己定吧

virmin

不是指针数组的分配问题，
是char *s="dd@......."这里的问题，
strtok是会修改目标字符串的，用char *s="....."的形式分配来保存字符串的空间是只读的，改成char s[] = "...."就可以了

unicorns

楼上正解.
strtok()这东西,能不用就不用.

mudga

[quote]原帖由 "virmin"]."就可以了[/quote 发表：

我试了一下，果然是这个问题。那我在调试设断点的时候就没有这个问题，是不是意味着gdb内部会为s重新分配了一个可以修改数据的内存？

mudga

原帖由 "gangcai" 发表：
愣ㄒ辶苏飧龆???椋??敲挥蟹峙淠诖姘　５比换岢鯿ore了
content=(char *)malloc( size*5);
size 自己定吧

char *content[5]不需要malloc吧，因为已经是数组了，定义后就已经有存储空间了

aero

[quote]原帖由 "virmin"]."就可以了[/quote 发表：


数组空间的分配也存在问题。
而且应该用strcpy将得到的token复制出去。

mudga

哪里还有问题？char *content[5]应该没有问题吧？

aero

virmin 说的没错。
你现在的程序看起来没有问题，可是非常危险。
因为你的char *content[5]中的5个元素全使指针，而这指针都使指的s中的不同位置。如果s改变或者销毁的话，那么这个5个就全使危险的元素。

如果你仅仅使为了如上的程序，打印出来，那么现在还没有问题。如果你想在以后，进一步应用char *content[5]。那么应当如virmin所说的，为他们开辟空间，并将内容复制一份。这样，char *content[5]才使安全的。