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 GetUserNameExW to query user name on windows #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tehKaiN
Copy link

@tehKaiN tehKaiN commented Aug 27, 2022

Currently, RTTR uses GetUserNameW to query user name, which isn't that accurate. On my OS the situation is following:

obraz

The display name is as follows:

obraz

And RTTR takes the user profile folder name to create default user name:

obraz

Whereas when migrating to GetUserNameExW it uses correct "display name"

obraz

The function is supported on win2k and up, so I guess it's okay?

@tehKaiN tehKaiN force-pushed the better-win32-username branch from a8895b2 to 5da8548 Compare August 27, 2022 09:21
@Flamefire Flamefire requested a review from Flow86 August 27, 2022 12:45
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.

1 participant