Skip to content

Fix powerview addnetuser#814

Merged
Cx01N merged 3 commits intoBC-SECURITY:mainfrom
asantoma:fix-powerview-addnetuser
Apr 19, 2026
Merged

Fix powerview addnetuser#814
Cx01N merged 3 commits intoBC-SECURITY:mainfrom
asantoma:fix-powerview-addnetuser

Conversation

@asantoma
Copy link
Copy Markdown

Describe your changes

I've added the missing Add-NetUser function to the powerview PowerShell script, it seems to have gotten removed in 12138db. I've also updated the add_netuser.yaml file by removing the documentation stating an Empire stager is executed at logon for the newly created user (the Add-NetUser function doesn't do this). I've removed the Listener parameter because it caused the function to crash since it's not in its parameter list and modified needs_admin to be true since the function requires administrative privileges to successfully execute.

I've manually tested the modified module and it is working.

Issue ticket number and link (if there is one)

Checklist before requesting a review

  • I have performed a self-review of my code
  • If it is a core feature, I have added thorough tests.
  • I have added an entry to CHANGELOG.md
  • I have updated the documentation in docs/ (if applicable)

@vinnybod vinnybod requested a review from Cx01N March 3, 2026 05:41
Copy link
Copy Markdown
Member

@Cx01N Cx01N left a comment

Choose a reason for hiding this comment

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

Looks good, just needs the format/lint ran

@Cx01N Cx01N merged commit 9952055 into BC-SECURITY:main Apr 19, 2026
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