Skip to content

Properly updated DynamicHooks(libDynamicHooks.a).#343

Closed
CookStar wants to merge 1 commit into
Source-Python-Dev-Team:masterfrom
CookStar:fix_libDynamicHooks
Closed

Properly updated DynamicHooks(libDynamicHooks.a).#343
CookStar wants to merge 1 commit into
Source-Python-Dev-Team:masterfrom
CookStar:fix_libDynamicHooks

Conversation

@CookStar

@CookStar CookStar commented Jul 31, 2020

Copy link
Copy Markdown
Contributor

Fix to d05527f.

Since libDynamicHooks.a is not recompiled, Segmentation Fault will be occurred at StackData.return_address.

return m_pHook->m_RetAddr[pESP].back();

If it is difficult to accept binary file due to security reasons, please edit the pull request, or close the pull request and commit the correct binary.

@Ayuto

Ayuto commented Aug 12, 2020

Copy link
Copy Markdown
Member

Strange, I actually recompiled and copied the library. But apparently zero bytes have changed, so I must have grabbed the old version.

Thanks for reporting! Also thank you for providing the recompiled library! Though, I don't merge compiled binaries.

Nevertheless, I will keep this open as a reminder.

Ayuto added a commit that referenced this pull request Aug 14, 2020
@Ayuto Ayuto closed this Aug 14, 2020
@CookStar CookStar deleted the fix_libDynamicHooks branch August 14, 2020 17:45
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.

2 participants