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

Names from typedefs #52

Merged
merged 2 commits into from
Aug 21, 2023
Merged

Names from typedefs #52

merged 2 commits into from
Aug 21, 2023

Conversation

mkonshie
Copy link
Collaborator

Resolves volatilityfoundation/volatility3#985

Prior to this commit, dwarf2json ignores typedefs and generates
unnamed user_types for anonymous structs. This makes it difficult
to refer to those types.

This PR uses a typedef's name to refer to a user_type when
dwarf2json encounters an anonymous struct instead of using a generated
name.

Melka Konshie added 2 commits August 17, 2023 13:34
In C, an anonymous struct can be named using a typedef. Prior to this
commit, dwarf2json ignores typedefs and generates unnamed user_types for
anonymous structs. This makes it difficult to refer to those types.

This commit uses a typedef's name to refer to a user_type when
dwarf2json encounters an anonymous struct instead of using a generated
name.
@gcmoreira
Copy link

gcmoreira commented Aug 19, 2023

@mkonshie this works for me. Tested using Fedora Core 38 with kernel 6.3.12 as it was reported in volatilityfoundation/volatility3#985 .. thanks

@mkonshie mkonshie merged commit 15baf31 into master Aug 21, 2023
1 check passed
@mkonshie mkonshie deleted the names-from-typedefs branch August 21, 2023 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Capabilities plugin breaks linux support for kernels without kernel_cap_struct
2 participants