_keytable_pool and _session_local_data_pool may cause memory leak.

[ehds]

Describe the bug (描述bug)
在 Server::StartInternal 中，_keytable_pool 和 _session_local_data_pool 通过 new 来分配内存，在 Server 析构时，会调用 Join 函数来释放其资源。 但是Join只有当 _status != RUNNING && _status != STOPPING 时才会进入析构的逻辑。

brpc/src/brpc/server.cpp

Lines 1187 to 1190 in fc885ad

	int Server::Join() {
	if (_status != RUNNING && _status != STOPPING) {
	return -1;
	}

所以如果在 Server::StartInternal 中如果状态未成为 RUNNING (由于 BuildAcceptor 失败等），那么其分配的内存无法被释放。

To Reproduce (复现方法)
开启ASAN检测，使 BuildAcceptor 失败（设置一个被占用的端口等），会检测出 memory leak 错误。

Expected behavior (期望行为)
即使 Server start 失败，应该不会内存泄漏发生。

Versions (各种版本)
OS:
Compiler:
brpc:
protobuf:

Additional context/screenshots (更多上下文/截图)

[lhsoft]

看代码注释主要是和bthread间同步的问题，导致不好删除，在没修复bug之前，可以先让asan ignore这个object

[ehds]

感谢回复，目前是通过ignore的方式暂时禁止了，因为leak的大小属于可接受范围. 发自我的 iPhone在 2023年7月24日，17:48，huliu ***@***.***> 写道： 看代码注释主要是和bthread间同步的问题，导致不好删除，在没修复bug之前，可以先让asan ignore这个object —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: ***@***.***>

[lhsoft]

你们是怎么禁止的呢，这个object是private的，我在外部拿不到，只能改brpc代码，要是有不改代码ignore的方式就完美了

[ehds]

加上这个 LSAN_OPTIONS 启动参数，指定忽略检测 leak 的规则文件：

LSAN_OPTIONS=suppressions=ignore.txt

ignore.txt 指定忽略 Server::StartInternal 这个函数的检测

cat ignore.txt :

leak:Server::StartInternal

更详细的用法参考：https://github.com/google/sanitizers/wiki/AddressSanitizerLeakSanitizer#flags

[chenBright]

1. 如果编译bRPC库的时候开了ASan，Instruct LeakSanitizer to ignore designated memory leaks of Server and Singleton #2858 会自动忽略这些内存泄漏。
2. 如果没开，代码里使用LSan注解也可以忽略：

brpc::Server server;
{
    ANNOTATE_SCOPED_MEMORY_LEAK;
    server.Start(...);
}