Project

General

Profile

Actions

Feature #8394

closed
YD JI

Task #8388: firewall: support protocol hooks for all app-layer protocols

firewall: support NTP hook states for firewall rule evaluation

Feature #8394: firewall: support NTP hook states for firewall rule evaluation

Added by Yash Datre 2 months ago. Updated 6 days ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Effort:
Difficulty:
Label:

Description

NTP is a fundamental network protocol present in virtually all environments. In Suricata 8.0.4, NTP app-layer hook states are not registered for firewall mode. Attempting to use any ntp:* hook in a firewall rule fails with the error: "protocol ntp does not support hook" .

Without NTP hooks, NTP traffic on UDP port 123 can only be controlled at the packet layer. While packet-layer rules (accept:hook udp:all any any <> any 123) can accept NTP packets, the app-layer parser still engages and the flow may be subject to default_app_policy behavior. Registering NTP hooks would allow the firewall engine to make explicit accept/drop decisions at the application layer, consistent with how other protocols (DNS, TLS, HTTP1) are handled.

Potential NTP states:
  • Client request sent
  • Server response received
  • Synchronized
  • Kiss-o'-Death received
  • Unsynchronized
  • Symmetric active/passive
  • Broadcast mode

These states should be mapped to firewall hook points. At minimum, a request/response pair would enable basic firewall control. Additional hooks for Kiss-o'-Death and broadcast mode would allow more granular policy enforcement.


Subtasks 4 (0 open4 closed)

Feature #8429: rules: add ntp.mode keywordClosedJason IshActions
Feature #8430: rules: add ntp.version keywordClosedJason IshActions
Feature #8431: rules: add ntp.stratum keywordClosedJason IshActions
Feature #8562: firewall: support NTP hook states for firewall rule evaluation (8.0.x backport)ClosedJason IshActions

Related issues 3 (1 open2 closed)

Related to Suricata - Task #8478: firewall: reconsider built-in hooks for UDP protocolsNewActions
Related to Suricata - Feature #8425: ntp: add ntp transaction loggingClosedJason IshActions
Related to Suricata - Feature #8488: ntp: use a buffer for reference id, not a u32ClosedJason IshActions
Actions

Also available in: PDF Atom