Skip to content

Fix SectionAlignment - #94

Open
HotKeyIt wants to merge 1 commit into
fancycode:masterfrom
HotKeyIt:master
Open

Fix SectionAlignment#94
HotKeyIt wants to merge 1 commit into
fancycode:masterfrom
HotKeyIt:master

Conversation

@HotKeyIt

@HotKeyIt HotKeyIt commented Mar 15, 2019 •
edited
Loading

Copy link
Copy Markdown

Some dlls do not use Native SectionAlignment, instead use SectionAlignment from PE.
Issue 54 should be solved then: #54
@HotKeyIt
Fix SectionAlignment
fd6ae57 
Some dlls do not use Native SectionAlignment, instead use SectionAlignment from PE.
@Elmue

Elmue commented Jul 2, 2020

Copy link
Copy Markdown

I have never seen such a DLL.
But your fix is correct.
@FuPeiJiang

FuPeiJiang commented Apr 13, 2024

Copy link
Copy Markdown

does this work without changing
result->pageSize = sysInfo.dwPageSize; to result->pageSize = old_header->OptionalHeader.SectionAlignment ?

module->pageSize is referenced multiple times,
but now it's undefined because the call to GetNativeSystemInfo(&sysInfo); has been removed,
or is SYSTEM_INFO sysInfo; initialized to zero ? I'm not sure
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants

@HotKeyIt @Elmue @FuPeiJiang