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 more information to the PROCESS_BASIC_INFORMATION struct #454

Closed
wants to merge 1 commit into from
Closed

Add more information to the PROCESS_BASIC_INFORMATION struct #454

wants to merge 1 commit into from

Conversation

FYN-Michiel
Copy link

@FYN-Michiel FYN-Michiel commented Apr 19, 2024

Not sure if this is the correct way to structure this but the extra information provided is very useful

@dahall
Copy link
Owner

dahall commented Apr 20, 2024

This structure is a port from the C structure defined by the API. Changing the field types will break the APIs with which it is associated.

@dahall
Copy link
Owner

dahall commented Apr 20, 2024

What are you trying to accomplish? There may be another way.

@FYN-Michiel
Copy link
Author

What are you trying to accomplish? There may be another way.

At the very least expose the InheritedFromUniqueProcessId value, we already use that currently.
All variables are listed on the Microsoft documentation page.

dahall added a commit that referenced this pull request Apr 23, 2024
…king field names and types, added properties to align with docs for GetProcessInformation
@dahall
Copy link
Owner

dahall commented Apr 23, 2024

Resolved with properties added to structure. Released in 4.0.1.

@dahall dahall closed this Apr 23, 2024
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.

2 participants