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.

Sign up for GitHub

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

Jump to bottom

retdec-fileinfo runs for a minute and uses 110 GB of RAM to analyze a 41 MB PE file #907

Closed
s3rvac opened this issue Dec 13, 2020 · 1 comment · Fixed by #908
Closed

retdec-fileinfo runs for a minute and uses 110 GB of RAM to analyze a 41 MB PE file #907

s3rvac opened this issue Dec 13, 2020 · 1 comment · Fixed by #908
Assignees
@ladislav-zezula
Labels
bug C-fileinfo P-run

Comments

@s3rvac
Copy link
Member

s3rvac commented Dec 13, 2020 •
edited
Loading

retdec-fileinfo uses 110 GB of RAM to analyze a 41 MB PE file.

Steps to reproduce

On Linux, run

$ /usr/bin/time -v retdec-fileinfo FILE

where FILE is FE015EB24B7EEA2907698A6D7142198644A757066DA4EB8D3A4B63900008CF5E.

Expected output

The file is analyzed in a reasonable time and using a reasonable amount of memory.

Actual output

$ /usr/bin/time -v retdec-fileinfo FILE
[..]
        User time (seconds): 61.19
[..]
        Maximum resident set size (kbytes): 116301540

That is, retdec-fileinfo ran for over a minute and used around 110 GB of memory during the analysis.

Configuration

  • Commit: 37dbfd1 (current master)
  • 64b Debian Buster, GCC 9, Debug build of RetDec
@ladislav-zezula ladislav-zezula self-assigned this Dec 18, 2020
ladislav-zezula pushed a commit that referenced this issue Dec 18, 2020
Fixed #907
a5374a3
@s3rvac s3rvac mentioned this issue Dec 18, 2020
Merged
s3rvac added a commit that referenced this issue Dec 18, 2020
@s3rvac
Add a CHANGELOG entry for #907.
ec5cd88
@s3rvac s3rvac mentioned this issue Jan 6, 2021
Closed
@s3rvac
Copy link
Member Author

s3rvac commented Jan 6, 2021

Possibly related issue: #911.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ladislav-zezula ladislav-zezula

Labels
bug C-fileinfo P-run
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Lz issue 907 avast/retdec
2 participants
@ladislav-zezula @s3rvac