search

garyttierney / ghidra-batteries-included

Builds of Ghidra 10.1 with additional components included.
65 stars 5 forks source link

PS3 support #5

Open r3sus opened 4 years ago

r3sus commented 4 years ago

Hello and thanks a lot for your work! That'd be very useful to add PS3 .ELF support in Ghidra. I tried various scripts and add-ons to Ghidra but got only issues:

  1. clienthax/Ps3GhidraScripts issue

  2. zecoxao/ps3_ghidra a. CELL PPU Processor Module b. find_stdu.java issue

I didn't tried: https://github.com/aerosoul94/GhidraSPU :> https://github.com/bevanweiss/GhidraSPU/tree/TrialChanges

Thanks.

garyttierney commented 4 years ago

Next build will include the PPU and SPU sleigh processor modules.

r3sus commented 4 years ago

Awesome, thank you! Xbox360 support will be useful too, but doesn't supports 9.2 now https://github.com/zeroKilo/XEXLoaderWV/issues/8

clienthax commented 4 years ago

@garyttierney To avoid issues with decompliation on ps3 the following change is needed in Processors\PowerPC\data\languages\ppc_64_32.cspec Add to the list.

Apart from that my scripts just need the indexes bumping due to a header change in ghidra.. (Why the heck don't they allow me to lookup things by name)

garyttierney commented 4 years ago

Hi @clienthax,

the following change is needed in Processors\PowerPC\data\languages\ppc_64_32.cspec Add to the list.

Is this a commit you're referring to? I think I have a copy of your PPC64 SLEIGH changes from maybe a year or so ago, but it seems to be missing from your repo now.

clienthax commented 4 years ago

Hi @clienthax,

the following change is needed in Processors\PowerPC\data\languages\ppc_64_32.cspec Add to the list.

Is this a commit you're referring to? I think I have a copy of your PPC64 SLEIGH changes from maybe a year or so ago, but it seems to be missing from your repo now.

Gah.. copypasta fail!

add r2 to the <unaffected> list

garyttierney commented 4 years ago

Ah, got it. I'll include a patch for that in my next PR. Is there any reason not to send that upstream to Ghidra?

clienthax commented 3 years ago

Ah, got it. I'll include a patch for that in my next PR. Is there any reason not to send that upstream to Ghidra?

I think it's been submitted.. somewhere, likely by aero

garyttierney commented 3 years ago

Yep, found it. For posterity: https://github.com/NationalSecurityAgency/ghidra/issues/570#issuecomment-701728185.

I'll open a PR for this.

garyttierney commented 3 years ago

Patch included here: https://github.com/garyttierney/ghidra-batteries-included/pull/28. I'll keep this open 'til I bundle the SELF analyzer script and the sleigh module for the SPU.

PikminGuts92 commented 3 years ago

Any update on this?

clienthax commented 3 years ago

@garyttierney my scripts are now an extension https://github.com/clienthax/Ps3GhidraScripts/releases