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

add example with quantified specs #99

Merged
merged 3 commits into from
Oct 16, 2023
Merged

add example with quantified specs #99

merged 3 commits into from
Oct 16, 2023

Conversation

ailrst
Copy link
Contributor

@ailrst ailrst commented Oct 13, 2023

This adds an example we tried to put together for the demo, with some notes on what was necessary to get the quantification specs to work.

This example isn't working well enough to become a test case or anything since the tool currently mis-identifies some stack regions as mem, but I want to keep the notes around.

@l-kent
Copy link
Contributor

l-kent commented Oct 13, 2023

Please add the .adt, .bir and .relf files for this example.

I'll merge this, but the specification won't work until you merge in #93 which this relies on.

Which stack regions are misidentified at present? Please submit an issue for that.

@ailrst
Copy link
Contributor Author

ailrst commented Oct 13, 2023

  l00000460:
    R8, Gamma_R8 := memory_load64_le(mem, bvadd64(R31, 32bv64)), (gamma_load64(Gamma_mem, bvadd64(R31, 32bv64)) || L(mem, bvadd64(R31, 32bv64)));

Should this load be from stack since its an offset from SP?

@l-kent
Copy link
Contributor

l-kent commented Oct 13, 2023

Minor note but it's .bir not .bil. BIL is a different output from BAP.

That does look like it should be a load from stack, but please make an issue for it.

@ailrst
Copy link
Contributor Author

ailrst commented Oct 13, 2023

Minor note but it's .bir not .bil. BIL is a different output from BAP.

Ah, sorry I remembered your comment on the other pr but couldn't remember which one was correct :(

@l-kent l-kent merged commit 8929c41 into main Oct 16, 2023
1 check passed
@l-kent l-kent deleted the cntlm-examples branch November 6, 2023 23:51
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