Project

General

Profile

Actions

Bug #4071

closed
PA PA

Null dereference in ipv4hdr GetData

Bug #4071: Null dereference in ipv4hdr GetData

Added by Philippe Antoine over 5 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
High
Target version:
Affected Versions:
Effort:
Difficulty:
Label:
Needs backport to 5.0

Description

Found by oss-fuzz
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=24288

Reproducer is
./src/suricata -r fail.pcap -S ip4hdr.rule
with ip4hdr.rule being a rule using ipv4.hdr alert ip any any -> any any (ipv4.hdr; content:"|00 00|"; offset:4; depth:2; sid:1234;)

The bug seems to be present for tcphdr as well


Files

fail.pcap (138 Bytes) fail.pcap Philippe Antoine, 10/14/2020 07:41 PM
tcp0.pcap (250 Bytes) tcp0.pcap Philippe Antoine, 10/14/2020 07:55 PM

Related issues 1 (0 open1 closed)

Copied to Suricata - Bug #4132: Null dereference in ipv4hdr GetDataClosedJeff LucovskyActions

PA Updated by Philippe Antoine over 5 years ago Actions #1

Pcap reproducer for tcp.hdr rule

PA Updated by Philippe Antoine over 5 years ago Actions #2

  • Status changed from Assigned to In Review

Gitlab

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

The fact that oss-fuzz produced unreproducible crashes seems to tell that DetectEngineReload does not reset everything.
I guess it does not somehow reset the call to DetectBufferSetActiveList(s, g_ipv4hdr_buffer_id)

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

  • Priority changed from Normal to High
  • Label Needs backport to 5.0 added

PA Updated by Philippe Antoine over 5 years ago Actions #5

Should I create private S-V tests for these ?

JL Updated by Jeff Lucovsky over 5 years ago Actions #6

  • Copied to Bug #4132: Null dereference in ipv4hdr GetData added

VJ Updated by Victor Julien over 5 years ago Actions #7

  • Status changed from In Review to Closed

JL Updated by Jeff Lucovsky over 5 years ago Actions #8

  • Private changed from Yes to No
Actions

Also available in: PDF Atom