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

GP-5219 - C++/PDB improve uniqueness mechanism for ClassFieldAttributes

Browse source
This commit is contained in:
ghizard 2024-12-29 16:34:13 -05:00
parent 1b27a76de2
commit d7f43f0854
3 changed files with 20 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

2
Ghidra/Features/PDB/src/main/java/ghidra/app/util/pdb/classtype/Access.java
View file

@ -19,7 +19,7 @@ import java.util.HashMap;
import java.util.Map; import java.util.Map;
/** /**
* * Class access attributes
*/ */
//---------------------------------------------------------------------------------------------- //----------------------------------------------------------------------------------------------
// TODO: Consider expanding these beyond C++. // TODO: Consider expanding these beyond C++.

24
Ghidra/Features/PDB/src/main/java/ghidra/app/util/pdb/classtype/ClassFieldAttributes.java
View file

@ -15,28 +15,36 @@
*/ */
package ghidra.app.util.pdb.classtype; package ghidra.app.util.pdb.classtype;
import java.util.*; import java.util.Objects;
import ghidra.app.util.bin.format.pdb2.pdbreader.type.ClassFieldMsAttributes; import ghidra.app.util.bin.format.pdb2.pdbreader.type.ClassFieldMsAttributes;
/** /**
* * Class field attributes
*/ */
public class ClassFieldAttributes { public class ClassFieldAttributes {
private static final Map<ClassFieldAttributes, ClassFieldAttributes> map = new HashMap<>(); private static final ClassFieldAttributes all[][] =
new ClassFieldAttributes[Access.values().length][Property.values().length];
// These initializations use the map above, so it must be initialized first static {
for (Access access : Access.values()) {
for (Property property : Property.values()) {
all[access.ordinal()][property.ordinal()] =
new ClassFieldAttributes(access, property);
}
}
}
// These initializations use the 'all' array above, so it must be initialized first
public static final ClassFieldAttributes UNKNOWN = get(Access.UNKNOWN, Property.UNKNOWN); public static final ClassFieldAttributes UNKNOWN = get(Access.UNKNOWN, Property.UNKNOWN);
public static final ClassFieldAttributes BLANK = get(Access.BLANK, Property.BLANK); public static final ClassFieldAttributes BLANK = get(Access.BLANK, Property.BLANK);
private final Access access; private final Access access;
private final Property property; private final Property property;
public synchronized static ClassFieldAttributes get(Access access, Property property) { public static ClassFieldAttributes get(Access access, Property property) {
ClassFieldAttributes key = new ClassFieldAttributes(access, property); return all[access.ordinal()][property.ordinal()];
ClassFieldAttributes cfa = map.putIfAbsent(key, key);
return (cfa != null) ? cfa : key;
} }
public static ClassFieldAttributes convert(ClassFieldMsAttributes msAtts, public static ClassFieldAttributes convert(ClassFieldMsAttributes msAtts,

2
Ghidra/Features/PDB/src/main/java/ghidra/app/util/pdb/classtype/Property.java
View file

@ -19,7 +19,7 @@ import java.util.HashMap;
import java.util.Map; import java.util.Map;
/** /**
* * Class properties
*/ */
public enum Property { public enum Property {
UNKNOWN("INVALID_PROPERTY", -1), UNKNOWN("INVALID_PROPERTY", -1),