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 about 2 months ago. Updated 19 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 1 (0 open — 1 closed)

Bug #9156: smtp: max-tx applies to cumulative tx count, not concurrent (8.0.x backport)ClosedPhilippe AntoineActions

VJ Updated by Victor Julien about 2 months ago Actions #1

  • Description updated (diff)

PA Updated by Philippe Antoine about 1 month 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 about 1 month 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 30 days ago Actions #4

  • Label Needs backport to 8.0 added

OT Updated by OISF Ticketbot 30 days ago Actions #5

  • Subtask #9156 added

OT Updated by OISF Ticketbot 30 days ago Actions #6

  • Label deleted (Needs backport to 8.0)

JI Updated by Jason Ish 29 days ago Actions #7

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

JI Updated by Jason Ish 29 days ago Actions #8

  • Private changed from Yes to No

PA Updated by Philippe Antoine 26 days ago Actions #9

  • Status changed from In Review to Resolved

PA Updated by Philippe Antoine 25 days ago Actions #10

  • Status changed from Resolved to Closed
Actions

Also available in: PDF Atom