Skip to content

Some fixes and improvements#166

Merged
swissiety merged 5 commits intodevelopfrom
CryptoAnalysis-compliment
Jun 4, 2025
Merged

Some fixes and improvements#166
swissiety merged 5 commits intodevelopfrom
CryptoAnalysis-compliment

Conversation

@smeyer198
Copy link
Collaborator

Some fixes and improvements from integrating the new Boomerang version into CryptoAnalysis:

  • Some fixes in the Opal Scope
  • The computation of final weights at the end of an object lifetime was not correct
  • The TransitionFunctionImpl stored the state changing transitions and state changing statement in two separate sets. This does not allow us to connect corresponding transitions and statements. Solution: Use a Map to map the transitions to their statements. For example, assuming a state machine 0 -[ new A() ]-> 1 -[ op ]-> 2 in the program
A a = new A();
if (...) {
    a.op();
}
// a is either in state 1 or 2

the current solution just stores the sets {[0->1], [0->2]} and {<init>, op}. With the map, we have the mapping {[0->1] -> <init>, [0->2] -> op}, giving us the information that state 1 is reached with the call to <init> and state 2 is reached with the call to op

@smeyer198 smeyer198 requested a review from swissiety June 3, 2025 09:26
@swissiety swissiety enabled auto-merge June 4, 2025 13:51
Copy link
Member

@swissiety swissiety left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great 👍 !

@swissiety swissiety merged commit 1bf94a8 into develop Jun 4, 2025
5 checks passed
@swissiety swissiety deleted the CryptoAnalysis-compliment branch June 4, 2025 13:55
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.

2 participants