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

Fixed issue https://github.com/avast/retdec/issues/921 #922

Merged
merged 1 commit into from
Feb 11, 2021

Conversation

ladislav-zezula
Copy link
Contributor

…s loader to return ERROR_NONE and loaderError to LDR_ERROR_NTHEADER_OUT_OF_FILE

…s loader to return ERROR_NONE and loaderError to LDR_ERROR_NTHEADER_OUT_OF_FILE
Copy link
Member

@s3rvac s3rvac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the fix 👍 The fix looks good to me. All tests are passing, so I am going to merge the PR.

@s3rvac s3rvac assigned s3rvac and unassigned metthal Feb 11, 2021
@s3rvac s3rvac removed the request for review from metthal February 11, 2021 16:20
@s3rvac s3rvac merged commit 2d903ef into master Feb 11, 2021
@s3rvac s3rvac deleted the LZ_Issue_921 branch February 11, 2021 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants