Skip to content
This repository was archived by the owner on Jun 26, 2023. It is now read-only.

avoid the unrelesed lock after the method returns for issue#1037#1041

Merged
nileshgovande merged 2 commits intoSeagate:mainfrom
true-ryan:main
Jul 21, 2021
Merged

avoid the unrelesed lock after the method returns for issue#1037#1041
nileshgovande merged 2 commits intoSeagate:mainfrom
true-ryan:main

Conversation

@true-ryan
Copy link
Copy Markdown
Contributor

@true-ryan true-ryan commented Jul 20, 2021

Update s3_memory_pool.c. Avoid the unreleased lock pool->lock after the method returns. This patch for issue#1037.

@true-ryan
avoid the unrelesed lock after the method returns
23882c4 
Update s3_memory_pool.c. Avoid the unreleased lock pool->lock after the method returns.
@cla-bot
Copy link
Copy Markdown

cla-bot bot commented Jul 20, 2021

Thanks for your contribution!
The CLA bot has flagged your contribution as not having a Contributor License Agreement
in place. Note that this is not needed in the overwhelming majority of instances and this warning will usually be ignored.
The code reviewers will make a determination and may ask you to sign a CLA or may choose to ignore this warning.
More information about this can be found here.

@cortx-admin
Copy link
Copy Markdown

cortx-admin commented Jul 20, 2021

Can one of the admins verify this patch?

@cla-bot
Copy link
Copy Markdown

cla-bot bot commented Jul 20, 2021

Thanks for your contribution!
The CLA bot has flagged your contribution as not having a Contributor License Agreement
in place. Note that this is not needed in the overwhelming majority of instances and this warning will usually be ignored.
The code reviewers will make a determination and may ask you to sign a CLA or may choose to ignore this warning.
More information about this can be found here.

@nileshgovande nileshgovande mentioned this pull request Jul 21, 2021
Closed
@nileshgovande nileshgovande merged commit dd6bfbc into Seagate:main Jul 21, 2021
@welcome
Copy link
Copy Markdown

welcome bot commented Jul 21, 2021

Thanks for your contribution to CORTX! 🎉

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

Reviewers

@bkirunge7 bkirunge7 Awaiting requested review from bkirunge7

@knrajnambiar76 knrajnambiar76 Awaiting requested review from knrajnambiar76

@nileshgovande nileshgovande Awaiting requested review from nileshgovande

@t7ko-seagate t7ko-seagate Awaiting requested review from t7ko-seagate

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@true-ryan @cortx-admin @nileshgovande