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

Potential lock leakage #3116

Closed
2 tasks done
Citymonstret opened this issue Jun 24, 2021 · 1 comment
Closed
2 tasks done

Potential lock leakage #3116

Citymonstret opened this issue Jun 24, 2021 · 1 comment

Comments

@Citymonstret
Copy link
Member

Server Implementation

Paper

Server Version

1.17.1

Describe the bug

There's a potential lock leakage in the LockRepository (https://github.com/IntellectualSites/PlotSquared/blob/v6/Core/src/main/java/com/plotsquared/core/synchronization/LockRepository.java#L73)

To Reproduce

Have the runnable throw an exception

Expected behaviour

The lock should still unlock even if failing exceptionally

Screenshots / Videos

No response

Error log (if applicable)

No response

Plot Debugpaste

.

PlotSquared Version

.

Checklist

Anything else?

No response

@Citymonstret Citymonstret added the Requires Testing This is a new issue which needs to be approved before labeled with "bug" label Jun 24, 2021
@stale
Copy link

stale bot commented Aug 23, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If the issue is still present and can be reproduced, please let the team know. Thank you for your contributions.

@NotMyFault NotMyFault removed the Requires Testing This is a new issue which needs to be approved before labeled with "bug" label Oct 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants