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

New Atomic - Excel Application Object for LM #2582

Merged
merged 12 commits into from
Nov 8, 2023

Conversation

tsale
Copy link
Contributor

@tsale tsale commented Nov 1, 2023

Details:
This lateral movement technique is within the Distributed Component Object Model (DCOM) of the Excel application, specifically utilizing the ActivateMicrosoftApp() method. It is based on initial research by Matt Nelson regarding lateral movement using Excel.Application and DCOM. The steps to reproduce including the PowerShell commands are highlighted on the referenced link from specterops blog post.

Testing:
This technique was tested in local and remote hosts. It was also tested on hosts that do not have the Excel application installed for error validation.

Associated Issues:
No issues found.

@josehelps josehelps self-assigned this Nov 1, 2023
@josehelps josehelps self-requested a review November 1, 2023 23:36
Copy link
Collaborator

@josehelps josehelps left a comment

Choose a reason for hiding this comment

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

Hey, there are some minor changes that need to be made but this is an amazing addition thank you!

Fixing yaml error
@tsale
Copy link
Contributor Author

tsale commented Nov 2, 2023

Thanks @josehelps. I fixed the yaml issue.

@josehelps
Copy link
Collaborator

Hey @tsale I removed the README.md and also the auto_generated_guid from the yaml since those things are auto generated but CI still failing because there is a hostname input argument that is not used in the execution command: https://github.com/redcanaryco/atomic-red-team/pull/2582/checks#step:6:16

Changed hostname field name to computer_name
Copy link
Contributor Author

@tsale tsale left a comment

Choose a reason for hiding this comment

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

Ah ok, I changed the field name from hostname to computer_name. Hopefully should be fine now.

@cyberbuff
Copy link
Collaborator

cyberbuff commented Nov 3, 2023

Ah ok, I changed the field name from hostname to computer_name. Hopefully should be fine now.

Hey @tsale ,

Looks like there is another issue. computer_name parameter is not used in the atomic. Can you change your command to

 $com = [System.Activator]::CreateInstance([type]::GetTypeFromProgID("Excel.Application","#{computer_name}"))

Copy link
Contributor Author

@tsale tsale left a comment

Choose a reason for hiding this comment

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

Done.

Loving the automation here🙂

@cyberbuff cyberbuff requested a review from josehelps November 6, 2023 22:43
Copy link
Collaborator

@josehelps josehelps left a comment

Choose a reason for hiding this comment

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

Awesome thank you!

Copy link
Collaborator

Choose a reason for hiding this comment

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

this markdown file is automatically generated from the yaml by the tooling in the project and will be clobbered would you mind removing it?

atomics/T1021.003/T1021.003.yaml Show resolved Hide resolved
@josehelps josehelps merged commit bf2251b into redcanaryco:master Nov 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants