Ghidra: Ghidra1's fixes for ConstantPropagationAnalyzer and PseudoDisassembler

Created on 22 Oct 2020  路  4Comments  路  Source: NationalSecurityAgency/ghidra

@Ghidra1 provided @ryanmkurtz and me with two fixes, one for each of ConstantPropagationAnalyzer and PseudoDisassembler, that have not yet been committed here on github. Could he please commit them? Thanks, @Ghidra1, for helping so much with those and other issues.
:smile:

(I don't think there were any github issues created for those problems.)

Most helpful comment

Speaking of ConstantPropagationAnalyzer's there is this also. #2157

I passed this other issue onto @emteere

All 4 comments

Speaking of ConstantPropagationAnalyzer's there is this also. https://github.com/NationalSecurityAgency/ghidra/pull/2157

Although things were a bit disorganized at the time I believe I incorporated all of the relevant changes into Ghidra for release with Ghidra 9.2 which should also be in master. Sorry to trouble you but could you please report any outstanding bugs/issues into this ticket that you believe should have been resolved. I checked my repository where I was working at the time and all changes I had made at the time appear to be present in the GitHib repo.

See commit

Please note that the changes were implemented somewhat differently in the final commit from the patch I sent to you at the time but should have the same behavior.

Speaking of ConstantPropagationAnalyzer's there is this also. #2157

I passed this other issue onto @emteere

It does look like the commit resolves the problems that I asked about in this issue, so I'm closing this.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

toor-de-force picture toor-de-force  路  3Comments

ghost picture ghost  路  3Comments

Kerilk picture Kerilk  路  3Comments

loudinthecloud picture loudinthecloud  路  3Comments

marcushall42 picture marcushall42  路  3Comments