Patrick Georgi has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/31806
Change subject: Docs/project_ideas: Add a stub for Ghidra integration ......................................................................
Docs/project_ideas: Add a stub for Ghidra integration
It may be useful to have a common, easily available toolbench for firmware analysis and Ghidra looks promising.
Change-Id: I56d0ff875bb939f6d31f088232f8a6fd168abbb6 Signed-off-by: Patrick Georgi pgeorgi@google.com --- M Documentation/contributing/project_ideas.md 1 file changed, 7 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/06/31806/1
diff --git a/Documentation/contributing/project_ideas.md b/Documentation/contributing/project_ideas.md index 117dcce..0c94560 100644 --- a/Documentation/contributing/project_ideas.md +++ b/Documentation/contributing/project_ideas.md @@ -172,3 +172,10 @@
### Mentors * Patrick Georgi patrick@georgi.software + +## Extend Ghidra to support analysis of firmware images +[Ghidra](https://ghidra-sre.org) is a recently released cross-platform +disassembler and decompiler that is extensible through plugins. Make it +useful for firmware related work: Automatically parse formats (eg. by +integrating UEFITool, cbfstool, decompressors), automatically identify +16/32bit code on x86, etc.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31806 )
Change subject: Docs/project_ideas: Add a stub for Ghidra integration ......................................................................
Patch Set 1: Code-Review+1
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31806 )
Change subject: Docs/project_ideas: Add a stub for Ghidra integration ......................................................................
Patch Set 1: Code-Review+1
(1 comment)
https://review.coreboot.org/#/c/31806/1/Documentation/contributing/project_i... File Documentation/contributing/project_ideas.md:
https://review.coreboot.org/#/c/31806/1/Documentation/contributing/project_i... PS1, Line 181: 16/32bit code on x86 Since most current UEFI firmware is 64-bit, I would be extending this to:
16/32/64bit code on x86_64
Hello Angel Pons, Paul Menzel, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/31806
to look at the new patch set (#2).
Change subject: Docs/project_ideas: Add a stub for Ghidra integration ......................................................................
Docs/project_ideas: Add a stub for Ghidra integration
It may be useful to have a common, easily available toolbench for firmware analysis and Ghidra looks promising.
Change-Id: I56d0ff875bb939f6d31f088232f8a6fd168abbb6 Signed-off-by: Patrick Georgi pgeorgi@google.com --- M Documentation/contributing/project_ideas.md 1 file changed, 7 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/06/31806/2
Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31806 )
Change subject: Docs/project_ideas: Add a stub for Ghidra integration ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/#/c/31806/1/Documentation/contributing/project_i... File Documentation/contributing/project_ideas.md:
https://review.coreboot.org/#/c/31806/1/Documentation/contributing/project_i... PS1, Line 181: 16/32bit code on x86
Since most current UEFI firmware is 64-bit, I would be extending this to: […]
Done
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31806 )
Change subject: Docs/project_ideas: Add a stub for Ghidra integration ......................................................................
Patch Set 3: Code-Review+2
Patrick Georgi has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/31806 )
Change subject: Docs/project_ideas: Add a stub for Ghidra integration ......................................................................
Docs/project_ideas: Add a stub for Ghidra integration
It may be useful to have a common, easily available toolbench for firmware analysis and Ghidra looks promising.
Change-Id: I56d0ff875bb939f6d31f088232f8a6fd168abbb6 Signed-off-by: Patrick Georgi pgeorgi@google.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/31806 Reviewed-by: Angel Pons th3fanbus@gmail.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M Documentation/contributing/project_ideas.md 1 file changed, 7 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Angel Pons: Looks good to me, approved
diff --git a/Documentation/contributing/project_ideas.md b/Documentation/contributing/project_ideas.md index d16e9fe..97499a8 100644 --- a/Documentation/contributing/project_ideas.md +++ b/Documentation/contributing/project_ideas.md @@ -172,3 +172,10 @@
### Mentors * Patrick Georgi patrick@georgi.software + +## Extend Ghidra to support analysis of firmware images +[Ghidra](https://ghidra-sre.org) is a recently released cross-platform +disassembler and decompiler that is extensible through plugins. Make it +useful for firmware related work: Automatically parse formats (eg. by +integrating UEFITool, cbfstool, decompressors), automatically identify +16/32/64bit code on x86/amd64, etc.
-
Angel Pons (Code Review) -
Patrick Georgi (Code Review)
-
Paul Menzel (Code Review)