ToxicMushroom/Dalamud
1
0
Fork 0
mirror of https://github.com/goatcorp/Dalamud.git synced 2026-02-24 21:51:49 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

fix: prevent AsmHook from corrupting ReadJmpCallSig

Browse source
This commit is contained in:
Loskh 2026-01-11 21:02:14 +08:00
parent 214d9027b5
commit 647e2b3c0f
1 changed files with 9 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

14
Dalamud/Game/SigScanner.cs
View file

@ -287,16 +287,15 @@ public class SigScanner : IDisposable, ISigScanner
}
var scanRet = Scan(this.TextSectionBase, this.TextSectionSize, signature);
if (this.IsCopy)
scanRet = new IntPtr(scanRet.ToInt64() - this.moduleCopyOffset);
var insnByte = Marshal.ReadByte(scanRet);
var baseAddress = this.IsCopy ? this.moduleCopyOffset + this.Module.BaseAddress : this.Module.BaseAddress;
if (insnByte == 0xE8 || insnByte == 0xE9)
{
scanRet = ReadJmpCallSig(scanRet);
var rel = scanRet - this.Module.BaseAddress;
var rel = scanRet - baseAddress;
if (rel < 0 || rel >= this.TextSectionSize)
{
throw new KeyNotFoundException(
@ -304,6 +303,11 @@ public class SigScanner : IDisposable, ISigScanner
}
}
if (this.IsCopy)
{
scanRet = scanRet - this.moduleCopyOffset;
}
// If this is below the module, there's bound to be a problem with the sig/resolution... Let's not save it
// TODO: THIS IS A HACK! FIX THE ROOT CAUSE!
if (this.textCache != null && scanRet.ToInt64() >= this.Module.BaseAddress.ToInt64())