Skip to content

Instruct LeakSanitizer to ignore designated memory leaks of Server and Singleton#2858

Merged
wwbmmm merged 2 commits into
apache:masterfrom
chenBright:lsan
Jan 6, 2025
Merged

Instruct LeakSanitizer to ignore designated memory leaks of Server and Singleton#2858
wwbmmm merged 2 commits into
apache:masterfrom
chenBright:lsan

Conversation

@chenBright

@chenBright chenBright commented Dec 31, 2024

Copy link
Copy Markdown
Contributor

What problem does this PR solve?

Issue Number: resolve #2164, resolve #1390, resolve #2470

Problem Summary:

What is changed and the side effects?

Changed:

让LSan忽略Server和Singleton一些设计上预期内的内存泄漏。

Side effects:

  • Performance effects(性能影响):

  • Breaking backward compatibility(向后兼容性):


Check List:

  • Please make sure your changes are compilable(请确保你的更改可以通过编译).
  • When providing us with a new feature, it is best to add related tests(如果你向我们增加一个新的功能, 请添加相关测试).
  • Please follow Contributor Covenant Code of Conduct.(请遵循贡献者准则).

Comment thread src/brpc/server.cpp Outdated
@wwbmmm

wwbmmm commented Jan 2, 2025

Copy link
Copy Markdown
Contributor

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

memory leak in brpc::Server _keytable_pool and _session_local_data_pool may cause memory leak. memory leak if binding failed

2 participants