yetangitu/ghidra
1
0
Fork 0
mirror of https://github.com/NationalSecurityAgency/ghidra.git synced 2025-10-03 09:49:23 +02:00
Code Issues Releases Wiki Activity

Merge remote-tracking branch 'origin/GP-5989_ForceUnionException' into

Browse source 
Ghidra_12.0 (Closes #7466)
This commit is contained in:
Ryan Kurtz 2025-09-25 05:03:56 -04:00
commit bcd9130aee
1 changed files with 36 additions and 32 deletions
Download patch file Download diff file Expand all files Collapse all files

64
Ghidra/Features/Decompiler/src/main/java/ghidra/app/plugin/core/decompile/actions/ForceUnionAction.java
View file

@ -128,28 +128,28 @@ public class ForceUnionAction extends AbstractDecompilerAction {
int opcode = accessOp.getOpcode(); int opcode = accessOp.getOpcode();
if (opcode == PcodeOp.PTRSUB) { if (opcode == PcodeOp.PTRSUB) {
parentDt = typeIsUnionRelated(accessOp.getInput(0)); parentDt = typeIsUnionRelated(accessOp.getInput(0));
if (parentDt == null) { if (parentDt != null) {
accessOp = null; accessVn = accessOp.getInput(0);
return; accessSlot = 0;
} if (accessOp.getInput(1).getOffset() == 0) { // Artificial op
accessVn = accessOp.getInput(0); do {
accessSlot = 0; Varnode tmpVn = accessOp.getOutput();
if (accessOp.getInput(1).getOffset() == 0) { // Artificial op PcodeOp tmpOp = tmpVn.getLoneDescend();
do { if (tmpOp == null) {
Varnode tmpVn = accessOp.getOutput(); break;
PcodeOp tmpOp = tmpVn.getLoneDescend(); }
if (tmpOp == null) { accessOp = tmpOp;
break; accessVn = tmpVn;
accessSlot = accessOp.getSlot(accessVn);
} }
accessOp = tmpOp; while (accessOp.getOpcode() == PcodeOp.PTRSUB &&
accessVn = tmpVn; accessOp.getInput(1).getOffset() == 0);
accessSlot = accessOp.getSlot(accessVn);
} }
while (accessOp.getOpcode() == PcodeOp.PTRSUB && return;
accessOp.getInput(1).getOffset() == 0);
} }
} }
else { else {
parentDt = null;
for (accessSlot = 0; accessSlot < accessOp.getNumInputs(); ++accessSlot) { for (accessSlot = 0; accessSlot < accessOp.getNumInputs(); ++accessSlot) {
accessVn = accessOp.getInput(accessSlot); accessVn = accessOp.getInput(accessSlot);
parentDt = typeIsUnionRelated(accessVn); parentDt = typeIsUnionRelated(accessVn);
@ -157,22 +157,26 @@ public class ForceUnionAction extends AbstractDecompilerAction {
break; break;
} }
} }
if (accessSlot >= accessOp.getNumInputs()) { if (parentDt != null) {
accessSlot = -1; if (opcode == PcodeOp.SUBPIECE && accessSlot == 0 &&
accessVn = accessOp.getOutput(); !(parentDt instanceof Pointer)) {
parentDt = typeIsUnionRelated(accessVn); // SUBPIECE acts directly as resolution operator
if (parentDt == null) { // Choose field based on output varnode, even though it isn't the union data-type
accessOp = null; accessSlot = -1;
return; // Give up, could not find type associated with field accessVn = accessOp.getOutput();
} }
} return;
if (opcode == PcodeOp.SUBPIECE && accessSlot == 0 && !(parentDt instanceof Pointer)) {
// SUBPIECE acts directly as resolution operator
// Choose field based on output varnode, even though it isn't the union data-type
accessSlot = -1;
accessVn = accessOp.getOutput();
} }
} }
accessSlot = -1;
accessVn = accessOp.getOutput();
if (accessVn != null) {
parentDt = typeIsUnionRelated(accessVn);
if (parentDt != null) {
return;
}
}
accessOp = null; // Give up, could not find type associated with field
} }
/** /**