yetangitu/ghidra
1
0
Fork 0
mirror of https://github.com/NationalSecurityAgency/ghidra.git synced 2025-10-05 02:39:44 +02:00
Code Issues Releases Wiki Activity

GP-4899 Full check for switch target in isDoNothing

Browse source
This commit is contained in:
caheckman 2024-09-30 23:02:15 +00:00
parent a3d0b40f36
commit 580226cfa0
1 changed files with 10 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

13
Ghidra/Features/Decompiler/src/decompile/cpp/block.cc
View file

@ -2574,9 +2574,16 @@ bool BlockBasic::isDoNothing(void) const
if (sizeIn() == 0) return false; // A block that does nothing but if (sizeIn() == 0) return false; // A block that does nothing but
// is a starting block, may need to be a // is a starting block, may need to be a
// placeholder for global(persistent) vars // placeholder for global(persistent) vars
if ((sizeIn()==1)&&(getIn(0)->isSwitchOut())) { for(int4 i=0;i<sizeIn();++i) {
if (getOut(0)->sizeIn() > 1) const FlowBlock *switchbl = getIn(i);
return false; // Don't remove switch targets if (!switchbl->isSwitchOut()) continue;
if (switchbl->sizeOut() > 1) {
// This block is a switch target
if (getOut(0)->sizeIn() > 1) { // Multiple edges coming together
// Switch edge may still be propagating a unique value
return false; // Don't remove it
}
}
} }
PcodeOp *lastop = lastOp(); PcodeOp *lastop = lastOp();
if ((lastop != (PcodeOp *)0)&&(lastop->code()==CPUI_BRANCHIND)) if ((lastop != (PcodeOp *)0)&&(lastop->code()==CPUI_BRANCHIND))