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

Add 4 miscellaneous persistence techniques #956

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

jorik-utwente
Copy link

Add 4 miscellaneous persistence techniques.

The BITS Jobs and WMI subscription rules may result in false positives because of the ole32.CoCreateInstance matching.
We might want to remove that part, but that would result in false negatives. Thoughts on this are welcome! :)

This PR requires #952 to be merged.

@jorik-utwente jorik-utwente changed the title Add 4 miscellaneous persistence techniques. Add 4 miscellaneous persistence techniques Nov 5, 2024
Copy link
Collaborator

@mr-tz mr-tz left a comment

Choose a reason for hiding this comment

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

very cool, see my initial feedback attached

nursery/persist-via-bits-job.yml Outdated Show resolved Hide resolved
nursery/persist-via-application-shimming.yml Show resolved Hide resolved
nursery/persist-via-application-shimming.yml Outdated Show resolved Hide resolved
nursery/persist-via-application-shimming.yml Show resolved Hide resolved
nursery/persist-via-application-shimming.yml Outdated Show resolved Hide resolved
nursery/persist-via-bits-job.yml Outdated Show resolved Hide resolved
nursery/persist-via-wmi-event-subscription.yml Outdated Show resolved Hide resolved
Comment on lines +20 to +25
- and:
- or:
- match: copy file
- match: move file
- match: host-interaction/file-system/write
- string: /\\spool\\PRTPROCS\\/i
Copy link
Collaborator

Choose a reason for hiding this comment

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

with the dynamic: call scope this would only work for static processing, is that intended?

Copy link
Author

@jorik-utwente jorik-utwente Nov 15, 2024

Choose a reason for hiding this comment

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

It is intended to work for dynamic as well.
When #952 is merged, this should work for dynamic as well (as it changes the scope of copy file, move file, and write file to the call scope).
Or is there another reason why it wouldn't work for dynamic analysis?

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