C 在循环中返回无效地址

Question

已关闭。此问题需要 debugging details 。目前不接受答案。

---

编辑问题以包含 desired behavior, a specific problem or error, and the shortest code necessary to reproduce the problem 。这将有助于其他人回答问题。

已关闭 6 年前。

Improve this question

我正在使用 C 程序循环遍历我的文件结构。当我点击一个新文件夹时，我将其路径附加到链接列表中，以便我可以迭代地查看所有子目录。

该程序包括:

主函数，调用迭代函数(循环遍历文件)

当我循环遍历所有文件时，一切正常。然而，当我在主函数中有一个 while 循环来更频繁地调用迭代函数时，由于段错误，它总是在第二次失败。

所以我进行了一些调查，似乎链接列表的一个元素有一个无效的地址。

我的元素的所有地址都具有以下格式和长度:0x2398ff0 或 0x2398ee0

但是非法指针的地址来自0x7f3770304c58

有人知道为什么这个地址这么长吗？

我已经通过 printf("%p", element) 检查了添加到链接列表中的每个新地址，并且该地址以前从未出现在代码中的任何位置。它就像神奇地出现了。

我可能正在考虑一个野指针，但在释放任何指针后，我将其设置为 NULL，这应该可以防止这种情况发生，对吗？

感谢您的任何提示。我现在还没有发布代码，因为它很长，我想也许有一些明显的事情我没有看到。

编辑:整个代码，包括主函数

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

void iterativefile(FILE *f, char** field, int looper){

    DIR *d;
    struct dirent *dir;

    typedef struct nextpath { // Define element type of linked list
        char *thispath;
        struct nextpath *next;
    }nextpath;

    nextpath *startpath = malloc(sizeof(nextpath));
    char * beginning = (char *) malloc(2); //create first element in linked list, starting on root node "."
    strcpy(beginning, ".");
    startpath->thispath = beginning;

    int found = 0;

    nextpath *currentzeiger = startpath;
    nextpath *firstelement = startpath;
    char *newdir, *currentfile, *currentpath;

    do {
        currentpath = currentzeiger->thispath;
        d = opendir(currentpath);
        if (!d){ //IF the path is invalid or cannot be opened

            firstelement = currentzeiger->next;
            free(currentzeiger);
            currentzeiger = firstelement;
            continue;
        }
        while((dir = readdir(d)) != NULL){
            if (dir->d_type != DT_REG){ // current element is a directory -> add it to linked list
                if (strcmp(dir->d_name, ".") != 0 && strcmp(dir->d_name, "..") != 0){
                    newdir = (char *) malloc(2+strlen(currentpath) + strlen(dir->d_name));      
                    strcpy(newdir, currentpath);
                    strcat(newdir, "/");
                    strcat(newdir, dir->d_name);
                    nextpath *new = malloc(sizeof(nextpath)); // add new folder to linked list
                    new->thispath = NULL;
                    new->thispath = strdup(newdir);
                    new->next = currentzeiger->next;
                    currentzeiger->next = new;
                    free(newdir);
                    newdir = NULL;              
                }
            }
            else { // current element is a file -> check if already included in list, if not, add it
                currentfile = (char *) malloc(2+strlen(currentpath)+strlen(dir->d_name));
                strcpy(currentfile, currentpath);
                strcat(currentfile, "/");
                strcat(currentfile, dir->d_name);
                found = 0;
                if (field != NULL) {
                    for (int z = 0; z < looper; z++){
                        if (field[z] != NULL){
                            if(strcmp(currentfile,field[z]) == 0){
                                found = 1;
                                free(field[z]);
                                field[z] = NULL;
                            }
                        }
                    }
                }
                if (found == 0){
                    char *renamefile = (char *) malloc(strlen(currentpath) + 6);
                    strcpy(renamefile, currentpath);
                    strcat(renamefile, ".cbsm");
                    free(renamefile);
                    renamefile = NULL;

                }
                free(currentfile);
                currentfile = NULL;
            }
        }

        firstelement = currentzeiger->next;
        free(currentzeiger->thispath);
        currentzeiger->thispath = NULL;
        free(currentzeiger);

        currentzeiger = firstelement;
        closedir(d);

    }while(currentzeiger != NULL);
}



int main()
{
    int counterofwhile = 1;
    while(1){
    printf("Loop number: %d\n", counterofwhile);
    counterofwhile++;
    FILE *fp = fopen("datasyn.txt", "rw+");
    if (fp == NULL) {
        printf("FILE ERROR");
        FILE *fp = fopen("datasyn.txt", "ab+");
        iterativefile(fp, NULL, 0);
    }
    else {
        int lines = 0;
        int ch = 0;
        int len = 0;
        int max_len = 0;
        while((ch = fgetc(fp)) != EOF){
            ++len;
            if (ch == '\n'){
                if(max_len < len)
                    max_len = len;
                ++lines;
                len = 0;
            }
        }

        if (len)
            ++lines;
        fprintf(stderr, "%d lines\n", lines);
        if (lines > 0){
            int numProgs = 0;
            char *programs[lines];
            char line[max_len + 1];
            rewind(fp);
            while(fgets(line, sizeof(line), fp)){
                int new_line = strlen(line) - 1;
                if (line[new_line] == '\n')
                    line[new_line] = '\0';
                programs[numProgs++] = strdup(line);
            }

            iterativefile(fp, programs, numProgs);
            for (int j = 0; j < numProgs; j++){
            free(programs[j]);
            }
        }
        else {
            iterativefile(fp, NULL, 0);
        }
    sleep(1);


    printf("Done\n");
    fclose(fp);
    }

    }
    return 0;
}

Answer 1

在函数 iterativefile() 中，您没有使用 calloc() 来分配 startpath 并且没有设置 startpath->next 为 null。 malloc() 返回的内存不一定归零。当您随后使用 startpath->next 时，一切都会崩溃。

您也不使用传递给 iterativefile() 的文件指针。当您从定义中删除参数时，您更改了调用，并且您在 main() 中得到了一个隐藏的 fp (在 if (fp = = NULL) block ，您创建了一个实际上不需要的新 FILE *fp )。确实不清楚还会发生什么；您没有就该程序的用途给出明确的说明。我没有 datasyn.txt 文件，但这应该不重要，因为未使用文件流。我从代码中得到了很多类似 FILE ERRORLoop number: 280 的行，但没有崩溃，而之前我遇到了崩溃。

使用更多警告选项进行编译。我调用了文件 fp17.c 并使用以下方法编译了我的破解版本:

gcc -O3 -g -std=c11 -Wall -Wextra -Werror -Wmissing-prototypes -Wstrict-prototypes \
    -Wold-style-definition fp17.c -o fp17

通过一些其他简单的更改(函数之前的static；int main(void))，代码可以干净地编译(并且需要-Wshadow 来发现阴影，如果不是因为“未使用的变量”警告向我指出了它)。