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

Added new Fortinet Fortigate rules #5197

Draft
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

inthecyber
Copy link

@inthecyber inthecyber commented Feb 20, 2025

Summary of the Pull Request

Adds several new rules related to Fortinet Fortigate Firewall Rules related to https://www.fortiguard.com/psirt/FG-IR-24-535

Changelog

new: Potential Persistence via New Local User Creation - Fortinet Fortigate Firewall
new:Potential Persistence via New Admin User Creation - Fortinet Fortigate Firewall
new:Potential Persistence via User Group Modification - Fortinet Fortigate Firewall
new:Potential Defense Evasion via Addition of Firewall Address Object - Fortinet Fortigate Firewall
new:Potential Remote Services Abuse via Addition of VPN SSL Web Portal - Fortinet Fortigate Firewall
new:Potential Remote Services Abuse via Modification of VPN SSL Settings - Fortinet Fortigate Firewall
new:Potential Defense Evasion via Firewall Policy Addition - Fortinet Fortigate Firewall

Example Log Event

new:Potential Persistence via New Local User Creation - Fortinet Fortigate Firewall
<190>date=2025-02-20 time=09:26:15 devname="<REDACTED>" devid="<REDACTED>" eventtime=1740039975614810031 tz="+0100" logid="0100044547" type="event" subtype="system" level="information" vd="root" logdesc="Object attribute configured" user="admin" ui="GUI(<REDACTED>)" action="Add" cfgtid=9503032 cfgpath="user.local" cfgobj="userlocal-test" cfgattr="type[password]passwd[*]" msg="Add user.local userlocal-test"
immagine

new:Potential Persistence via New Admin User Creation - Fortinet Fortigate Firewall
<190>date=2025-02-20 time=09:23:57 devname="<REDACTED>" devid="<REDACTED>" eventtime=1740039837394811711 tz="+0100" logid="0100044547" type="event" subtype="system" level="information" vd="root" logdesc="Object attribute configured" user="admin" ui="GUI(<REDACTED>)" action="Add" cfgtid=9503031 cfgpath="system.admin" cfgobj="admin_test" cfgattr="accprofile[prof_admin]vdom[root]gui-global-menu-favorites[]gui-vdom-menu-favorites[]password[*]" msg="Add system.admin admin_test"
immagine

new:Potential Persistence via User Group Modification - Fortinet Fortigate Firewall
<190>date=2025-02-20 time=10:09:30 devname="<REDACTED>" devid="<REDACTED>" eventtime=1740042570164823711 tz="+0100" logid="0100044547" type="event" subtype="system" level="information" vd="root" logdesc="Object attribute configured" user="admin" ui="GUI(<REDACTED>)" action="Edit" cfgtid=9503050 cfgpath="user.group" cfgobj="group-test" cfgattr="member[userlocal-test->userlocal-test user-test2]" msg="Edit user.group group-test"
immagine

new:Potential Defense Evasion via Addition of Firewall Address Object - Fortinet Fortigate Firewall
<190>date=2025-02-20 time=09:34:17 devname="<REDACTED>" devid="<REDACTED>" eventtime=1740040456994829291 tz="+0100" logid="0100044547" type="event" subtype="system" level="information" vd="root" logdesc="Object attribute configured" user="admin" ui="GUI(<REDACTED>)" action="Add" cfgtid=9503035 uuid="77a21396-ef65-51ef-8203-730f3c7d7e7d" cfgpath="firewall.address" cfgobj="192_all_iZ" cfgattr="subnet[192.168.0.0 255.255.0.0]" msg="Add firewall.address 192_all_iZ"
immagine

new:Potential Remote Services Abuse via Addition of VPN SSL Web Portal - Fortinet Fortigate Firewall
<190>date=2025-02-20 time=09:35:05 devname="<REDACTED>" devid="<REDACTED>" eventtime=1740040505314840911 tz="+0100" logid="0100044547" type="event" subtype="system" level="information" vd="root" logdesc="Object attribute configured" user="admin" ui="GUI(<REDACTED>)" action="Add" cfgtid=9503036 cfgpath="vpn.ssl.web.portal" cfgobj="tunnel-test" cfgattr="tunnel-mode[enable]web-mode[enable]ip-pools[SSLVPN_TUNNEL_SUBNET1]split-tunneling-routing-address[10_all_iZ 172_all_iZ 192_all_iZ]bookmark-group:gui-bookmarks[]" msg="Add vpn.ssl.web.portal tunnel-test"
immagine

new:Potential Remote Services Abuse via Modification of VPN SSL Settings - Fortinet Fortigate Firewall
<190>date=2025-02-20 time=09:35:24 devname="<REDACTED>" devid="<REDACTED>" eventtime=1740040524444853331 tz="+0100" logid="0100044546" type="event" subtype="system" level="information" vd="root" logdesc="Attribute configured" user="admin" ui="GUI(<REDACTED>)" action="Edit" cfgtid=9503046 cfgpath="vpn.ssl.settings" cfgattr="authentication-rule:9[source-interface[]users[userlocal-test]groups[]portal[tunnel-test]]" msg="Edit vpn.ssl.settings "
immagine

new:Potential Defense Evasion via Firewall Policy Addition - Fortinet Fortigate Firewall
<190>date=2025-02-20 time=09:40:59 devname="<REDACTED>" devid="<REDACTED>" eventtime=1740040859104834851 tz="+0100" logid="0100044547" type="event" subtype="system" level="information" vd="root" logdesc="Object attribute configured" user="admin" ui="GUI(<REDACTED>)" action="Add" cfgtid=9503047 uuid="67496868-ef66-51ef-4a52-f5fae30a4e10" cfgpath="firewall.policy" cfgobj="132" cfgattr="name[Policy VPN Test]srcintf[ssl.root]dstintf[<REDACTED>]action[accept]srcaddr[SSLVPN_TUNNEL_SUBNET1]dstaddr[10_all_iZ 172_all_iZ 192_all_iZ]srcaddr6[]dstaddr6[]src-vendor-mac[]schedule[always]service[ALL]groups[]users[userlocal-test]fsso-groups[]custom-log-fields[]" msg="Add firewall.policy 132"

immagine

Fixed Issues

SigmaHQ Rule Creation Conventions

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

Important:
I think the sigma taxonomy should be updated by including "product: fortigate" under "category: firewall", now I used category: firewall but the rules would work only on Elasticsearch, if you think the rules logic is ok we can work together on defining the new taxonomy

@github-actions github-actions bot added the Rules label Feb 20, 2025
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Welcome @inthecyber 👋

It looks like this is your first pull request on the Sigma rules repository!

Please make sure to read the SigmaHQ conventions document to make sure your contribution is adhering to best practices and has all the necessary elements in place for a successful approval.

Thanks again, and welcome to the Sigma community! 😃

@frack113 frack113 added Author Input Required changes the require information from original author of the rules Work In Progress Some changes are needed and removed Author Input Required changes the require information from original author of the rules labels Feb 21, 2025
@inthecyber inthecyber requested a review from frack113 February 24, 2025 13:48
@frack113 frack113 marked this pull request as draft February 25, 2025 05:44
@inthecyber
Copy link
Author

I think the problem is in the ATT&CK version used, the subtechnique T1098.007 is available only since v16 (in v15 is missing)

@frack113 frack113 removed the Work In Progress Some changes are needed label Feb 25, 2025
@nasbench nasbench added the 2nd Review Needed PR need a second approval label Feb 25, 2025
@nasbench nasbench self-requested a review February 25, 2025 21:52
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.

3 participants