Skip to content

Commit 4631ece

Browse files
authored
[cDAC] Resolve breaking change caused by removing ThunkHeap descriptor (#113737)
1 parent 97a8af9 commit 4631ece

File tree

5 files changed

+1
-11
lines changed

5 files changed

+1
-11
lines changed

src/native/managed/cdacreader/Microsoft.Diagnostics.DataContractReader.Abstractions/Contracts/ILoader.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,6 @@ public interface ILoader : IContract
4444
public virtual string GetFileName(ModuleHandle handle) => throw new NotImplementedException();
4545

4646
public virtual TargetPointer GetLoaderAllocator(ModuleHandle handle) => throw new NotImplementedException();
47-
public virtual TargetPointer GetThunkHeap(ModuleHandle handle) => throw new NotImplementedException();
4847
public virtual TargetPointer GetILBase(ModuleHandle handle) => throw new NotImplementedException();
4948
public virtual ModuleLookupTables GetLookupTables(ModuleHandle handle) => throw new NotImplementedException();
5049

src/native/managed/cdacreader/Microsoft.Diagnostics.DataContractReader.Contracts/Contracts/Loader_1.cs

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -56,12 +56,6 @@ TargetPointer ILoader.GetLoaderAllocator(ModuleHandle handle)
5656
return module.LoaderAllocator;
5757
}
5858

59-
TargetPointer ILoader.GetThunkHeap(ModuleHandle handle)
60-
{
61-
Data.Module module = _target.ProcessedData.GetOrAdd<Data.Module>(handle.Address);
62-
return module.ThunkHeap;
63-
}
64-
6559
TargetPointer ILoader.GetILBase(ModuleHandle handle)
6660
{
6761
Data.Module module = _target.ProcessedData.GetOrAdd<Data.Module>(handle.Address);

src/native/managed/cdacreader/Microsoft.Diagnostics.DataContractReader.Contracts/Data/Module.cs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ public Module(Target target, TargetPointer address)
2121
Assembly = target.ReadPointer(address + (ulong)type.Fields[nameof(Assembly)].Offset);
2222
Base = target.ReadPointer(address + (ulong)type.Fields[nameof(Base)].Offset);
2323
LoaderAllocator = target.ReadPointer(address + (ulong)type.Fields[nameof(LoaderAllocator)].Offset);
24-
ThunkHeap = target.ReadPointer(address + (ulong)type.Fields[nameof(ThunkHeap)].Offset);
2524
DynamicMetadata = target.ReadPointer(address + (ulong)type.Fields[nameof(DynamicMetadata)].Offset);
2625
Path = target.ReadPointer(address + (ulong)type.Fields[nameof(Path)].Offset);
2726
FileName = target.ReadPointer(address + (ulong)type.Fields[nameof(FileName)].Offset);
@@ -40,7 +39,6 @@ public Module(Target target, TargetPointer address)
4039
public uint Flags { get; init; }
4140
public TargetPointer Base { get; init; }
4241
public TargetPointer LoaderAllocator { get; init; }
43-
public TargetPointer ThunkHeap { get; init; }
4442
public TargetPointer DynamicMetadata { get; init; }
4543
public TargetPointer Path { get; init; }
4644
public TargetPointer FileName { get; init; }

src/native/managed/cdacreader/src/Legacy/SOSDacImpl.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -844,7 +844,6 @@ int ISOSDacInterface.GetModuleData(ulong moduleAddr, DacpModuleData* data)
844844
data->metadataSize = readOnlyMetadata.Size;
845845

846846
data->LoaderAllocator = contract.GetLoaderAllocator(handle);
847-
data->ThunkHeap = contract.GetThunkHeap(handle);
848847

849848
Target.TypeInfo lookupMapTypeInfo = _target.GetTypeInfo(DataType.ModuleLookupMap);
850849
ulong tableDataOffset = (ulong)lookupMapTypeInfo.Fields[Constants.FieldNames.ModuleLookupMap.TableData].Offset;
@@ -861,6 +860,7 @@ int ISOSDacInterface.GetModuleData(ulong moduleAddr, DacpModuleData* data)
861860
data->dwModuleID = 0;
862861
data->dwBaseClassIndex = 0;
863862
data->dwModuleIndex = 0;
863+
data->ThunkHeap = 0;
864864
}
865865
catch (global::System.Exception e)
866866
{

src/native/managed/cdacreader/tests/MockDescriptors/MockDescriptors.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,6 @@ internal record TypeFields
132132
new(nameof(Data.Module.Flags), DataType.uint32),
133133
new(nameof(Data.Module.Base), DataType.pointer),
134134
new(nameof(Data.Module.LoaderAllocator), DataType.pointer),
135-
new(nameof(Data.Module.ThunkHeap), DataType.pointer),
136135
new(nameof(Data.Module.DynamicMetadata), DataType.pointer),
137136
new(nameof(Data.Module.Path), DataType.pointer),
138137
new(nameof(Data.Module.FileName), DataType.pointer),

0 commit comments

Comments
 (0)