Project

General

Profile

Actions

Security #5243

closed
PA PA

protocol detection: exploitable type confusion due to concurrent protocol changes

Security #5243: protocol detection: exploitable type confusion due to concurrent protocol changes

Added by Philippe Antoine almost 4 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Normal
Target version:
Affected Versions:
Label:
CVE:
Git IDs:

cedffdf14cf1fdd4d551f16c331e5b3e7f0a6927

Severity:
HIGH
Disclosure Date:

Description

Found by oss-fuzz
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=45941&q=label%3AProj-suricata&can=2

If a STMP session has STARTTLS, it requests AppLayerRequestProtocolChange
We call TCPProtoDetect to rerun protocol detection
So far, so good.

But then, if the protocol is HTTP1 requesting an upgrade, HTTP/1.1 101\nUpgrade:h2c
we call again AppLayerRequestProtocolChange which erases values like alproto_orig which leads to type confusion...


Files

range.pcap (621 Bytes) range.pcap Philippe Antoine, 04/08/2022 12:32 PM

Related issues 2 (0 open2 closed)

Related to Suricata - Feature #5509: App-layer event for protocol change failureClosedPhilippe AntoineActions
Copied to Suricata - Security #5254: protocol detection: exploitable type confusion due to concurrent protocol changesClosedShivani BhardwajActions

PA Updated by Philippe Antoine almost 4 years ago Actions #1

  • Private changed from No to Yes

PA Updated by Philippe Antoine almost 4 years ago Actions #2

  • Status changed from New to In Review

Gitlab MR

JL Updated by Jeff Lucovsky almost 4 years ago Actions #3

  • Copied to Security #5254: protocol detection: exploitable type confusion due to concurrent protocol changes added

VJ Updated by Victor Julien almost 4 years ago Actions #5

  • Status changed from In Review to Closed

VJ Updated by Victor Julien almost 4 years ago Actions #6

  • Severity changed from MODERATE to HIGH

PA Updated by Philippe Antoine over 3 years ago Actions #7

  • Related to Feature #5509: App-layer event for protocol change failure added

VJ Updated by Victor Julien over 3 years ago Actions #8

  • Private changed from Yes to No
  • Label deleted (Needs backport, Needs backport to 6.0)
Actions

Also available in: PDF Atom