C语言在SIGINT信号触发时释放malloc分配内存的方法是否合理？

你当前的实现和疑问可以从以下几个角度梳理：

• 首先是手动释放内存的必要性存疑：所有现代操作系统都会在进程正常/异常终止时，完整回收该进程占用的所有内存资源，包括你调用malloc()分配的堆内存。所以如果你写这个程序只是为了测试内存分配上限，终止后就直接退出，完全不需要手动释放内存，Ctrl+C终止后不会有任何内存泄漏。
• 其次是当前信号处理逻辑存在严重缺陷：
  1. 你只保留了最后一次malloc()返回的指针，之前分配的所有内存地址都被循环覆盖了，信号触发时调用free(ptr)只能释放最后一块内存，前面的所有内存根本没被释放，这个清理逻辑是无效的。
  2. free()和exit()都不是异步信号安全的函数：如果SIGINT信号刚好触发在程序执行malloc()的内部逻辑时，信号处理函数里再次调用free()会触发未定义行为，轻则崩溃重则出现内存损坏。
• 关于你注释里的疑问：信号处理函数的int参数是触发本次回调的信号编号，如果你把同一个处理函数绑定给多个信号（比如同时绑定SIGINT和SIGTERM），就可以通过这个参数区分触发源，你当前场景用不到所以可以忽略，但函数签名必须符合signal()的要求不能省略。

最优实现方案 #

你这个场景最推荐的做法是直接删掉信号处理相关的所有代码，简化后代码如下：

#include <stdlib.h>

int main(void) {
   int *ptr;
   while ((ptr = malloc(sizeof *ptr)) != NULL) {
         *ptr = 0;
   }
}

无论是malloc()失败自然退出，还是你按Ctrl+C手动终止，操作系统都会帮你回收所有内存，不需要你做任何额外处理，代码更简洁也不会有安全问题。

如果你确实需要手动处理信号的场景 #

如果后续你的程序需要扩展（比如分配失败或者收到终止信号时要输出统计日志、持久化数据等），可以用「信号处理函数仅做标记、主循环执行清理」的安全实现：

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

// 异步信号安全的标记变量
volatile sig_atomic_t exit_flag = 0;

void inthandler(int sig) {
    exit_flag = 1;
}

int main(void) {
    // 用来存储所有分配的指针的容器，示例简化用固定数组，实际可按需改用链表
    int *ptrs[1024 * 1024] = {0};
    size_t alloc_cnt = 0;

    signal(SIGINT, inthandler);
    
    while (!exit_flag && alloc_cnt < sizeof(ptrs)/sizeof(ptrs[0])) {
        int *ptr = malloc(sizeof *ptr);
        if (!ptr) break;
        *ptr = 0;
        ptrs[alloc_cnt++] = ptr;
    }

    // 统一清理所有已分配内存
    printf("共分配%zu块内存，开始清理\n", alloc_cnt);
    for (size_t i = 0; i < alloc_cnt; i++) {
        free(ptrs[i]);
    }

    return exit_flag ? -1 : 0;
}

这种实现完全规避了异步信号安全问题，也能正确释放所有已分配的内存。

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