C语言实现通用文件读取函数时动态内存仅分配8字节且触发段错误求助

• 未初始化局部变量导致的逻辑完全失控：代码中int c定义后没有赋初值，属于栈内存上的随机值，第一次进入while(c != EOF)判断时结果完全不可预测，会直接打乱文件大小的计算流程。
• 文件大小计算逻辑完全错误：逐字符读文件算大小的循环里，每次读到EOF时也会执行一次size++，会把EOF对应的字节数错误计入文件大小；加上传入函数的size参数本身没有被初始化为0，最后算出的size值极小（常见情况就是刚好为8），这就是你看到malloc只分配了8字节内存的根本原因——你传给malloc的分配长度就是8，并非malloc默认给小值。
• 非法内存访问直接触发段错误：
  • 文件打开失败时file为NULL，代码在if判断分支外写了rewind(file)，对空指针调用文件操作函数，直接触发段错误
  • 函数定义为返回char*类型，实际不管流程走哪个分支最后都return 0（即空指针），调用方拿到的返回值是空指针，后续对指针做读写操作必然段错误
  • 因为malloc分配的内存块远小于实际文件大小，读文件时会越界写入不属于自己的内存区域，也是段错误的核心诱因
• 字符串格式不兼容C标准：就算前面流程全跑通，读入的内容最后也没有手动补'\0'结束符，后续用字符串函数处理这个内容时会直接读越界。
• 冗余逻辑效率极低：手动逐字符读文件算大小的方式，相比直接用文件偏移量获取大小的方式效率极低，还容易出逻辑bug。

去掉了冗余的传入size参数，直接在函数内部完成文件大小计算、内存分配、内容读取全流程：

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

char* readFile(const char *fileName) {
    FILE *file = fopen(fileName, "r");
    // 打开文件失败直接返回空指针
    if (file == NULL) {
        printf("\nError: Unable to open the file for Reading.\n");
        return NULL;
    }

    // 直接通过文件偏移计算准确大小，无需逐字符遍历
    fseek(file, 0, SEEK_END);
    long fileSize = ftell(file);
    rewind(file); // 回到文件起始位置

    // 分配内存：文件实际长度 + 1个字节存储字符串结束符
    char *content = (char*)malloc((fileSize + 1) * sizeof(char));
    if (content == NULL) {
        fclose(file);
        return NULL;
    }

    // 一次性读取全部文件内容
    size_t readLength = fread(content, sizeof(char), fileSize, file);
    content[readLength] = '\0'; // 手动补字符串结束符

    fclose(file);
    printf("Done Reading\n");
    return content;
}

// 调用示例
// int main() {
//     char *fileContent = readFile("目标文件路径");
//     if (fileContent != NULL) {
//         // 对读取到的内容做后续操作
//         free(fileContent); // 用完必须释放动态分配的内存，避免泄漏
//     }
//     return 0;
// }

• 所有栈上定义的局部变量必须显式初始化，避免未定义行为
• 所有文件操作（rewind、fread、fseek等）执行前，必须先确认对应的FILE指针非空
• 动态分配的内存块，调用方用完后必须手动调用free()释放
• C语言标准字符串必须以'\0'作为结束标记，否则所有字符串处理函数都会出现越界访问
• 无特殊需求时，不要把需要计算的内部状态作为传入参数（比如原代码里的size参数），内部自行计算的逻辑可维护性和容错性远高于外部传入

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