Tarantool 内存泄漏问题分析：coio_connect_timeout 中的 getaddrinfo 内存管理

在 Tarantool 数据库系统中，网络连接处理是其核心功能之一。最近在代码审查过程中，我们发现了一个潜在的内存泄漏问题，该问题出现在异步网络连接处理的底层函数中，特别是在处理 DNS 解析时未能正确释放内存资源。

问题背景

当 Tarantool 通过 net.box 模块尝试建立网络连接时，底层会调用 coio_connect_timeout 函数。这个函数负责在指定超时时间内完成连接建立。在连接过程中，系统需要进行 DNS 解析来获取目标地址信息，这时会调用 getaddrinfo 函数。

问题现象

通过 AddressSanitizer (ASAN) 内存检测工具，我们可以清晰地观察到内存泄漏现象。当执行简单的连接操作时，如尝试连接到一个不存在的端口，系统会泄漏 76 字节的内存。这个泄漏发生在 getaddrinfo 函数调用过程中分配的内存没有被正确释放。

技术分析

1. 内存泄漏路径：

   • 首先通过 malloc 分配 76 字节内存
   • 在 getaddrinfo 函数内部生成地址信息
   • 任务完成后，这些内存没有被释放

2. 异步处理机制： Tarantool 使用 libeio 库进行异步 I/O 操作。在 coio_task.c 中，getaddrinfo 被封装为异步任务执行。当任务完成时，虽然连接失败，但分配的资源没有被清理。

3. 根本原因： 问题出在错误处理路径上。当连接失败时，代码没有调用相应的资源释放函数来清理 getaddrinfo 分配的内存结构。

解决方案

正确的做法是在异步任务完成后，无论成功与否，都应该释放 getaddrinfo 分配的内存。这可以通过调用 freeaddrinfo 函数来实现。修复方案需要确保：

1. 在 getaddrinfo_cb 回调函数中正确处理两种结果
2. 在任务完成路径上统一释放资源
3. 保持错误处理的一致性

影响范围

该问题从 Tarantool 2.10.0-beta1 版本引入，影响了后续的 2.11 和 3.2 版本。虽然每次泄漏的内存量不大，但在高频率连接失败场景下可能累积成显著的内存消耗。

最佳实践建议

对于网络编程中的资源管理，建议：

1. 始终为异步操作配对资源分配和释放
2. 使用自动化工具如 ASAN 进行内存泄漏检测
3. 在错误处理路径上保持与正常路径相同的资源清理逻辑
4. 对于第三方库分配的资源，要仔细查阅文档了解其生命周期管理要求

这个问题提醒我们，在网络编程中，即使是看似简单的连接操作，也需要仔细管理所有可能分配的资源，特别是在异步处理场景下。