Project

General

Profile

Actions

Bug #6038

closed
JL VJ

TCP resets have incorrect len, nh in IPv6

Bug #6038: TCP resets have incorrect len, nh in IPv6

Added by Jamie Lavigne almost 3 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Affected Versions:
Effort:
Difficulty:
Label:

Description

Suricata's TCP rejects over IPv6 appear to have an incorrect length field value, which can cause these packets to be dropped as invalid elsewhere in the network before reaching their destination. We believe the problem is at [1] - it looks like Suricata is populating that field with the packet total length (which is correct for IPv4) instead of the payload length (correct for IPv6). We have tested a patched version of Suricata with this line corrected (lpacket.len = LIBNET_TCP_H;) which solves the problem for us.

[1] https://github.com/OISF/suricata/blob/f8ec993401aaeb25f96b6f752ea8095bf5213af6/src/respond-reject-libnet11.c#L440


Subtasks 1 (0 open1 closed)

Bug #6039: TCP resets have incorrect len, nh in IPv6 (6.0.x backport)ClosedVictor JulienActions

JL Updated by Jamie Lavigne almost 3 years ago Actions #1

I believe we also needed to fix this next protocol number in our local version as well: https://github.com/OISF/suricata/blob/f8ec993401aaeb25f96b6f752ea8095bf5213af6/src/respond-reject-libnet11.c#L461

VJ Updated by Victor Julien almost 3 years ago Actions #2

  • Subject changed from TCP resets have incorrect length in IPv6 to TCP resets have incorrect len, nh in IPv6
  • Status changed from New to Assigned
  • Assignee changed from OISF Dev to Victor Julien
  • Target version changed from TBD to 7.0.0-rc2

Thanks for pointing these out, they look wrong indeed.

OT Updated by OISF Ticketbot almost 3 years ago Actions #3

  • Subtask #6039 added

OT Updated by OISF Ticketbot almost 3 years ago Actions #4

  • Label deleted (Needs backport to 6.0)

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

  • Status changed from In Review to Resolved

VJ Updated by Victor Julien almost 3 years ago Actions #7

  • Status changed from Resolved to Closed
Actions

Also available in: PDF Atom