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

Fix incorrect comparison for symbolic file wildcards #2454

Merged
merged 6 commits into from
Jun 2, 2021

Conversation

ekilmer
Copy link
Contributor

@ekilmer ekilmer commented May 15, 2021

  • Add a test that makes sure the fileio.c example passes with the correct exploration of the program.
  • Add a test that ensures a symbolic file with wildcard characters returns the correct number of detected wildcard characters when opened and read

Addresses #2453

Python has no character type, so we need to assert that the length of
the passed wildcard is 1 character
@ehennenfent ehennenfent added this to In progress in Manticore May 18, 2021
@ekilmer ekilmer requested a review from ehennenfent May 18, 2021 17:59
@ekilmer
Copy link
Contributor Author

ekilmer commented May 18, 2021

I'm not entirely sure how I should test the examples/linux/fileio.c file.

Should I write a test in the tests/native directory? Should a Manticore script live in the examples/scripts directory?

@ekilmer ekilmer added the bug label May 25, 2021
It takes way too long (4 hrs) in CI now that the bug is fixed
Manticore automation moved this from In progress to Reviewer approved Jun 2, 2021
Copy link
Contributor

@ehennenfent ehennenfent left a comment

Choose a reason for hiding this comment

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

Looks good! Thanks for adding a new example.

@ekilmer ekilmer merged commit 5205fa3 into master Jun 2, 2021
@ekilmer ekilmer deleted the fix-symbolic-file-wildcard branch June 2, 2021 13:30
Manticore automation moved this from Reviewer approved to Done Jun 2, 2021
ekilmer added a commit that referenced this pull request Jul 15, 2021
* master:
  Install pinned version of truffle to fix CI (#2467)
  Use fixed owner and attacker accounts in multi_tx_analysis (#2464)
  Manticore 0.3.6 (#2456)
  Fix IntrospectionAPIPlugin Name (#2459)
  Portfolio of parallel solvers (#2420)
  Replace Quick mode with Thorough mode (#2457)
  Fix incorrect comparison for symbolic file wildcards (#2454)
  Reduce the number of calls to the SMT solver in EVM (#2411)
  Fixes to Unicorn emulation - start/stop/resume (#1796)
  Add support for multiple compilation units (#2444)
  Basic solver stats (#2415)
ekilmer added a commit that referenced this pull request Jul 27, 2021
* master: (35 commits)
  Track last_pc in StateDescriptors (#2471)
  Expose Result Register for Native CPU (#2470)
  Install pinned version of truffle to fix CI (#2467)
  Use fixed owner and attacker accounts in multi_tx_analysis (#2464)
  Manticore 0.3.6 (#2456)
  Fix IntrospectionAPIPlugin Name (#2459)
  Portfolio of parallel solvers (#2420)
  Replace Quick mode with Thorough mode (#2457)
  Fix incorrect comparison for symbolic file wildcards (#2454)
  Reduce the number of calls to the SMT solver in EVM (#2411)
  Fixes to Unicorn emulation - start/stop/resume (#1796)
  Add support for multiple compilation units (#2444)
  Basic solver stats (#2415)
  Fix the generation of EVM tests (#2426)
  Disabled EVM events in testcases by default (#2417)
  added proper timeouts for cvc4 and boolector (#2418)
  Removed use of global solver from Native Memory (#2414)
  Support to use boolector as the SMT solver (#2410)
  Update CI and suggest to use pip3 instead of pip (#2409)
  Expressions use keyword-only arguments for init (#2395)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Manticore
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants