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.

Sign up for GitHub

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

Jump to bottom

Testsuite memory leaks via AccessControlContext in EnhancedQueueExecutor #19249

Closed
Sanne opened this issue Aug 5, 2021 · 2 comments
Closed

Testsuite memory leaks via AccessControlContext in EnhancedQueueExecutor #19249

Sanne opened this issue Aug 5, 2021 · 2 comments
Assignees
@Sanne
Labels
area/core kind/bug Something isn't working

Comments

@Sanne
Copy link
Member

Sanne commented Aug 5, 2021 •
edited
Loading

Describe the bug

The EnhancedQueueExecutor eagerly captures an AccessControlContext to run MBeanRegisterAction as a privileged action, and this holds a reference to the context classloader which was set during initialization.

As we switch classloaders, this reference is leaking. Ironically we don't use MBeans at all, so the fix is simple: don't capture any AccessControlContext when MBeans are disabled.
@Sanne Sanne added kind/bug Something isn't working area/core labels Aug 5, 2021
@Sanne Sanne self-assigned this Aug 5, 2021
@Sanne
Copy link
Member Author

Sanne commented Aug 5, 2021

This will need jbossas/jboss-threads#109

@Sanne
Copy link
Member Author

Sanne commented Oct 5, 2023

This has been resolved since #31734

@Sanne Sanne closed this as completed Oct 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Sanne Sanne

Labels
area/core kind/bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Sanne