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

Kernel config extraction in S24 #364

Merged
merged 5 commits into from
Oct 25, 2022

Conversation

m-1-k-3
Copy link
Member

@m-1-k-3 m-1-k-3 commented Oct 24, 2022

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Feature
Small bug fixes

  • What is the current behavior? (You can also link to an open issue here)

No extraction (and no testing) of kernel config from binary kernel

  • What is the new behavior (if this is a feature change)? If possible add a screenshot.

Extract kernel config from binary kernel with https://raw.githubusercontent.com/torvalds/linux/master/scripts/extract-ikconfig (which is integrated in s24 module)

  • Limitations:

Kernel config checks with EMBA parameter -k is currently disabled and needs some further attention!

Testfirmware: https://github.com/OWASP/IoTGoat/releases/download/v1.0/IoTGoat-raspberry-pi2.img
Start EMBA the following way: sudo ./emba.sh -f ~/Downloads/IoTGoat-raspberry-pi2.img -l ~/emba_logs_IoTGoat -S -p ./scan-profiles/default-scan.emba -m s24

image

image

image

@m-1-k-3 m-1-k-3 added the enhancement New feature or request label Oct 24, 2022
@BenediktMKuehne
Copy link
Member

BenediktMKuehne commented Oct 25, 2022

Tested with openwrt image
image
guessing this is expected/normal behavior

@m-1-k-3
Copy link
Member Author

m-1-k-3 commented Oct 25, 2022

Tested with openwrt image image guessing this is expected/normal behavior

This is an error from vmlinux_to_elf. Happens on different firmware files. This error does not affect the rest of the process.

@m-1-k-3
Copy link
Member Author

m-1-k-3 commented Oct 25, 2022

Probably we should dump the errors to /dev/null

@BenediktMKuehne BenediktMKuehne merged commit b691612 into e-m-b-a:master Oct 25, 2022
@m-1-k-3 m-1-k-3 deleted the kernel_config branch October 30, 2022 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants