Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DAOS-6808 rpc: fix memleak in MGMT_TGT_CREATE corpc (#5037) #5110

Merged
merged 1 commit into from
Mar 26, 2021

Conversation

liuxuezhao
Copy link
Contributor

  1. provide co_post_reply to free memory
  2. change corpc to not call co_post_reply on root node.

Signed-off-by: Xuezhao Liu xuezhao.liu@intel.com

1. provide co_post_reply to free memory
2. change corpc to not call co_post_reply on root node.

Signed-off-by: Xuezhao Liu <xuezhao.liu@intel.com>
Copy link
Collaborator

@daosbuild1 daosbuild1 left a comment

Choose a reason for hiding this comment

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

LGTM. No errors found by checkpatch.

@liuxuezhao liuxuezhao requested a review from a team March 26, 2021 00:53
@liuxuezhao
Copy link
Contributor Author

@gnailzenh or other gatekeeper, can it be landed? the NLT warning reported is completing unrelated with this patch

@mjmac mjmac merged commit ed434c5 into release/1.2 Mar 26, 2021
@mjmac mjmac deleted the lxz/fix_leak_1.2 branch March 26, 2021 18:50
@ashleypittman ashleypittman mentioned this pull request Apr 28, 2021
@ashleypittman ashleypittman mentioned this pull request May 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

6 participants