-
-
Notifications
You must be signed in to change notification settings - Fork 223
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
Docker Integration #32
Conversation
Adds an option to use docker container directly in emba using -D.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like your integration of docker into the basic emba command.
some short notes:
- Could you please add a short parameter description in the help output
- In my tests I had some issues with parameters. For example I got an error with the following combination:
./emba.sh -z -s -l ./logs_dwr-118 -f /home/m1k3/git-repos/testimages/testimages/testimages/DWR-118_RevA_Firmware101b02.bin -D
=> results in the following error:
./emba.sh: illegal option -- [-] Invalid option
- Please include a check on docker-compose
Thanks for your contribution
emba.sh
Outdated
done | ||
|
||
print_output "[*] Emba initializes kali docker container." "no_log" | ||
FIRMWARE="$FIRMWARE_PATH" LOG="$LOG_DIR" docker-compose run emba ./emba.sh -l /log/ -f /firmware/ "$ARGS" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please do a check to ensure docker-compose is available on the host?
small improvements before merging
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Anemosx Very nice addon for emba. Thank you for your contribution.
I will open some Issues to track the current limitations:
- No binwalk within docker
- No cwe module
- No cve-search
Adds an option to use docker container directly in emba using -D.