Project

General

Profile

Actions

Bug #9038

closed
VJ PA

smtp: max-tx applies to cumulative tx count, not concurrent

Bug #9038: smtp: max-tx applies to cumulative tx count, not concurrent

Added by Victor Julien 27 days ago. Updated 5 days ago.

Status:
Closed
Priority:
Normal
Target version:
Affected Versions:
Effort:
Difficulty:
Label:

Description

SMTP's max-tx documents a limit on live transactions per flow, but SMTPTransactionCreate() gates on tx_cnt, a monotonic high-water mark never decremented when the engine frees completed transactions. The cap thus bounds lifetime envelopes per connection: with max-tx=16, once 17 txs are created, the 18th envelope's creation fails, the parser errors with APP_LAYER_ERROR, and app-layer parsing is disabled for the whole flow.

The example also shows a off by one: max-tx 16 should allow 16 txs, not 17.

The security implication is that beyond max-tx logging and detection will fail as the parser hits an error state. So it's observable. In IPS mode the session by default will drop.


Subtasks

VJ Updated by Victor Julien 27 days ago Actions #1

  • Description updated (diff)

PA Updated by Philippe Antoine 12 days ago Actions #2

  • Status changed from New to Triaged
  • Assignee set to OISF Dev
  • Severity set to LOW

Proposing LOW severity as limited scope evasion

PA Updated by Philippe Antoine 12 days ago Actions #3

  • Status changed from Triaged to In Review
  • Assignee changed from OISF Dev to Philippe Antoine

Gitlab MR

PA Updated by Philippe Antoine 9 days ago Actions #4

  • Label Needs backport to 8.0 added

OT Updated by OISF Ticketbot 9 days ago Actions #5

  • Subtask #9156 added

OT Updated by OISF Ticketbot 9 days ago Actions #6

  • Label deleted (Needs backport to 8.0)

JI Updated by Jason Ish 8 days ago Actions #7

  • Tracker changed from Security to Bug
  • Severity deleted (LOW)

JI Updated by Jason Ish 8 days ago Actions #8

  • Private changed from Yes to No

PA Updated by Philippe Antoine 5 days ago Actions #9

  • Status changed from In Review to Resolved

PA Updated by Philippe Antoine 5 days ago Actions #10

  • Status changed from Resolved to Closed
Actions

Also available in: PDF Atom