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 windows UBR to os_version table #8265

Merged
merged 3 commits into from
Feb 29, 2024
Merged

Add windows UBR to os_version table #8265

merged 3 commits into from
Feb 29, 2024

Conversation

pradishmp
Copy link
Contributor

Fixes #7908 and #7663
The purpose of this PR is to include the build revision to the build number on windows.

Rationale for this change:
On executing the query “select build from os_version;” the build does not display the full build number including what is called the updated build revision (UBR) , the implementation of os_version uses the Win32_OperatingSystem WMI class to get all the build information, but it does not have any property that returns the UBR.

How is this fixed:
Get this information from windows registry and concatenate it to the build value.

Below screen shot shows, how this would look post fix
issue_7663

…ncorrect build and version from os_version

not as expected on Windows (osquery#7663)
@pradishmp pradishmp requested review from a team as code owners February 5, 2024 10:13
Copy link

linux-foundation-easycla bot commented Feb 5, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

@directionless directionless added this to the 5.12.0 milestone Feb 6, 2024
@directionless directionless changed the title os_version table missing full OS build number as expected on windows (#7908 and #7663) Add windows UBR to os_version table Feb 6, 2024
Copy link
Member

@zwass zwass left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @pradishmp!

@zwass zwass merged commit 8bd7359 into osquery:master Feb 29, 2024
16 checks passed
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.

Table os_version is missing full OS build number
3 participants