Project

General

Profile

Actions

Optimization #4318

open

app-layer: "close" all txs if protocol reaches error state

Added by Shivani Bhardwaj over 3 years ago. Updated almost 2 years ago.

Status:
Assigned
Priority:
Low
Assignee:
Target version:
Effort:
high
Difficulty:
high
Label:

Description

Currently, in case there was a mis-detection of protocol or something led to incomplete data error in a protocol parser, the cleanup is not done properly.
Victor says "we should have logic that "closes" all txs (marks them as done), or handle this in the lower layers"
This came up when DCE/RPC over UDP packet was misdetected as Wireguard protocol and took too much of processing power.

Actions #1

Updated by Victor Julien over 3 years ago

  • Subject changed from Logic that "closes" all txs to app-layer: "close" all txs if protocol reaches error state
Actions #2

Updated by Victor Julien about 3 years ago

  • Assignee changed from Victor Julien to Philippe Antoine
Actions #3

Updated by Philippe Antoine about 3 years ago

  • Status changed from Assigned to In Review
Actions #4

Updated by Philippe Antoine almost 3 years ago

When an error is returned by an app-layer parser over UDP, nothing happens (the return value of AppLayerHandleUdp is ignored)
There is no equivalent of STREAMTCP_FLAG_APP_LAYER_DISABLED
Should we do something ?

Actions #5

Updated by Philippe Antoine about 2 years ago

  • Priority changed from Normal to Low
  • Effort set to high
  • Difficulty set to high
Actions #6

Updated by Philippe Antoine about 2 years ago

  • Target version changed from 7.0.0-beta1 to TBD
Actions #7

Updated by Philippe Antoine about 2 years ago

  • Status changed from In Review to Assigned
Actions #8

Updated by Philippe Antoine almost 2 years ago

  • Assignee changed from Philippe Antoine to OISF Dev

Not sure what the best way to deal with this is, and this seems high effort and low priority to get it right...

Actions

Also available in: Atom PDF