-
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 and add Cabinet archive related rules #808
Conversation
@mike-hunhoff, can you take a look? |
@mike-hunhoff bump :) |
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.
Thank you @jtothej ! I'm not sure why I missed all of the mentions here and I apologize for the delayed review. Please check out my comments.
@@ -0,0 +1,23 @@ | |||
rule: | |||
meta: | |||
name: create Cabinet file |
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.
Thoughts on changing this to create Cabinet on Windows
to make it more clear?
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.
Thanks! Renamed the rule in ac09516
- or: | ||
- api: cabinet.FCIAddFile = add file to Cabinet | ||
- api: cabinet.FCIFlushFolder = flush current folder under construction | ||
- api: cabinet.FCIFlushCabinet = completes current cabinet |
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.
Add cabinet.FCIDestroy
?
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.
Thanks! Added in ac09516
@@ -0,0 +1,21 @@ | |||
rule: | |||
meta: | |||
name: extract files from Cabinet |
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.
Thoughts on changing this to extract Cabinet on Windows
to make it more clear?
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.
Thanks! Renamed the rule in ac09516
@@ -1,12 +1,14 @@ | |||
rule: | |||
meta: | |||
name: open cabinet file | |||
namespace: host-interaction/file-system | |||
name: create File Compression Interface context |
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.
Thoughts on adding on Windows
to make it more clear?
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.
Thanks! Renamed the rule in ac09516
@@ -0,0 +1,14 @@ | |||
rule: | |||
meta: | |||
name: create File Decompression Interface context |
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.
Thoughts on adding on Windows
to make it more clear?
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.
Thanks! Renamed the rule in ac09516
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.
thanks, I think these look good now
Suggesting following rearrangement as FCI/FDI API functions are dependent on each other so I'm not sure if it's worth to have separate rules for each of them:
FCICreate
andFDICreate
can be potentially used for shellcode execution via callback functions (TODO) so adding them as library functions:lib/create-file-compression-interface-context.yml
lib/create-file-decompression-interface-context.yml
Rules indicating creation or extraction of data from Cabinet file:
data-manipulation/compression/create-cabinet-file.yml
data-manipulation/compression/extract-files-from-cabinet.yml
CC: @mike-hunhoff