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

Update all plugins to BAP 2.0 #58

Closed
codyroux opened this issue Aug 19, 2019 · 4 comments
Closed

Update all plugins to BAP 2.0 #58

codyroux opened this issue Aug 19, 2019 · 4 comments
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed VSA WP Involves the Weakest Precondition computation

Comments

@codyroux
Copy link
Contributor

Not sure what's required for this one, hopefully no too much except the BitVector module.

It also sounds like they're moving away from having, for every module M, a sub-module M.Set and M.Map, which I use extensively in WP, so maybe some re-factoring there as well.

@codyroux codyroux added enhancement New feature or request help wanted Extra attention is needed WP Involves the Weakest Precondition computation VSA labels Aug 19, 2019
@ivg
Copy link

ivg commented Aug 20, 2019

Nothing is required, the Bap.Std interface is backward compatible. It would be good to eventually update cbat_tools to use the more general Bap 2.0 interfaces (i.e., Bap_core_theory and Bap_knowledge) and make VSA available in the knowledge base. But this is not immediately required. Again, old analysis will continue to work for the observed future, it just won't be able to benefit from all the new features.

It also sounds like they're moving away from having, for every module M, a sub-module M.Set and M.Map, which I use extensively in WP, so maybe some re-factoring there as well.

Well, our new modules are not exposing those interfaces (old are still having the same interface), but Base/Core_kernel is making the obsolete, so expect them to disappear from Base/Core_kernel sooner or later. The change is easy, instead of M.Map one should use just Map :)

@ivg
Copy link

ivg commented Aug 20, 2019

I've recalled one semantic change that might affect you. When we reify program semantics into BIR we no longer create the ret terms. Now BAP 2.x you won't have any return dst terms, instead it will be reified to a more general call dst with noreturn, where dst is some expression denoting the set of possible return destinations.

Do you think it will affect you?

@codyroux
Copy link
Contributor Author

codyroux commented Aug 20, 2019 via email

@fortunac
Copy link

Handled in #65

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed VSA WP Involves the Weakest Precondition computation
Projects
None yet
Development

No branches or pull requests

5 participants