请教segmentation fault

slowhand

Centos 5.5 X64

如果 MAXDIRLOGSIZE 大于 1499999 就执行 segmentation fault ，同样的在 freebsd 下就ok。

请问修改centos那个参数 可以避免 segmentation fault ，谢谢

代码：

1. int replace_dirlog = 0;      /* change this to 1 if you want to replace
   
2.                                 the old dirlog with the new one by default   */
   
3. int delete_orig_dirlog = 0;  /* set this to 1 if you don't want a backup
   
4.                                 of the original dirlog                       */
   
5. 
   
6. #define MAXDIRLOGSIZE 10000
   
7. 
   
8. #include <sys/param.h>
   
9. #include <sys/stat.h>
   
10. #include <sys/types.h>
    
11. 
    
12. #include <stdio.h>
    
13. #include <stdlib.h>
    
14. #include <string.h>
    
15. #include <ctype.h>
    
16. #include <unistd.h>
    
17. #include "glconf.h"
    
18. 
    
19. char datapath[255] = "/ftp-data";        /* assign default datapath, will use value from conf if defined there */
    
20. char rootpath[255] = "/glftpd";                /* assign default rootpath, will use value from conf if defined there */
    
21. char config_file[MAXPATHLEN];
    
22. 
    
23. void trim(char *str) {
    
24.     char *ibuf, *obuf;
    
25. 
    
26.     if (str) {
    
27.         for (ibuf = obuf = str; *ibuf;) {
    
28.             while (*ibuf && (isspace(*ibuf)))
    
29.                 ibuf++;
    
30.             if (*ibuf && (obuf != str))
    
31.                 *(obuf++) = ' ';
    
32.             while (*ibuf && (!isspace(*ibuf)))
    
33.                 *(obuf++) = *(ibuf++);
    
34.         }
    
35.         *obuf = '\0';
    
36.     }
    
37.     return;
    
38. }
    
39. 
    
40. void load_sysconfig(void) {
    
41.     FILE *configfile;
    
42.     char lvalue[64], rvalue[MAXPATHLEN], work_buff[MAXPATHLEN];
    
43.     int  x, y;
    
44. 
    
45.     if ((configfile = fopen(config_file, "r")) == NULL) {
    
46.         fprintf(stderr, "Unable to open config file %s, using defaults.\n", config_file);
    
47.         return;
    
48.     }
    
49. 
    
50.     while (fgets(work_buff, sizeof(work_buff), configfile) != NULL) {
    
51.       for (x = 0; x < (int)strlen(work_buff); x++)  /* Clip out comments */
    
52.          if (work_buff[x] == '#')
    
53.             work_buff[x] = '\0';
    
54. 
    
55.       trim(work_buff);
    
56.       memset(lvalue, '\0', sizeof(lvalue));
    
57.       memset(rvalue, '\0', sizeof(rvalue));
    
58. 
    
59.       y = 0;
    
60.       for (x = 0; x < (int)strlen(work_buff) && work_buff[x] != ' '; x++)
    
61.          if (isprint(work_buff[x]))
    
62.             lvalue[y++] = work_buff[x];
    
63. 
    
64.       y = 0; x++;
    
65.       for (; x < (int)strlen(work_buff); x++)
    
66.          if (isprint(work_buff[x]))
    
67.             rvalue[y++] = work_buff[x];
    
68. 
    
69.       if (strcasecmp(lvalue, "datapath") == 0)
    
70.                 strncpy( datapath, rvalue, sizeof(datapath));
    
71.       if (strcasecmp(lvalue, "rootpath") == 0)
    
72.                 strncpy( rootpath, rvalue, sizeof(rootpath));
    
73.     }
    
74. 
    
75.     fclose(configfile);
    
76.     return;
    
77. }
    
78. void show_help(char *argv) {
    
79.         printf("\nThis util removes non-existent directories from your dirlog and\n");
    
80.         printf(" sorts it so that newest directories show up first in \"site new\".\n");
    
81.         printf("\nUsage: %s [options]\n",argv);
    
82.         printf("Options:\n");
    
83.         printf("         -r path   Specify alternate path to config file\n");
    
84.         printf("         -P        Force replacing of dirlog\n");
    
85.         printf("         -p        Force not replacing of dirlog [default]\n");
    
86.         printf("         -D        Force deleting of old dirlog\n");
    
87.         printf("         -d        Force keeping of old dirlog [default]\n");
    
88.         printf("\n");
    
89.         exit(0);
    
90. }
    
91. 
    
