函数执行完最后一个for循环后触发Segmentation Fault，求故障原因

首先，咱们先揪出导致段错误的核心原因，再一步步修正代码：

1. 指针参数的传值陷阱（直接导致问题的根源） #

你写的函数参数int a[]在C语言里会被解析成int *a——这是一个局部指针变量。当你在函数里执行a = (int *) malloc(n * sizeof(int));时，你只是修改了这个局部指针的指向，完全不会影响调用者传入的那个指针变量。

举个例子，如果你的调用代码是这样的：

int main() {
    int *myArr; // 未初始化的野指针
    initArray(myArr, 5);
    printf("%d", myArr[0]); // 这里访问野指针，直接触发段错误！
    return 0;
}

函数initArray里的a指向了malloc分配的堆内存，但main里的myArr还是原来的野指针，后续访问自然会崩溃。哪怕你传入的是栈上的数组，比如int myArr[5]; initArray(myArr,5);，函数里的操作也和这个栈数组完全无关，相当于白做了。

2. 其他需要修正的小问题 #

• rand()未初始化：每次运行程序都会生成完全相同的随机序列，你需要在程序启动时（比如main函数开头）调用srand(time(NULL));来初始化随机数生成器。
• malloc的返回值检查：如果malloc分配内存失败会返回NULL，后续操作NULL指针也会导致段错误，最好加上检查。

修正后的代码方案 #

方案一：让函数返回分配好的数组指针 #

这种方式更直观，调用者直接接收函数返回的堆指针：

#include <stdio.h>
#include <stdlib.h>
#include <time.h>

int* initArray(int n) {
    if (n <= 0) {
        return NULL; // 非法输入直接返回
    }
    int *a = (int *) malloc(n * sizeof(int));
    if (a == NULL) {
        perror("malloc failed");
        return NULL;
    }
    int i, j;
    for(i = 0; i < n; i++){
        a[i] = i+1;
    }
    for(j = n-1; j >= 0; j--){
        int num1 = rand() % (j+1); // 生成0到j的随机数，范围正确
        int container2 = a[j];
        a[j] = a[num1];
        a[num1] = container2;
    }
    // 数组检查用
    int k;
    for(k = 0; k < n; k++){
        printf("%i ", a[k]);
    }
    printf("\n");
    return a; // 返回堆指针，由调用者负责free
}

int main() {
    srand(time(NULL)); // 初始化随机数生成器
    int *arr = initArray(5);
    if (arr != NULL) {
        // 这里可以使用arr
        free(arr); // 记得释放内存
    }
    return 0;
}

方案二：使用指针的指针传递参数 #

如果你坚持要通过参数传递来修改调用者的指针，需要传入指针的指针：

#include <stdio.h>
#include <stdlib.h>
#include <time.h>

void initArray(int **a, int n) {
    if (n <= 0 || a == NULL) {
        return;
    }
    *a = (int *) malloc(n * sizeof(int));
    if (*a == NULL) {
        perror("malloc failed");
        return;
    }
    int i, j;
    for(i = 0; i < n; i++){
        (*a)[i] = i+1;
    }
    for(j = n-1; j >= 0; j--){
        int num1 = rand() % (j+1);
        int container2 = (*a)[j];
        (*a)[j] = (*a)[num1];
        (*a)[num1] = container2;
    }
    // 数组检查用
    int k;
    for(k = 0; k < n; k++){
        printf("%i ", (*a)[k]);
    }
    printf("\n");
}

int main() {
    srand(time(NULL));
    int *arr = NULL;
    initArray(&arr, 5);
    if (arr != NULL) {
        // 使用arr
        free(arr);
    }
    return 0;
}

这样修改后，不仅解决了段错误的问题，也让代码的逻辑更清晰，避免了内存泄漏和无效操作。

内容的提问来源于stack exchange，提问作者Jsandesu