malloc引发Segment fault (core dumped)，C语言新手求助代码问题定位

核心问题1：全局作用域非法执行语句 #

C语言明确规定，函数外部的全局作用域只能做变量声明或初始化，不能直接编写malloc赋值、for循环这类执行逻辑代码。你的代码把内存分配、循环检查等操作直接放在全局区域，这本身属于语法错误，即便侥幸编译通过，也会引发运行时未定义行为。

核心问题2：循环边界计算错误 #

sizeof(tasks)计算的是整个指针数组的总字节数，而非数组元素个数。在64位系统中，每个指针占8字节，6个指针组成的数组总字节数为6*8=48，你的循环条件i < sizeof(tasks)会让循环执行48次，远远超出数组仅有的6个元素（索引范围0-5），访问tasks[6]及以后的位置属于数组越界，直接触发段错误。

修复后的完整代码 #

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

int main() {
    char *tasks[6];
    const int task_count = sizeof(tasks) / sizeof(tasks[0]); // 动态计算数组元素个数

    // 内存分配与错误检查
    for (int i = 0; i < task_count; i++) {
        tasks[i] = malloc(5 * sizeof(char)); // C语言无需强制转换malloc返回值
        if (!tasks[i]) {
            perror("malloc failed");
            // 分配失败时清理已分配内存，避免泄漏
            for (int j = 0; j < i; j++) {
                free(tasks[j]);
            }
            return 1;
        }
    }

    // 复制字符串
    strncpy(tasks[0], "Dish", 5);
    strncpy(tasks[1], "Laun", 5);
    strncpy(tasks[2], "Bath", 5);
    strncpy(tasks[3], "Flor", 5);
    strncpy(tasks[4], "Tras", 5);
    strncpy(tasks[5], "Offi", 5);

    // 打印任务
    for (int i = 0; i < task_count; i++) {
        printf("%s\n", tasks[i]);
    }

    // 释放内存
    for (int i = 0; i < task_count; i++) {
        free(tasks[i]);
    }

    return 0;
}

额外优化说明 #

• 用sizeof(tasks)/sizeof(tasks[0])动态计算元素个数，避免硬编码数字，提升代码可维护性。
• 移除malloc返回值的强制转换，C语言中void*可自动转为其他指针类型，强制转换反而可能隐藏类型不匹配问题。
• 新增内存分配失败时的清理逻辑，防止已分配内存泄漏。

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