Actions
Bug #1289
closedMPM b2gm matcher has questionable code
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.
Updated by Victor Julien about 10 years ago
Probably not worth fixing, instead we can remove this code. Don't think anyone uses it.
Updated by Ken Steele about 10 years ago
My work around was to remove the b2gm and b2gc MPM.
Updated by Andreas Herz almost 9 years ago
- Assignee set to OISF Dev
- Target version set to TBD
Do we have some updates for that? :)
Updated by Victor Julien almost 9 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.
Updated by Victor Julien over 8 years ago
- Target version changed from 3.1 to 3.1rc1
Updated by Victor Julien over 8 years ago
- Status changed from Assigned to Closed
Actions