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

Diagnostic commands require additional Permission's #41

Closed
pfirmstone opened this issue Dec 10, 2024 · 3 comments
Closed

Diagnostic commands require additional Permission's #41

pfirmstone opened this issue Dec 10, 2024 · 3 comments

Comments

@pfirmstone
Copy link
Owner

pfirmstone commented Dec 10, 2024 •
edited
Loading

src/hotspot/share/services/diagnosticCommand.cpp

Refer to Line 155.
@pfirmstone
Copy link
Owner Author

see also

src/hotspot/share/services/diagnosticFramework.hpp

line 264

@pfirmstone
Copy link
Owner Author

Commit with relevant details
openjdk@59d4e28

@pfirmstone pfirmstone changed the title Enhanced JMX Agent Support DiagnosticCommandMBean commands require additional Permission's DiagnosticCommandMBean commands require additional Permission's Dec 16, 2024
@pfirmstone
Copy link
Owner Author

Two commands were added recently without Permission's

8337199: Add jcmd Thread.vthread_scheduler and Thread.vthread_pollers diagnostic commands

pfirmstone added a commit that referenced this issue Dec 16, 2024
@pfirmstone
DiagnosticCommandMBean commands require additional Permission's #41
0ceaa35 
Add permission checks to jcmd Thread.vthread_scheduler and Thread.vthread_pollers diagnostic commands
@pfirmstone pfirmstone changed the title DiagnosticCommandMBean commands require additional Permission's Diagnostic commands require additional Permission's Dec 16, 2024
@pfirmstone pfirmstone mentioned this issue Dec 30, 2024
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@pfirmstone