如何解决C语言多线程求平均值程序的Segmentation Fault问题？

作业要求 #

编写一个多线程程序，用于计算一组数值的统计值。该程序需从命令行接收一系列数值，创建工作线程来计算这些数值的平均值。

问题描述 #

我在这个项目上花费了大量时间，但还没来得及验证代码是否正常运行就出现了段错误，仅收到提示信息：zsh: segmentation fault。这是我两年多来的第一个项目，也是第一个C语言项目，完全不知道该从何处开始排查。

相关代码 #

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

/* Variables that hold the final results, integer data type for simplicity */
int avgNum;
int numNum;
int i;
int sum = 0;

/* Functions' declaration. Definition is at the bottom*/
void *avgfunc(void *param);

int main(int argc, char* argv[])
{
    numNum = argc;
    
    /*Creating the thread ids*/
    pthread_t tid0;

    /*Creating the threads*/
    pthread_create(&tid0, NULL, avgfunc, (int *)&avgNum);

    /*Enabling the threads to operate simultaneously*/
    pthread_join(tid0, NULL);

    /*Exiting the threads*/
    pthread_exit(NULL);

    /*Outputing the final average, maximum and minimum results*/
    printf("Average: %d", avgNum);
}

void *avgfunc(void *param)
{
    char** args = (char**) param;   
    args++;                        

    if(numNum > 1)
    {
        for(i = 1; i < numNum; i++)
        {
            sum += atoi(args[i]);
        }
        avgNum = sum / numNum;
    }
}

核心错误分析 #

1. 线程参数类型不匹配
   创建线程时传递的是avgNum的地址（(int *)&avgNum），但在avgfunc里将其强制转换为char**（命令行参数数组类型）。这种错误转换导致后续访问args[i]时，实际操作的是无效内存区域，直接触发段错误。

2. 主线程执行顺序错误
   pthread_exit(NULL)会直接终止主线程，导致后续的printf语句永远无法执行，属于逻辑错误。

3. 平均值计算逻辑错误
   numNum等于argc，而argc包含了程序本身的名称，实际输入的数值个数是argc-1，用numNum作为除数会导致计算结果偏差。

4. 线程函数缺少返回值
   线程函数要求返回void*类型，但avgfunc没有return语句，会引发未定义行为。

5. 全局变量滥用
   i、sum等全局变量虽当前不影响单线程，但会降低代码扩展性，后续增加线程时会出现竞争问题。

修复后的代码 #

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

// 封装线程所需参数，避免类型错误和全局变量
typedef struct {
    char** argv;
    int argc;
    int* avg_result;
} ThreadData;

void *avgfunc(void *param);

int main(int argc, char* argv[])
{
    int avgNum;
    ThreadData thread_data = {
        .argv = argv,
        .argc = argc,
        .avg_result = &avgNum
    };
    
    pthread_t tid;
    // 传递封装好的参数结构体地址
    pthread_create(&tid, NULL, avgfunc, &thread_data);

    // 等待工作线程完成计算
    pthread_join(tid, NULL);

    // 输出结果后正常退出主线程
    printf("Average: %d\n", avgNum);
    return 0;
}

void *avgfunc(void *param)
{
    ThreadData* data = (ThreadData*)param;
    int sum = 0;

    if (data->argc > 1) {
        // 从索引1开始遍历，跳过程序名
        for (int i = 1; i < data->argc; i++) {
            sum += atoi(data->argv[i]);
        }
        // 用实际数值个数（argc-1）计算平均值
        *data->avg_result = sum / (data->argc - 1);
    } else {
        *data->avg_result = 0;
        printf("No input numbers provided\n");
    }

    // 线程函数必须返回void*类型值
    return NULL;
}

关键修复说明 #

• 参数安全传递：用ThreadData结构体封装所有线程需要的参数，彻底避免类型不匹配的强制转换问题。
• 修正计算逻辑：将除数改为argc-1，确保平均值计算基于实际输入的数值个数。
• 移除全局变量：把sum、i改为线程内局部变量，避免潜在的线程安全隐患。
• 调整主线程流程：移除提前终止主线程的pthread_exit，确保结果能正常输出。
• 补充返回值：线程函数末尾添加return NULL，符合POSIX线程函数的要求。

新手段错误排查技巧 #

• 使用GDB调试：编译时添加-g参数（如gcc -g -pthread your_code.c -o avg），运行gdb ./avg后输入run 1 2 3（带测试参数），崩溃后输入bt查看调用栈，快速定位出错代码行。
• 检查指针转换：强制类型转换时必须确认原指针与目标类型匹配，不匹配的转换几乎必然导致内存访问错误。
• 逐步简化代码：先实现单线程版本的平均值计算，确认逻辑正确后再加入多线程代码，降低问题定位难度。

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