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

Copy link
Copy Markdown

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

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

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants