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

Add support for limiting the overall memory of our unpacker (#290) #312

Merged
merged 3 commits into from
May 28, 2018

Conversation

s3rvac
Copy link
Member

@s3rvac s3rvac commented May 26, 2018

This PR adds support for limiting the overall memory of our unpacker (see #290).

Regression tests for this feature were added in avast/retdec-regression-tests#7 (merge the present PR before merging the PR with tests).

s3rvac added 3 commits May 26, 2018 15:55
They can be used to limit the maximal memory to (1) the given number of bytes
or (2) half of system RAM.
…-max-memory-half-ram.

They are just forwarded to retdec-unpacker. They can be used to limit the
maximal memory to (1) the given number of bytes or (2) half of system RAM.
…h from retdec-decompiler.sh.

Now that retdec-unpacker.sh supports limiting of memory, we should take
advantage of that. Previously, the above parameters were passed only to
fileinfo, bin2llvmir, and llvmir2hll.
@s3rvac s3rvac self-assigned this May 26, 2018
@s3rvac s3rvac requested a review from metthal May 26, 2018 14:17
@metthal metthal merged commit 03aa388 into master May 28, 2018
@metthal metthal deleted the feature-unpacker-limit-overall-memory branch May 28, 2018 08:15
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.

2 participants