memset 抛出分段错误

Question

我在编写一个从文件中提取字符串作为更大程序的一部分的函数时遇到了一些问题。一切似乎都运行良好，除非我使用 memset 或 bzero 擦除我一直在使用的字符数组。我已经解决这个问题一个多小时了，无论我做什么，我都会遇到段错误。我在 bzero 和 memset 上都收到了这个错误。请帮帮我。 我在下面附上我的代码。打印了“Come out of addfront”语句，但没有打印“Done with all bzero”语句。那时我遇到了分段错误。谢谢

void extractFileData(FILE *fp , char clientName[])
{
    char tempFileName[50], tempFilePath[100], tempFileSize[50];
    struct stat fileDetails;

    while(fgets(tempFileName, sizeof(tempFileName), fp)!= NULL)
    {
        if((newLinePos = strchr(tempFileName, '\n')) != NULL)
        {
            *newLinePos = '\0';
        }

        strcat(tempFilePath, "SharedFiles/");
        strcat(tempFilePath, tempFileName);

        if(stat(tempFilePath, &fileDetails) < 0)
        {
            perror("Stat error");
            exit(1);
        }

        //Copy it into a string
        sprintf(tempFileSize, "%zu", fileDetails.st_size);
        printf("temp file size: %s\n", tempFileSize);

        //Add all these details to the file list by creating a new node
        addFront(tempFileName, tempFileSize, clientName);

        printf("Come out of addfront\n");

        memset(&tempFileName, 0, 45);
        printf("Done with all bzero\n");
        memset(&tempFileSize, 0, sizeof(tempFileSize));
        memset(&tempFilePath, 0, sizeof(tempFilePath));

        printf("Done with all bzero\n");
    }
}   

编辑：

void addFront(char fileName[], char fileSize[], char clientName[])
{
    FILENODE* n;
    printf("Inside add front function\n");
    strcpy(n->fileName, fileName);
    printf("n->filename: %s\n", n->fileName);
    strcpy(n->fileSize, fileSize);
    printf("n->filesize: %s\n", n->fileSize);
    strcpy(n->ownerName, clientName);
    printf("n->ownername: %s\n", n->ownerName);
    myFileList.head = n;
    printf("Did it go past myfilelist head = n\n");
    myFileList.numOfNodes++;
    printf("num of nodes: %d\n", myFileList.numOfNodes);
}

我已经为 addFront 函数添加了我的代码。它基本上增加了 struct myFileList 的详细信息，它基本上是一个实现 的一个链表。 FILENODE 代表列表中的每个条目。

编辑：

添加我正在使用的结构

 struct fileNode
 {
      char fileName[50];
      char fileSize[50];
          char ownerName[25];
      struct fileNode* next;
 };

 struct fileList
 {
      struct fileNode* head;
      struct fileNode* tail;
       int numOfNodes;
 };

 typedef struct fileList FILELIST;
 typedef struct fileNode FILENODE;

Answer 1

我不知道你的程序为什么会在那里崩溃。但我可以在程序中出现另一个错误。先修复另一个错误，看看是否还有问题。

这是错误：

strcat(tempFilePath, "SharedFiles/");
strcat(tempFilePath, tempFileName);

tempFilePath 变量未初始化。这可能巧合不会崩溃，但你不能指望它不会崩溃。它可能会在您的堆栈上乱涂乱画。

改为这样做：

snprintf(tempFilePath, sizeof(tempFilePath), "SharedFiles/%s", tempFileName);

最后，无需将数组归零。数组的内容不会在下一次循环迭代中使用，所以你不妨忽略它们。

void extractFileData(FILE *fp , char clientName[])
{
    char tempFileName[50], tempFilePath[100], *newLinePos;
    struct stat fileDetails;
    while (fgets(tempFileName, sizeof(tempFileName), fp)) {
        if ((newLinePos = strchr(tempFileName, '\n')))
            *newLinePos = '\0';
        snprintf(tempFilePath, sizeof(tempFilePath),
                 "SharedFiles/%s", tempFileName);
        if (stat(tempFilePath, &fileDetails) < 0) {
            perror("Stat error");
            exit(1);
        }
        printf("temp file size: %zu\n", tempFileSize);
        addFront(tempFileName, tempFileSize, clientName);
    }
}   

snprintf() 函数确实是在 C 中进行此类工作的第一选择。使用 snprintf() 编写“显然不会崩溃”的代码很容易，而不是“显然不会崩溃”的代码崩溃”。

如果您的代码仍然崩溃，则说明其他地方存在错误。

Answer 2

addFront() 需要 n = malloc( sizeof *n) 才能对它进行任何操作。