Skip to content

[fix](lru_cache) fix heap-use-after-free problem for lru cache - #8569

Merged
caiconghui merged 1 commit into
apache:masterfrom
caiconghui:lru_cache
Mar 21, 2022
Merged

[fix](lru_cache) fix heap-use-after-free problem for lru cache#8569
caiconghui merged 1 commit into
apache:masterfrom
caiconghui:lru_cache

Conversation

@caiconghui

@caiconghuicaiconghui commented Mar 21, 2022

Copy link
Copy Markdown
Contributor

Proposed changes

Issue Number: close#8571

Problem Summary:

Describe the overview of changes.

Checklist(Required)

  1. Does it affect the original behavior: (Yes/No/I Don't know)
  2. Has unit tests been added: (Yes/No/No Need)
  3. Has document been added or modified: (Yes/No/No Need)
  4. Does it need to update dependencies: (Yes/No)
  5. Are there any changes that cannot be rolled back: (Yes/No)

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

@caiconghuicaiconghui changed the title [fix](lru_cache) fix memory leak for lru cache[fix](lru_cache) fix heap-use-after-free problem for lru cacheMar 21, 2022
@caiconghuicaiconghui added kind/fix Categorizes issue or PR as related to a bug. and removed Memory Leak labels Mar 21, 2022
@caiconghui

Copy link
Copy Markdown
ContributorAuthor

cc @xinyiZzz

@caiconghuicaiconghui added the dev/backlog waiting to be merged in future dev branch label Mar 21, 2022

@weizuo93weizuo93 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions

Copy link
Copy Markdown
Contributor

PR approved by at least one committer and no changes requested.

@github-actionsgithub-actionsBot added approved Indicates a PR has been approved by one committer. reviewed labels Mar 21, 2022
@github-actions

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

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

Labels

approvedIndicates a PR has been approved by one committer.dev/backlogwaiting to be merged in future dev branchkind/fixCategorizes issue or PR as related to a bug.reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] lru cache ut failed

2 participants

@caiconghui@weizuo93