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

THREAD_SAFETY_VIOLATION (from Sonatype Lift) #99

Closed
vorburger opened this issue Mar 6, 2023 · 2 comments
Closed

THREAD_SAFETY_VIOLATION (from Sonatype Lift) #99

vorburger opened this issue Mar 6, 2023 · 2 comments

Comments

@vorburger
Copy link
Owner

Sonatype Lift raised a THREAD_SAFETY_VIOLATION in ManagedProcess#assertWaitForIsValid() :

THREAD_SAFETY_VIOLATION: Read/Write race. Non-private method ManagedProcess.assertWaitForIsValid() indirectly reads without synchronization from this.isAlive. Potentially races with write in method ManagedProcess.startExecute().
Reporting because another access to the same memory occurs on a background thread, although this access may not.

@cardamon is this the sort of thing you would enjoy contributing a fix for?

I'm hesitant whether or not to already merge your PR #96 as-is. Thoughts?

@mosesn
Copy link
Contributor

mosesn commented Apr 12, 2023

I would vote for merging in #96 as-is since it doesn't make the problem worse AFAICT.

@vorburger
Copy link
Owner Author

The issue on Sonatype Lift is now gone.

Closing this as fixed by #103.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants