Skip to content

Class not found fix #643 #996

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

Merged
merged 1 commit into from
Sep 23, 2022
Merged

Class not found fix #643 #996

merged 1 commit into from
Sep 23, 2022

Conversation

sergeypospelov
Copy link
Member

Description

There were two problems:

  • <clinit> executing happens in any thread, so UtContext could be empty and resolving classes inside static initializer failed, since we hadn't got the correct classloader. This is exactly what was going on and we were getting ClassNotFoundException in concrete execution.
  • AccessControlException was wrapped in ExceptionInInitializerError, but we didn't handle it, so there were no comments about sandboxing.

Fixed these ones.

Fixes #643

Type of Change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Manual Scenario

To Reproduce section from #643 and internal SAT issue passes.

Checklist:

  • The change followed the style guidelines of the UTBot project
  • Self-review of the code is passed
  • The change contains enough commentaries, particularly in hard-to-understand areas
  • No new warnings
  • All tests pass locally with my changes

@sergeypospelov sergeypospelov merged commit eba7ff4 into main Sep 23, 2022
@sergeypospelov sergeypospelov deleted the sergey/class_not_found_fix branch September 23, 2022 11:22
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

Successfully merging this pull request may close these issues.

ConcreteExecutor can't initialize a class
2 participants