-2

我有一个代码可以扫描整个目录以查找 .wor 文件,并从中提取信息。当我在包含 5 个 .wor 文件的目录上运行该文件时,它运行良好。但是当我尝试在包含 70 多个 .wor 文件的目录上运行它时,它给了我一个“内存故障(核心转储”错误)。

所以我尝试用 gdb 调试代码,这就是我得到的:

Program received signal SIGSEGV, Segmentation fault.
0x7af9fcbc in _sbrk () from /usr/lib/libc.2
(gdb) bt
#0  0x7af9fcbc in _sbrk () from /usr/lib/libc.2
#1  0x7af9db7c in _sigfillset () from /usr/lib/libc.2
#2  0x7af9bfe8 in __thread_callback_np () from /usr/lib/libc.2
#3  0x7afa0e64 in malloc () from /usr/lib/libc.2
#4  0x7b001160 in _findbuf () from /usr/lib/libc.2
#5  0x7afff7f4 in _filbuf () from /usr/lib/libc.2
#6  0x7afff404 in __fgets_unlocked () from /usr/lib/libc.2
#7  0x7afff5fc in fgets () from /usr/lib/libc.2
#8  0x2c14 in main (argc=1, argv=0x7dff11b4) at book2.c:99

我查看了第 99 行,我有命令:while( fgets(buff, 100,fs) != NULL ),我不太了解 c 以找到错误所在,非常感谢这里的任何帮助,谢谢进步!

这是我的代码:

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <sys/types.h>
#include <dirent.h>
#include <unistd.h>
#include <errno.h>


int main(int argc, char **argv) {

char directory[100];
char buff[100];
char delims[] = " :=";
char *result = NULL;
char *customer;
char *device;
char *testprog;
char *software;
char *dutboardid;
char *corlbox;
char *testerid;
char *temp;
char *buffer;
int i=0;
DIR * FD;
struct dirent* in_file;
int c=0;


printf ("Enter directory:");                                   
    scanf ("%s",directory);


    FILE * ft = fopen ( "workorderlist.csv", "w" ) ;    /* Open file to write to*/
    if ( ft == NULL )
    {
           puts ( "Cannot open target file" ) ;
           exit( 1 ) ;
    }


fprintf (ft, "Work Order,Customer,Device,Test Program,Software,DUT_board_id,Corl box, Tempurature, Test Station\n");



/* Open Directory*/

if (NULL == (FD = opendir (directory))) 
    {

        puts ("Cannot open directory");

    return 1;
    }


    while ((in_file = readdir(FD))) 
    {

    if (!strcmp (in_file->d_name, "."))
        continue;
    if (!strcmp (in_file->d_name, ".."))    
        continue;



    /* Open files to read from  */

buffer = (char*) malloc (200);
sprintf(buffer, "%s/%s", directory, in_file->d_name);


size_t len = strlen(buffer);
if (len >= 4 && memcmp(buffer + len - 4, ".wor", 4) == 0)   /* checks if file ends with .wor */
{

FILE * fs = fopen(buffer, "r");       /* open file to read */

customer = "0";
device = "0";
testprog = "0";
software = "0";
dutboardid = "0";
corlbox = "0";
testerid = "0";
temp = "0";

    if (fs == NULL)
    {
        puts ("Cannot open source file");

        return 1;
    }

    /* Scanning each file for targeted words: */


while( fgets(buff, 100,fs) != NULL )      
{   



 result = strtok( buff, delims );          
    while(result != NULL){   


            if((strcmp(result,"Customer")==0)){ 
        result = strtok(NULL,delims);  
        customer = (char*)malloc((strlen(result)+1)*sizeof(char));
        strcpy(customer, result);
        for(i=0;i<strlen(customer)+1;i++){ if(customer[i] == '\n') break; }
        customer[i] = ' ';
        }

            if((strcmp(result,"name")==0)){ 
        result = strtok(NULL,delims);  
        customer = (char*)malloc((strlen(result)+1)*sizeof(char));
        strcpy(customer, result);
        for(i=0;i<strlen(customer)+1;i++){ if(customer[i] == '\n') break; }
        customer[i] = ' ';
        }


            if(strcmp(result,"device")==0){ 
        result = strtok(NULL,delims);  
        device = (char*)malloc((strlen(result)+1)*sizeof(char));
        strcpy(device, result);
        for(i=0;i<strlen(device)+1;i++){ if(device[i] == '\n') break; } 
        device[i] = ' ';
        }


            if(strcmp(result,"test_prog")==0 || strcmp(result, "testprog")==0){ 
        result = strtok(NULL,delims);  
        testprog = (char*)malloc((strlen(result)+1)*sizeof(char));
        strcpy(testprog, result);
        for(i=0;i<strlen(testprog)+1;i++){ if(testprog[i] == '\n') break; } 
        testprog[i] = ' ';
        }


            if(strcmp(result,"Rev")==0 || strcmp(result,"use")==0){ 
        result = strtok(NULL,delims);  
        software = (char*)malloc((strlen(result)+1)*sizeof(char));
        strcpy(software, result);
        for(i=0;i<strlen(software)+1;i++){ if(software[i] == '\n') break; } 
        software[i] = ' ';
        }

        if(strcmp(result,"rev")==0){ 
        result = strtok(NULL,delims);  
        software = (char*)malloc((strlen(result)+1)*sizeof(char));
        strcpy(software, result);
        for(i=0;i<strlen(software)+1;i++){ if(software[i] == '\n') break; } 
        software[i] = ' ';
        }

            if(strcmp(result,"DUT_board_id")==0){ 
        result = strtok(NULL,delims);  
        dutboardid = (char*)malloc((strlen(result)+1)*sizeof(char));
        strcpy(dutboardid, result);
        for(i=0;i<strlen(dutboardid)+1;i++){ if(dutboardid[i] == '\n') break; } 
        dutboardid[i] = ' ';
        }                   

            if(strcmp(result,"box")==0){ 
        result = strtok(NULL,delims);  
        corlbox = (char*)malloc((strlen(result)+1)*sizeof(char));
        strcpy(corlbox, result);
        for(i=0;i<strlen(corlbox)+1;i++){ if(corlbox[i] == '\n') break; } 
        corlbox[i] = ' ';
        }

        if(strcmp(result,"Tester_id")==0){ 
        result = strtok(NULL,delims);  
        testerid = (char*)malloc((strlen(result)+1)*sizeof(char));
        strcpy(testerid, result);
        for(i=0;i<strlen(testerid)+1;i++){ if(testerid[i] == '\n') break; } 
        testerid[i] = ' ';
        }

        if(strcmp(result,"@")==0){ 
        result = strtok(NULL,delims);  
        temp = (char*)malloc((strlen(result)+1)*sizeof(char));
        strcpy(temp, result);
        for(i=0;i<strlen(temp)+1;i++){ if(temp[i] == '\n') break; } 
        temp[i] = ' ';
        }


        result = strtok(NULL,delims);
    }

}

if (strcmp(customer, "0")==0){
customer = "N/A";
}
if (strcmp(device, "0")==0){
device = "N/A";
}
if (strcmp(testprog, "0")==0){
testprog = "N/A";
}
if (strcmp(software,"0")==0){
software = "N/A";
}
if (strcmp(dutboardid, "0")==0){
dutboardid = "N/A";
}
if (strcmp(corlbox, "0")==0){
corlbox = "N/A";
}
if (strcmp(testerid, "0")==0){
testerid = "N/A";
}
if (strcmp(temp, "0")==0){
temp = "N/A";
}

fprintf (ft, "%s,%s,%s,%s,%s,%s,%s,%s,%s\n", in_file->d_name, customer, device, testprog, software, dutboardid, corlbox, temp, testerid);


printf(in_file->d_name);
printf("\n");
fclose (fs) ;
c++;

}
}
printf("Total Workorders Found: %d (Info saved to workorderlist.csv)\n", c);
    fclose ( ft ) ;
return 0;

}

4

2 回答 2

1

内存不足。

例如,您调用 malloc 并将其分配给循环内的“客户”,并且似乎永远不会释放内存。

您有内存泄漏,最终您的内存使用量达到了系统限制。

于 2013-01-16T22:53:17.627 回答
1

如果我们举这个例子:

customer = (char*)malloc((strlen(result)+1)*sizeof(char));
strcpy(customer, result);
for(i=0;i<strlen(customer)+1;i++){ if(customer[i] == '\n') break; }
customer[i] = ' ';

它复制结果,占用strlen(result)+1字节。然后您将结果复制到客户中。之后,您在字符串中搜索 '\n'。如果字符串中没有 '\n' 会发生什么?什么是价值i

我很确定问题在于,在某些时候,这种代码风格是在分配的内存之外编写的——因为我不知道你的输入到底是什么,我不能说几个副本中的哪一个这段代码出错了,但我敢打赌这就是发生的事情。这反过来意味着堆结构本身中的一些数据被破坏了,所以在调用 malloc 的某个时候,它在代码中使用了这些坏数据。

于 2013-01-16T23:11:26.837 回答