Project

General

Profile

Actions

Bug #5073

closed
AW AW

Off-by-one in flow-manager flow_hash row allocation

Bug #5073: Off-by-one in flow-manager flow_hash row allocation

Added by Arne Welzel about 4 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Affected Versions:
Effort:
low
Difficulty:
Label:
Needs backport to 5.0, Needs backport to 6.0

Description

When configuring more than one flow manager, a single row between the per-flow-manager ranges is left-out.

The current code doesn't cover all rows when more than one flow manager is
used. It leaves a single row between ftd->max and ftd->min of the next
manager orphaned. As an example:

hash_size=1000
flowmgr_number=3
range=333

instance  ftd->min  ftd->max
0         0         333
1         334       666
2         667       1000

Rows not covered: 333, 666

PR: https://github.com/OISF/suricata/pull/6993


Related issues 2 (0 open2 closed)

Copied to Suricata - Bug #5112: Off-by-one in flow-manager flow_hash row allocationClosedShivani BhardwajActions
Copied to Suricata - Bug #5113: Off-by-one in flow-manager flow_hash row allocationClosedJeff LucovskyActions

AW Updated by Arne Welzel about 4 years ago Actions #1

  • Status changed from New to In Review

JL Updated by Jeff Lucovsky about 4 years ago Actions #2

  • Label Needs backport to 5.0, Needs backport to 6.0 added

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

  • Target version changed from TBD to 7.0.0-beta1
  • Affected Versions 5.0.8 added

JL Updated by Jeff Lucovsky about 4 years ago Actions #4

  • Copied to Bug #5112: Off-by-one in flow-manager flow_hash row allocation added

JL Updated by Jeff Lucovsky about 4 years ago Actions #5

  • Copied to Bug #5113: Off-by-one in flow-manager flow_hash row allocation added

JL Updated by Jeff Lucovsky about 4 years ago Actions #6

  • Assignee changed from OISF Dev to Arne Welzel

JL Updated by Jeff Lucovsky about 4 years ago Actions #7

  • Status changed from In Review to Closed
Actions

Also available in: PDF Atom