92. int main(int argc, char *argv[]) {
    
93.   FILE *file, *file2;
    
94.   uid_t oldid = geteuid();
    
95.   char dirlogpath[MAXPATHLEN], newdirlog[MAXPATHLEN], testdir[MAXPATHLEN+8];
    
96.   struct dirlog direntry, *dirptr, *dirlist[MAXDIRLOGSIZE];
    
97.   struct stat st;
    
98.   int curr=0, curr2, ArgCounter;
    
99. 
    
100.   strcpy(config_file, GLCONF);
     
101. 
     
102.   printf("Olddirclean for glftpd, v2.2 by Usurper, November 2000\n");
     
103.   printf("Use option -h (%s -h) for syntax help.\n", argv[0]);
     
104. 
     
105.   ArgCounter = 1;
     
106.   while (argc > ArgCounter) {
     
107.         if (argv[ArgCounter][0] != '-') {
     
108.           printf("Unknown option, skipping [%s]\n",argv[ArgCounter]);
     
109.         } else {
     
110.           switch (argv[ArgCounter][1]) {
     
111.                 case 'r':
     
112.                     if (argv[ArgCounter][2])
     
113.                       strcpy(config_file, argv[ArgCounter]+2);
     
114.                     else if (argv[ArgCounter+1])
     
115.                       strcpy(config_file, argv[++ArgCounter]);
     
116.                     else
     
117.                       printf("No argument to the r option\n");
     
118.                     break;
     
119.                 case 'P':
     
120.                     replace_dirlog = 1;
     
121.                     break;
     
122.                 case 'p':
     
123.                     replace_dirlog = 0;
     
124.                     break;
     
125.                 case 'D':
     
126.                     delete_orig_dirlog = 1;
     
127.                     break;
     
128.                 case 'd':
     
129.                     delete_orig_dirlog = 0;
     
130.                     break;
     
131.                 case 'h':
     
132.                 case '?':
     
133.                     show_help(argv[0]);
     
134.                     break;
     
135.                 default:
     
136.                     printf("Unknown option, skipping [%s]",argv[ArgCounter]);
     
137.             } /* switch */
     
138.         } /* else */
     
139.         ArgCounter++;
     
140.   } /* while */
     
141. 
     
142.   load_sysconfig();
     
143.   sprintf(dirlogpath,"%s%s/logs/dirlog", rootpath, datapath);
     
144.   printf("\nReading %s\n", dirlogpath);
     
145. 
     
146.   seteuid(0);
     
147.   if ((file = fopen(dirlogpath, "r+b")) == NULL) {
     
148.     fprintf(stderr, "Unable to open dirlog\n");
     
149.     return 0;
     
150.   }
     
151.   snprintf(newdirlog, sizeof(newdirlog), "%s2", dirlogpath);
     
152.   if((file2 = fopen(newdirlog, "w+b")) == NULL) {
     
153.     fprintf(stderr, "Unable to create dirlog2\n");
     
154.     return 0;
     
155.   }
     
156.   while(!feof(file)) {
     
157.      if (fread(&direntry, sizeof(struct dirlog), 1, file) < 1)
     
158.        break;
     
159.      snprintf(testdir, sizeof(testdir),"%s%s", rootpath, direntry.dirname);
     
160.      if (stat(testdir, &st) != 0)
     
161.        printf("Removing %s\n",direntry.dirname);
     
162.      else {
     
163.     /*  fwrite(&direntry, sizeof(struct dirlog), 1, file2);  */
     
164.         dirlist[curr] = (struct dirlog *)malloc(sizeof (struct dirlog));
     
165.         *dirlist[curr++] = direntry;
     
166.         if (curr >= MAXDIRLOGSIZE) {
     
167.           printf("Too many records in dirlog - recompile %s with MAXDIRLOGSIZE set to a greater value!\n", argv[0]);
     
168.           exit(1);
     
169.         }
     
170.      }
     
171.   }
     
172.   fclose(file);
     
173.   printf("\nSorting dirlog");
     
174.   for (curr=0; dirlist[curr] != NULL; curr++) {
     
175.     for (curr2=curr+1; dirlist[curr2] != NULL; curr2++) {
     
176.       if (dirlist[curr]->uptime > dirlist[curr2]->uptime) {
     
177.         dirptr = dirlist[curr];
     
178.         dirlist[curr] = dirlist[curr2];
     
179.         dirlist[curr2] = dirptr;
     
180.       }
     
181.     }
     
182.     printf(".");
     
183.   }
     
184.   printf("\n\nWriting new dirlog...\n");
     
185.   for (curr=0; dirlist[curr] != NULL; curr++)
     
186.     fwrite(dirlist[curr], sizeof(struct dirlog), 1, file2);
     
187.   fclose(file2);
     
188.   if (replace_dirlog) {
     
189.     /* using testdir as a temporary buffer */
     
190.     snprintf(testdir, sizeof(testdir), "%stmp", dirlogpath);
     
191.     if (rename(dirlogpath, testdir) != 0) {
     
192.       fprintf(stderr, "Unable to rename %s!\n", dirlogpath);
     
193.       return 0;
     
194.     }
     
195.     if (rename(newdirlog, dirlogpath) != 0) {
     
196.       fprintf(stderr, "Unable to rename %s!\n", newdirlog);
     
197.       return 0;
     
198.     }
     
199.     if (rename(testdir, newdirlog) != 0) {
     
200.       fprintf(stderr, "Unable to rename %s!\n", testdir);
     
201.       return 0;
     
202.     }
     
203.     printf("Dirlog replaced.\n");
     
204.     if (delete_orig_dirlog) {
     
205.       if (unlink(newdirlog) != 0)
     
206.         fprintf(stderr, "Unable to delete %s\n", newdirlog);
     
207.       else
     
208.         printf("%s deleted.\n",newdirlog);
     
209.     }
     
210.   } else
     
211.     printf("Done. Rename %s to %s for changes to take effect.\n",newdirlog,dirlogpath);
     
212.   setuid(oldid);
     
213.   return 0;
     
214. }

复制代码

hellioncu

栈溢出了，那个大数组改用动态申请

slowhand

怎么修改呢？

zhanglistar

回复 3# slowhand


    malloc

langue

Segmentation fault 是不应该避免的，接收到这个信号表明程序存在严重的错误，比如越界或者方式不正确的内存访问。

从第 96 行可以找到一处定义：

struct dirlog direntry, *dirptr, *dirlist[MAXDIRLOGSIZE];

显然，这个 dirlist 是定义在函数内部的，通常是把该对象放到栈上。进程的栈空间极其宝贵，这样做的后果当然是栈溢出。最省事的 workaround 是把它改变为“全局”对象。

但是，请注意，我坚持认为你这样继续试图分配很大的数组是不正确的，你的程序突然需要大量的空间就一定存在着设计上的不足，也许代码应该重写。

slowhand

谢谢，各位的解答