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

[bug] Remove a dollar sign in UserAccountControlCommand.cs #127

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

Conversation

KinakoExE
Copy link

Hello,
I identified a tiny bug in Seatbelt.

In UserAccountControlCommand.cs, the result string is not printed correctly when dto.ConsentPromptBehaviorAdmin is 0 due to a dollar sign.
So I just removed it and now it looks good now.

before:

  • 0 : 1 - No prompting is a buggy format string.
====== UAC ======

  0                              : 1 - No prompting
  EnableLUA (Is UAC enabled?)    : 1
  LocalAccountTokenFilterPolicy  :
  FilterAdministratorToken       :
    [*] Default Windows settings - Only the RID-500 local admin account can be used for lateral movement.

after:

====== UAC ======

  ConsentPromptBehaviorAdmin     : 0 - No prompting
  EnableLUA (Is UAC enabled?)    : 1
  LocalAccountTokenFilterPolicy  :
  FilterAdministratorToken       :
    [*] Default Windows settings - Only the RID-500 local admin account can be used for lateral movement.

@KinakoExE KinakoExE changed the title Remove a dollar sign in UserAccountControlCommand.cs [bug] Remove a dollar sign in UserAccountControlCommand.cs Mar 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.

None yet

1 participant