Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update ldr_amd64.s resolver to use OriginalBase #3

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

guervild
Copy link

Hello

Thank you for your awesome work.

I used it on internal testing with one of my tool and noticed i had to change the value of the Dll address against a security product to use the one in OriginalBase (see https://www.vergiliusproject.com/kernels/x64/Windows%2010%20|%202016/2110%2021H2%20(November%202021%20Update)/_LDR_DATA_TABLE_ENTRY).

If you think it should be merge, it would be interesting to change the comment too ^^

Signed-off-by: guervild <11190755+guervild@users.noreply.github.com>
@f1zm0 f1zm0 added the question Further information is requested label Jun 21, 2023
@f1zm0
Copy link
Owner

f1zm0 commented Jun 21, 2023

Thanks for the contribution!

Out of curiosity, have you also tried to retrieve the address in DllBase with equivalent assembly instructions?

If the detection is due to the original instructions that match a specific pattern of opcodes the security solution classifies as dangerous, maybe we can still retrieve the address from DllBase using alternative instructions very easily.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants