Project

General

Profile

Actions
Copy link

Bug #7019

closed
PA PA

snmp: probing parser returns ALPROTO_FAILED instead of ALPROTO_UNKNOWN if slice.len() < 4

Bug #7019: snmp: probing parser returns ALPROTO_FAILED instead of ALPROTO_UNKNOWN if slice.len() < 4

Added by Philippe Antoine almost 2 years ago. Updated 11 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Philippe Antoine
Target version:
8.0.0-rc1
Affected Versions:
Effort:
Difficulty:
Label:
Beginner, Good First Issue, Needs Suricata-Verify test, Protocol, Rust

Description

Found with https://github.com/OISF/suricata/pull/11062

This would allow protocol detection evasion on TCP by splitting the PDU into a first small slice and the rest once the first packet is packed

There may be other protocols to check.

Subtasks 1 (0 open — 1 closed)

Bug #7716: snmp: probing parser returns ALPROTO_FAILED instead of ALPROTO_UNKNOWN if slice.len() < 4 (7.0.x backport)ClosedPhilippe AntoineActions

PA Updated by Philippe Antoine almost 2 years ago Actions
Copy link
 #1

  • Subject changed from snmp: robin parser returns ALPROTO_FAILED instead of ALPROTO_UNKNOWN if slice.len() < 4 to snmp: probing parser returns ALPROTO_FAILED instead of ALPROTO_UNKNOWN if slice.len() < 4

PA Updated by Philippe Antoine almost 2 years ago Actions
Copy link
 #2

  • Label Beginner, Good First Issue, Needs Suricata-Verify test, Protocol, Rust added

Easy fix, hard thing is to craft a pcap for testing

PA Updated by Philippe Antoine almost 2 years ago Actions
Copy link
 #3

I think this one can be postponed after 8

VJ Updated by Victor Julien about 1 year ago Actions
Copy link
 #4

  • Target version changed from 8.0.0-beta1 to 8.0.0-rc1

PA Updated by Philippe Antoine 11 months ago Actions
Copy link
 #5

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

PA Updated by Philippe Antoine 11 months ago Actions
Copy link
 #6

  • Status changed from In Review to Resolved
  • Label Needs backport to 7.0 added

OT Updated by OISF Ticketbot 11 months ago Actions
Copy link
 #7

  • Subtask #7716 added

OT Updated by OISF Ticketbot 11 months ago Actions
Copy link
 #8

  • Label deleted (Needs backport to 7.0)

PA Updated by Philippe Antoine 11 months ago Actions
Copy link
 #9

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: PDF Atom