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

Fix Mint Sandstorm - Log4J Wstomcat Process Execution Condition #4603

Merged
merged 2 commits into from
Nov 29, 2023
Merged

Fix Mint Sandstorm - Log4J Wstomcat Process Execution Condition #4603

merged 2 commits into from
Nov 29, 2023

Conversation

0x616c6578
Copy link
Contributor

@0x616c6578 0x616c6578 commented Nov 29, 2023

Fixed detection condition in c97c625-0350-4f0a-8943-f6cadc88125e

The MSTIC reference provides the following KQL query for Log4J Wstomcat Process Execution:

DeviceProcessEvents
| where InitiatingProcessFileName has "ws_tomcatservice.exe" and FileName !in~("repadmin.exe")

The current version of c97c625-0350-4f0a-8943-f6cadc88125e treats the !in~ operator as an in~ operator, causing the rule to trigger on false positive events:

detection:
    selection:
        ParentImage|endswith: '\ws_tomcatservice.exe'
        Image|endswith: '\repadmin.exe'
    condition: selection

This PR changes it to the following:

detection:
    selection:
        ParentImage|endswith: '\ws_tomcatservice.exe'
    filter:
        Image|endswith: '\repadmin.exe'
    condition: selection and not filter

Changelog

fix: Mint Sandstorm - Log4J Wstomcat Process Execution - Add missing filter

Example Log Event

N/A

Fixed Issues

N/A

SigmaHQ Rule Creation Conventions

  • If your PR adds new rules, please consider following and applying these conventions

@nasbench nasbench changed the title Update proc_creation_win_apt_mint_sandstorm_log4j_wstomcat_execution.yml Fix Mint Sandstorm - Log4J Wstomcat Process Execution Condition Nov 29, 2023
@nasbench nasbench merged commit 5638f89 into SigmaHQ:master Nov 29, 2023
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants