Project

General

Profile

Actions

Bug #7887

closed
PA SB

detect/tls: zero characters in keywords such as alt name are mishandled

Bug #7887: detect/tls: zero characters in keywords such as alt name are mishandled

Added by Philippe Antoine 7 months ago. Updated 3 months ago.

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

Description

See #7881 and SV test to come associated with it

Rust Cstring new fails and returns NULL ptr

solution would be to store the connp->cert0_sans_len length of each connp->cert0_sans


Subtasks 3 (0 open3 closed)

Bug #7982: detect/tls: zero characters in keywords such as alt name are mishandled (8.0.x backport)ClosedShivani BhardwajActions
Bug #7996: tls: certificate SAN is freed in case of any errorClosedShivani BhardwajActions
Bug #8020: tls: certificate SAN is freed in case of any error (8.0.x backport)ClosedShivani BhardwajActions

Related issues 1 (0 open1 closed)

Copied from Suricata - Security #7881: detect/tls: keyword tls.subjectaltname leads to NULL Deref if tls.subjectaltname contains zeroClosedPhilippe AntoineActions

PA Updated by Philippe Antoine 7 months ago Actions #1

  • Copied from Security #7881: detect/tls: keyword tls.subjectaltname leads to NULL Deref if tls.subjectaltname contains zero added

PA Updated by Philippe Antoine 7 months ago Actions #2

  • Affected Versions 8.0.0 added

SB Updated by Shivani Bhardwaj 7 months ago Actions #3

  • Status changed from New to Assigned
  • Assignee changed from OISF Dev to Shivani Bhardwaj

SB Updated by Shivani Bhardwaj 6 months ago Actions #4

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

OT Updated by OISF Ticketbot 6 months ago Actions #5

  • Subtask #7982 added

OT Updated by OISF Ticketbot 6 months ago Actions #6

  • Label deleted (Needs backport to 8.0)

SB Updated by Shivani Bhardwaj 6 months ago Actions #7

  • Subtask #7996 added

SB Updated by Shivani Bhardwaj 6 months ago Actions #8

  • Status changed from Assigned to In Progress

SB Updated by Shivani Bhardwaj 5 months ago Actions #9

  • Subject changed from detect/tls: handle zero characters in keywords such as alt name to detect/tls: zero characters in keywords such as alt name are mishandled

PA Updated by Philippe Antoine 5 months ago Actions #10

By the way, I am not sure they are logged properly (not only detection problem)

SB Updated by Shivani Bhardwaj 5 months ago Actions #11

Philippe Antoine wrote in #note-10:

By the way, I am not sure they are logged properly (not only detection problem)

Indeed the output is truncated at the nul byte

SB Updated by Shivani Bhardwaj 4 months ago Actions #12

  • Private changed from Yes to No

SB Updated by Shivani Bhardwaj 4 months ago Actions #13

  • Status changed from In Progress to In Review

SB Updated by Shivani Bhardwaj 4 months ago Actions #14

  • Difficulty set to medium

SB Updated by Shivani Bhardwaj 3 months ago Actions #15

  • Status changed from In Review to Resolved

SB Updated by Shivani Bhardwaj 3 months ago Actions #16

  • Difficulty changed from medium to high

SB Updated by Shivani Bhardwaj 3 months ago Actions #17

  • Status changed from Resolved to Closed
Actions

Also available in: PDF Atom