Project

General

Profile

Actions

Bug #1289

closed

MPM b2gm matcher has questionable code

Added by Ken Steele over 9 years ago. Updated about 8 years ago.

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

Description

The B2GM MPM matcher has questionable code related to use of B2gmPattern1 structures. These are only used for patterns of length 1.

The array ctx->patterns1 is of type uint8_t*, but are then always cast to B2gmPattern1 pointers. Why is patterns1 not of type (B2gmPattern1*)?
Offsets into the uint8_t array are started at 1 with a comment about skipping the first byte of the buffer. This will always result in unaligned access to the structure.

Actions #1

Updated by Victor Julien over 9 years ago

Probably not worth fixing, instead we can remove this code. Don't think anyone uses it.

Actions #2

Updated by Ken Steele over 9 years ago

My work around was to remove the b2gm and b2gc MPM.

Actions #3

Updated by Andreas Herz about 8 years ago

  • Assignee set to OISF Dev
  • Target version set to TBD

Do we have some updates for that? :)

Actions #4

Updated by Victor Julien about 8 years ago

  • Status changed from New to Assigned
  • Assignee changed from OISF Dev to Victor Julien
  • Target version changed from TBD to 3.1

Scheduled for removal in 3.1.

Actions #5

Updated by Victor Julien about 8 years ago

  • Target version changed from 3.1 to 3.1rc1
Actions #6

Updated by Victor Julien about 8 years ago

  • Status changed from Assigned to Closed
Actions

Also available in: Atom PDF