-
Notifications
You must be signed in to change notification settings - Fork 164
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
update rule format documentation with dynamic details #851
Conversation
@yelhamer please take a peek |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just a few minor suggestions, thanks!
doc/format.md
Outdated
|
||
In general, capa collects and merges the features from lower scopes into higher scopes; | ||
for example, features extracted from individual instructions are merged into the function scope that contains the instructions. | ||
This way, you can use the match results against instructions ("the constant X is for crypto algorithm Y") to recognize function-level capabilities ("crypto function Z"). | ||
|
||
| feature | static scope | dynamic scope | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we better indicate that this is the lowest available scope? e.g. in the text or in the headings?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Co-authored-by: Moritz <[email protected]>
Co-authored-by: Moritz <[email protected]>
Co-authored-by: Moritz <[email protected]>
Co-authored-by: Moritz <[email protected]>
Co-authored-by: Moritz <[email protected]>
Co-authored-by: Moritz <[email protected]>
Co-authored-by: Moritz <[email protected]>
Co-authored-by: Moritz <[email protected]>
Co-authored-by: Moritz <[email protected]>
Co-authored-by: Moritz <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
awesome, thanks! please squash when merging
good reminder, thanks! |
documentation updates for mandiant/capa#1697