mirror of
https://github.com/NationalSecurityAgency/ghidra.git
synced 2025-10-06 03:50:02 +02:00
GP-2477_James_prevent_large_scalar_conversions
This commit is contained in:
parent
62c0e444a5
commit
e8bda10bc1
1 changed files with 3 additions and 1 deletions
|
@ -51,6 +51,8 @@ public abstract class ConvertConstantAction extends AbstractDecompilerAction {
|
||||||
* that corresponds with the selected constant in the decompiler window.
|
* that corresponds with the selected constant in the decompiler window.
|
||||||
*/
|
*/
|
||||||
private final static int MAX_INSTRUCTION_WINDOW = 20;
|
private final static int MAX_INSTRUCTION_WINDOW = 20;
|
||||||
|
|
||||||
|
private static final int MAX_SCALAR_SIZE = 8;
|
||||||
protected DecompilePlugin plugin;
|
protected DecompilePlugin plugin;
|
||||||
private FontMetrics metrics = null;
|
private FontMetrics metrics = null;
|
||||||
private int convertType; // The EquateSymbol conversion type performed by the action
|
private int convertType; // The EquateSymbol conversion type performed by the action
|
||||||
|
@ -209,7 +211,7 @@ public abstract class ConvertConstantAction extends AbstractDecompilerAction {
|
||||||
return null;
|
return null;
|
||||||
}
|
}
|
||||||
Varnode convertVn = tokenAtCursor.getVarnode();
|
Varnode convertVn = tokenAtCursor.getVarnode();
|
||||||
if (convertVn == null || !convertVn.isConstant()) {
|
if (convertVn == null || !convertVn.isConstant() || convertVn.getSize() > MAX_SCALAR_SIZE) {
|
||||||
return null;
|
return null;
|
||||||
}
|
}
|
||||||
HighSymbol symbol = convertVn.getHigh().getSymbol();
|
HighSymbol symbol = convertVn.getHigh().getSymbol();
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue