Tarantool项目中Vinyl引擎内存泄漏问题分析

问题背景

在Tarantool数据库项目的测试过程中，发现Vinyl存储引擎存在内存泄漏问题。该问题在运行特定测试用例时被AddressSanitizer工具检测到，表现为间接泄漏4字节内存。

问题现象

测试人员在执行replication/prune.test.lua测试用例时，发现Vinyl实例因SIGSEGV信号终止。进一步分析日志发现，在测试过程中存在内存泄漏：

Indirect leak of 4 byte(s) in 1 object(s) allocated from:
    #0 0x562a6a9a0c89 in calloc
    #1 0x562a6acab2bc in vy_page_new
    #2 0x562a6acb9f6b in vy_slice_stream_read_page
    #3 0x562a6acb7c4b in vy_slice_stream_search
    ...
SUMMARY: AddressSanitizer: 1059 byte(s) leaked in 3 allocation(s).

技术分析

泄漏根源

经过深入分析，发现内存泄漏发生在Vinyl引擎的页面处理流程中。具体来说，当vy_slice_stream_search()函数调用vy_page_find_key()失败时，未能正确释放之前分配的页面内存。

相关代码路径

1. vy_page_new()函数在src/box/vy_run.c中分配新页面内存
2. vy_slice_stream_read_page()函数读取页面数据
3. vy_slice_stream_search()函数执行搜索操作
4. 当搜索失败时，缺少对已分配页面的释放操作

问题复现

开发者提供了简化的复现方法：

yes gh-4864 | head -n50 | xargs ./test-run.py --builddir ../build/debug-asan

这可以稳定重现内存泄漏问题，验证了问题的存在。

影响范围

该内存泄漏问题影响以下Tarantool版本：

• 2.11.x系列
• 3.2.x系列
• 3.3.x系列

解决方案

修复方案相对直接：在vy_slice_stream_search()函数的错误处理路径中添加对vy_page_delete()的调用，确保在搜索失败时正确释放已分配的页面内存。

技术意义

虽然泄漏量不大（仅4字节），但在数据库系统中，任何内存泄漏都可能随着系统长时间运行而累积，最终导致内存耗尽。特别是对于Vinyl这样的LSM-tree存储引擎，频繁的页面分配和释放是其核心操作之一，确保内存管理的正确性至关重要。

经验教训

1. 错误处理路径中的资源释放容易被忽视，需要特别关注
2. 自动化测试工具（如AddressSanitizer）对于发现内存问题非常有效
3. 即使是小量内存泄漏也不应忽视，特别是在数据库核心组件中

这个问题也提醒开发者在编写错误处理代码时，需要全面考虑所有已分配资源的释放，确保程序的健壮性。