Skip to content

last_bit_seen potential uninitialzed variable usage in crash.py #1438

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
Closed
atcuno opened this issue Dec 18, 2024 · 2 comments · May be fixed by #1449
Closed

last_bit_seen potential uninitialzed variable usage in crash.py #1438

atcuno opened this issue Dec 18, 2024 · 2 comments · May be fixed by #1449
Assignees
@atcuno
Labels
parity-release

Comments

@atcuno
Copy link
Contributor

atcuno commented Dec 18, 2024

last_bit_seen needs to be declared before the loop like the other related variables are:

https://github.com/volatilityfoundation/volatility3/blob/develop/volatility3/framework/layers/crash.py#L192C17-L192C30
gcmoreira added a commit to gcmoreira/volatility3 that referenced this issue Dec 19, 2024
@gcmoreira
windows: crashdump: fix potential uninitialzed variable volatilityfou…
c77ed91 
…ndation#1438
@gcmoreira gcmoreira mentioned this issue Dec 19, 2024
Open
@ikelos ikelos linked a pull request Jan 29, 2025 that will close this issue
Windows crashdump fix and refactor #1449
Open
@gcmoreira
Copy link
Contributor

@ikelos @atcuno this should have been fixed in #1449

@gcmoreira gcmoreira assigned atcuno and unassigned gcmoreira Jan 30, 2025
@ikelos
Copy link
Member

ikelos commented Jan 30, 2025

Yep, just waiting to get that one merged, thanks 5:)

@ikelos ikelos closed this as completed Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@atcuno atcuno

Labels
parity-release
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Windows crashdump fix and refactor gcmoreira/volatility3
3 participants
@atcuno @ikelos @gcmoreira