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

Use KEY_WOW64_64KEY to access MachineGuid #7294

Merged
merged 1 commit into from
Jun 12, 2018

Conversation

andrewkroh
Copy link
Member

KEY_WOW64_64KEY on a 64-bit OS means that the registry access, no matter if it's a 32 or 64 bit process, will access the 64 bit registry view.

This fixes an issue where the Metricbeat windows-service metricset would not start if the 32-bit binary was used on a 64-bit machine. The flag will have no effect on 32-bit machines.

KEY_WOW64_64KEY on a 64-bit OS means that the registry access, no matter if it's a 32 or 64 bit process, will access the 64 bit registry view.

This fixes an issue where the Metricbeat windows-service metricset would not start if the 32-bit binary was used on a 64-bit machine. The flag will have no effect on 32-bit machines.
@andrewkroh andrewkroh force-pushed the bugfix/mb/win-svc-registry branch from 92bc25a to 8feb8f5 Compare June 12, 2018 01:45
@ruflin ruflin merged commit 434c567 into elastic:master Jun 12, 2018
@ruflin ruflin deleted the bugfix/mb/win-svc-registry branch June 12, 2018 09:05
@ruflin
Copy link
Contributor

ruflin commented Jun 12, 2018

Worth a backport?

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

Successfully merging this pull request may close these issues.

2 participants