yetangitu/ghidra
1
0
Fork 0
mirror of https://github.com/NationalSecurityAgency/ghidra.git synced 2025-10-04 18:29:37 +02:00
Code Issues Releases Wiki Activity

GP-1086 - Fix PDB constructor signature bug

Browse source
This commit is contained in:
ghizard 2021-06-29 14:12:55 -04:00
parent fffc06c0bc
commit e12d02eeb2
1 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

8
Ghidra/Features/PDB/src/main/java/ghidra/app/util/pdb/pdbapplicator/AbstractFunctionTypeApplier.java
View file

@ -105,7 +105,7 @@ public abstract class AbstractFunctionTypeApplier extends MsTypeApplier {
/**
* Returns whether the function has a "this" pointer
* @return {@code true} if it has a "this" pointer
* @return {@code true} if it has a "this" pointer
*/
protected abstract boolean hasThisPointer();
@ -131,7 +131,7 @@ public abstract class AbstractFunctionTypeApplier extends MsTypeApplier {
/**
* Method to create the {@link DataType} based upon the type indices of the calling
* convention, return type, and arguments list.
* convention, return type, and arguments list.
* @param callingConventionParam Identification of the {@link AbstractMsType} record of the
* {@link CallingConvention}.
* @param hasThisPointerParam true if has a this pointer
@ -212,7 +212,7 @@ public abstract class AbstractFunctionTypeApplier extends MsTypeApplier {
private boolean setReturnType() {
if (isConstructor()) {
return false;
return true;
}
DataType returnDataType = returnApplier.getDataType();
@ -234,7 +234,7 @@ public abstract class AbstractFunctionTypeApplier extends MsTypeApplier {
// Since we are a member function, we will always assume a _thiscall...
// but how do we know it is not a atatic member function (no "this")?
switch (callingConvention) {
// TODO: figure all of these out.
// TODO: figure all of these out.
case THISCALL: // "this" passed in register (we have not yet seen this)
convention = GenericCallingConvention.thiscall; // Is this correct if in reg?
break;