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

Malfind broken #1400

Open
KQBTD opened this issue Dec 13, 2024 · 8 comments · May be fixed by #1407
Open

Malfind broken #1400

KQBTD opened this issue Dec 13, 2024 · 8 comments · May be fixed by #1407

Comments

@KQBTD
Copy link

KQBTD commented Dec 13, 2024

Describe the bug
Hi, I was using malfind plugin and it flagged this error.

Context
Volatility Version: volatility 3 2.8.0
Operating System: windows 11
Python Version: 3.13
Suspected Operating System:
Command: python vol.py -f memorydump.dmp windows.malfind > Output.txt

To Reproduce
Steps to reproduce the behavior:

  1. Use command
    A. pip install -r requirements.txt
    B. python vol.py -f memorydump.dmp windows.malfind > Output.txt

Error:
image

@eve-mem
Copy link
Contributor

eve-mem commented Dec 13, 2024

Thanks for the report. Any chance you could upload the full -vvv output as text for us to look into?

@atcuno
Copy link
Contributor

atcuno commented Dec 13, 2024

I am almost postive from the backtrace that this code is the issue:

https://github.com/volatilityfoundation/volatility3/blob/develop/volatility3/framework/symbols/windows/extensions/__init__.py#L271

There are a range of kernel versions where the CommitCharge member is stored as Core.CommitCharge which the code linked above does not handle.

@eve-mem do you have a moment to put a PR request handling the Core check to look for CommitCharge directly under it and then do the current way if its not there?

@eve-mem
Copy link
Contributor

eve-mem commented Dec 13, 2024

I can certainly have a look. It might be early next week rather than the weekend now. I don't have the biggest supply of windows machines to test on but hopefully one of them will be similar enough. Your explanation @atcuno is clear enough for me on what the issue is.

@KQBTD
Copy link
Author

KQBTD commented Dec 14, 2024

so will this issue get fixed next week?

@eve-mem
Copy link
Contributor

eve-mem commented Dec 14, 2024

If i can find a similar issue in a sample i have I'll fix it next week yes. Perhaps you are able to share your sample?

@KQBTD
Copy link
Author

KQBTD commented Dec 14, 2024

what do you mean by sample?

@eve-mem
Copy link
Contributor

eve-mem commented Dec 14, 2024

A memory dump that I can test with.

@eve-mem
Copy link
Contributor

eve-mem commented Dec 16, 2024

Hello @KQBTD - I couldn't find a memory dump to test with.

Are you able to check to see if the changes in #1407 fix your problem?

I'm not expecting it to work right away to be honest - if you still get an error could you please share you logs with me.

Thanks <3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants