访问内存 malloc 时出现段错误

Question

我正在尝试编写一个函数，该函数接受路径 (char *) 并将其拆分为基于“/”分隔符的字符串数组。简化代码如下：

int split_path(char * path, char ** out) {
    out = NULL;
    char * token = strtok(path, "/");
    int count = 0;

    while(token) {
        out = realloc(out, sizeof(char*) * (++count));
        out[count-1] = malloc(sizeof(char) * strlen(token)+1);
        strcpy(out[count-1], token);
        fprintf(stderr, "%s\n", out[count-1]);

        token = strtok(NULL, "/");
    }   

    out = realloc(out, sizeof(char*) * (count+1));
    out[count] = NULL;

    return count;
}

int main(int argc, char * argv[]) {
    char path[] = "/home/pirates/are/cool/yeah";

    char ** out;

    int count = split_path(path, out);

    fprintf(stdout, "count: %d\n", count);
    fprintf(stderr, "1st: %s\n", out[0]); // segfaults here
    return 0;
}

split_path 函数中的所有打印语句都打印完美，输出如下所示：

count: 1, string: home
count: 2, string: pirates
count: 3, string: are
count: 4, string: cool
count: 5, string: yeah
count: 5
1st: ./a.out
[1]    5676 segmentation fault (core dumped)  ./a.out

但由于某种原因，当我回到主函数时，双字符数组不再有效。我认为这可能是因为它指向在该 split_path 函数中声明的内存，但我正在执行 strcpy 以将字符串放入其中，因此它不应该指向该函数本地的内存。非常感谢任何帮助。

Answer 1

out 参数管理不善。 main() 中的 out 变量从未分配过有效的内存地址，因此是段错误。 split_path() 中的 out 参数永远不会更新 main() 中的 out 变量。您需要将变量的地址传递给split_path()，以便它可以更新变量，并访问变量指向的内存。

还要注意strtok() 修改了它正在解析的字符串，因此您应该制作一个副本然后解析该副本，以免原始文件被破坏。否则，请考虑使用strchr() 而不是strtok()。

尝试类似的方法：

int split_path(char * path, char *** out) {
    *out = NULL;
    char * tmp = strdup(path);
    if (!tmp) { ... }
    char * token = strtok(tmp, "/"');
    int count = 0;
    char ** newout;

    while (token) {
        newout = realloc(*out, sizeof(char**) * (++count));
        if (!newout) { ... }
        *out = newout;
        (*out)[count-1] = malloc(sizeof(char) * (strlen(token)+1));
        if (!(*out)[count-1]) { ... }
        strcpy((*out)[count-1], token);
        fprintf(stderr, "%s\n", token);

        token = strtok(NULL, "/");
    }   

    newout = realloc(*out, sizeof(char**) * (count+1));
    if (!newout) { ... }
    *out = newout;
    (*out)[count] = NULL;

    free (tmp);
    return count;
}

int main(int argc, char * argv[]) {
    char path[] = "/home/pirates/are/cool/yeah";

    char ** out;

    int count = split_path(path, &out);

    fprintf(stdout, "count: %d\n", count);
    fprintf(stderr, "1st: %s\n", out[0]); // segfaults here

    free (out);
    return 0;
}

不要忘记错误处理。为简洁起见，我已将其从示例中省略，但您不应将其从实际代码中删除。