是realloc 引起的问题么?

shiner

1. 
   
2. #include <stdio.h>
   
3. #include <malloc.h>
   
4. #define MAXLINE 10
   
5. 
   
6. int getline(char *);
   
7. void copy(char *, char *);
   
8. char *line;
   
9. char *longest;
   
10. 
    
11. int main(void)
    
12. {
    
13.         int len;
    
14.         int max;  
    
15.         line = (char *)malloc(MAXLINE);
    
16.         longest = (char *)malloc(MAXLINE);
    
17.        
    
18.         if(!line && !longest)
    
19.                 perror("main malloc ");
    
20.                
    
21.         max = 0;
    
22.        
    
23.         while ((len = getline(line)) > 0)
    
24.                 if (len > max) {
    
25.                         max = len;
    
26.                         copy(longest, line);
    
27.                 }
    
28.                
    
29.         if (max > 0)
    
30.                 printf("%s\n", longest);
    
31.                
    
32.         free(line);
    
33.         free(longest);
    
34.                
    
35.         return 0;
    
36. }
    
37. 
    
38. int getline(char *line)
    
39. {
    
40.         int c, i;
    
41.        
    
42.         for (i=0;(c=getchar())!= EOF && c != '\n'; ++i){
    
43.                 if(i > MAXLINE - 1)
    
44.                         line = (char *)realloc(line,MAXLINE + sizeof(char));
    
45. 
    
46.                 if(!line){
    
47.                         perror("getline realloc ");
    
48.                         return 0;
    
49.                 }
    
50.                 *(line + i) = c;
    
51.         }
    
52.        
    
53.         if(i > MAXLINE - 1)
    
54.                 line = (char *)realloc(line,MAXLINE + sizeof(char));
    
55.        
    
56.         if(!line){
    
57.                         perror("getline realloc ");
    
58.                         return 0;
    
59.                 }
    
60.                
    
61.         *(line + i) = '\0';
    
62.         return i;
    
63. }
    
64. 
    
65. void copy(char *longest, char *line)
    
66. {
    
67.         int i;
    
68.        
    
69.         for(i = 0; *(line + i) != '\0'; i++){
    
70.                 if(i > MAXLINE - 1)
    
71.                                 longest = (char *)realloc(longest,MAXLINE + sizeof(char));
    
72.                                
    
73.                 if(!longest){
    
74.                         perror("copy realloc ");
    
75.                         return ;
    
76.                 }
    
77.                
    
78.                 *(longest + i) = *(line + i);
    
79.         }
    
80.        
    
81.         if(i > MAXLINE - 1)
    
82.                 longest = (char *)realloc(longest,MAXLINE + sizeof(char));
    
83.        
    
84.         if(!longest){
    
85.                         perror("copy realloc ");
    
86.                         return ;
    
87.                 }
    
88.                
    
89.         *(longest + i) = '\0';
    
90. }
    
91. 
    

复制代码

老问题,新提法... 　
说说我的问题吧,在输入字符串大于 10 时就会 core dumped 掉。经过我反复查证是由于copy函数中的　for(i = 0; *(line + i) != '\0'; i++)　在*(line + i) ＝＝'\0'时没有跳出循环，无限的在循环下去，直到超过系统规定最大值限制时就　core dumped　掉。
很郁闷到底是怎么引起的，难道是realloc引起的？

win_hate

1. 
   
2. int getline(char *line)
   
3. {
   
4.         int c, i;
   
5.       
   
6.         for (i=0;(c=getchar())!= EOF && c != '\n'; ++i){
   
7.                 if(i > MAXLINE - 1)
   
8.                         line = (char *)realloc(line,MAXLINE + sizeof(char));
   
9. 
   
10.                 if(!line){
    
11.                         perror("getline realloc ");
    
12.                         return 0;
    
13.                 }
    
14.                 *(line + i) = c;
    
15.         }
    
16.       
    
17.         if(i > MAXLINE - 1)
    
18.                 line = (char *)realloc(line,MAXLINE + sizeof(char));
    
19.       
    
20.         if(!line){
    
21.                         perror("getline realloc ");
    
22.                         return 0;
    
23.                 }
    
24.                
    
25.         *(line + i) = '\0';
    
26.         return i;
    
27. }
    

复制代码

看下面：


MAXLINE 是个常量，MAXLINE+sizeof(char) 就是 11

if(i > MAXLINE - 1)
                line = (char *)realloc(line,MAXLINE + sizeof(char));   ／／ MAXLINE 是常量，realloc 没有真正分配新的空间

*(line + i) = c;          // i 比较大时越界

*(line + i) = '\0';      // i 比较大时越界

[ 本帖最后由 win_hate 于 2007-11-27 18:48 编辑 ]

sanbiangongzi

1. 
   
2. char * get_line()
   
3. {
   
4.         char *str;
   
5.         int ch;
   
6.         unsigned int bufsize;
   
7.         unsigned int i;
   
8. #define BUFFER_INCREMENT (4*1024)
   
9. 
   
10.         bufsize = BUFFER_INCREMENT;
    
11.         str = malloc(bufsize);
    
12.         if (NULL == str) {
    
13.                 goto exitinput;
    
14.         }
    
15. 
    
16.         _putch(':');
    
17.         i = 0;
    
18. getnextchar:
    
19.         while (i < bufsize) {
    
20.                 ch = _getch();
    
21. 
    
22.                 if (EOF == ch
    
23.                 || 0x04 == ch
    
24.                 || '\r' == ch) {
    
25.                         /* End input if encounter
    
26.                          * 1.End Of File
    
27.                          * 2.CTRL+D
    
28.                          * 3.Carrier Return
    
29.                          */
    
30.                         _putch('\n');
    
31.                         str[i] = '\0';
    
32.                         goto exitinput;
    
33.                 } else if ('\b' == ch) {
    
34.                         if (i>0) {
    
35.                                 _putch('\b');
    
36.                                 _putch(' ');
    
37.                                 _putch('\b');
    
38.                                 i--;
    
39.                         }
    
40.                         continue;
    
41.                 }
    
42.                 _putch(ch);
    
43.                 str[i++] = ch;
    
44.         }
    
45. 
    
46.         bufsize += BUFFER_INCREMENT;
    
47.         str = realloc(str, bufsize);
    
48.         if (NULL != str) {
    
49.                 goto getnextchar;
    
50.         }
    
51. exitinput:
    
52.         return str;
    
53. }
    

复制代码

shiner

谢谢老大，我明白了

shiner

谢谢你！非常感谢！

[ 本帖最后由 shiner 于 2007-11-28 11:18 编辑 ]