-
Notifications
You must be signed in to change notification settings - Fork 163
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
Thread Creation category #10
base: main
Are you sure you want to change the base?
Conversation
fe69298
to
fa24790
Compare
Thanks @jdu2600, could you please provide some examples from the Elastic EDR events? |
CreateThread = Here's a sample event -
|
Thanks for that @jdu2600. I did have a look and I can confirm that, unlike what the article suggests, other EDR are also monitoring for this activity. Although it would be a good practise to track all different variations of thread activities, this can get out of hand very quickly. We are planning to include one more level to the current sub-categories. We could change the "Remote Thread Creation" to "Thread Activities" and include all variations. But that is just too much for this stage of the project. We can re-evaluate this then. Thanks again for submitting the information and contributing to this project, appreciate it! |
Noted. Though perhaps the current category should be updated to |
Thanks for understanding. I re-opened this PR to consider the change for the sub-category name from "Remote Thread Creation" to "Thread Creation". @jdu2600 - Could you please provide a justification as to why you would like Sysmon to have an "Partially Implemented" mark instead of a "Implemented"? This justification would have to make its way to the official notes for the main table. @inodee - Any objections here for this change to the sub-category? Makes sense? |
The sysmon documentation indicates that its Event ID 8 is Remote Thread Creation only - not Thread Creation more generally. |
Good enough for me. For the record, difference explained here:
@jdu2600, could you please edit your proposed changes by only renaming the Sub-Category field to "Thread Creation" and change only the Sysmon value from "Yes" to "Partially"? Thanks! |
fa24790
to
1c269f6
Compare
Done. Are we confident that Crowdstrike/LimaCharlie/MDE/S1/WatchGuard all monitor local thread creations? |
The simplest test would be to see if telemetry is generated for a local unbacked thread. C++ snippet -
|
It seems like we indeed need a dedicated cat for "Thread Activities"! Thanks @jdu2600 for your contribs! Yes, we need to validate/adjust the others. |
Or a python one-liner if that is simpler. |
Description
Thread Creation events (ideally via a
PsSetCreateThreadNotifyRoutine
callback) are a useful telemetry source.References -
https://bruteratel.com/release/2022/11/17/Release-Resurgence/
"Several changes were also made to how a local thread was created following some detections from Elastic EDR, as unlike any other EDR, Elastic also monitors local threads."
Type of change