调用fclose前如何校验文件句柄是否仍处于有效打开状态

你遇到的free(): invalid pointer报错属于重复释放类错误：传入fclose的fHandler是悬空指针——要么该指针指向的FILE结构体已经被之前的fclose调用释放，要么fHandler被意外篡改、或是fopen失败后未做错误处理直接使用。
仅判断fHandler != NULL无法覆盖场景的原因是：fclose执行时只会释放FILE结构对应的堆内存，不会主动将你持有的fHandler指针置为NULL，此时指针会变成指向已释放内存的野指针，再次传入fclose就会触发内存访问异常。

方案1：配套管理指针状态，关闭后立即置空（推荐） #

该方案跨平台通用，无额外依赖，逻辑简单可靠：

1. 定义fHandler时就初始化为NULL：FILE *fHandler = NULL;
2. 每次执行关闭逻辑时按如下写法实现：

// 仅当指针非空时执行关闭，关闭后立刻置空
if (fHandler != NULL) {
    fclose(fHandler);
    fHandler = NULL;
}

只要严格遵循该写法，后续只要判断fHandler != NULL即可确认句柄处于有效打开状态，不会出现重复关闭的问题。

方案2：POSIX平台专属校验（仅适用于Linux/Unix环境） #

如果程序仅运行在POSIX兼容系统，可以借助系统API校验文件描述符有效性：

#include <unistd.h>
#include <fcntl.h>
#include <errno.h>

// 返回1表示句柄有效，0表示无效
int is_file_handle_valid(FILE *fp) {
    if (fp == NULL) return 0;
    int fd = fileno(fp);
    if (fd == -1) return 0;
    errno = 0;
    return fcntl(fd, F_GETFD) != -1 || errno != EBADF;
}

注意该方案存在边界问题：如果原句柄对应的文件描述符被后续新打开的文件复用，会出现误判，仅适合临时排查问题使用。

还需要重点排查如下编码问题：

• fopen调用后未判断返回值直接使用，如果fopen因为权限、路径等问题返回NULL，后续操作也会触发异常
• 多线程场景下如果存在对fHandler的并发读写、并发关闭操作，未加锁保护会导致指针状态错乱
• 局部变量缓冲区溢出，比如你代码中的buffer数组出现越界写，刚好覆盖了相邻栈上的fHandler变量内存，导致指针被篡改

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