-
-
Notifications
You must be signed in to change notification settings - Fork 228
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
Python runner module #1264
Comments
Thank you for contributing an issue! Welcome to the EMBA firmware analysis community! We are glad you are here and appreciate your contribution. Please keep in mind our contributing guidelines here and here. If you like EMBA you have the chance to support us by becoming a Sponsor or buying some beer here. To show your love for EMBA with nice shirts or other merch you can check our Spreadshop. This is an automatic message. Allow for time for the EMBA community to be able to read the issue and comment on it. |
Hi @B1TC0R3 good to hear you are working on EMBA stuff. Looking forward to the results. I would recommend to build a S-module. Looks as you are doing some further python analysis. Probably the easiest way would be to integrate it into the already available python module (S21) as dedicated function. If your module is getting bigger a dedicated module would be another option. The next available number would be S28. You have already found the contribution docs. We are currently refactoring EMBA to name all local variables in the format lVAR_NAME -> see the l as first letter. You should always ensure your code passes the strict mode (start firmware analysis with -S parameter) and test it with multiple linters via check_project.sh. If you have further questions feel free to open issues or use this one |
Now I have seen your module and you should use a dedicated module for such a runner. btw good idea to build such a python runner |
Thank you! Should I still add it as an "S" module? There's a lot of bench-marking necessary before I can tell whether Bash or Python is more viable for my use case, In case Bash is marginally faster, I will add these other changes separately in the "L" or "Q" category tho. |
As you are poking with the emulation engine you need to move your modules to the L(ive testing) area (after L10 which is doing the main emulation stuff). If you plan modules which are working on the filesystem (no system emulation) you need to add S modules. Some of your mentioned checks are already available: |
Makes sense. In this case I will add the current code for the python runner as "S28" and see whether and how I can add I still need to verify my code with strict mode and check_project anyways. |
@m-1-k-3 The changes are almost done, I am currently verifying the last few things. What would be the best option to provide you with my doc files once the patch is fully ready? |
Yea ... The Github wiki is a bit ugly in collaborating. Probably the easiest way would be to write the doc into a comment here and I will transfer it to the wiki and add you as author. |
I understand. I will see that I get the bigger changes and documentation written until the end of the month. |
Is your feature request related to a problem? Please describe.
There is no problem. :)
Describe the solution you'd like
Hello, devs! 👋
I am a German IT student working on EMBA and dynamic firmware analysis as part of my Bachelors Thesis.
Currently, I am implementing new EMBA module that has the ability to execute user-supplied Python scripts during analysis.
It's still needs a bunch of work, but I wanted to clear up some questions in advance:
Priority issue
Are you already a Sponsor? - [N]
Additional context
The text was updated successfully, but these errors were encountered: