Project

General

Profile

Actions

Optimization #4475

open

Rust: Make default_port in parser registration an Option

Added by Shivani Bhardwaj 5 months ago. Updated 3 months ago.

Status:
In Review
Priority:
Normal
Target version:
Effort:
Difficulty:
Label:

Related issues

Related to Optimization #3315: app-layer: unify registration logicClosedShivani BhardwajActions
Actions #1

Updated by Shivani Bhardwaj 3 months ago

Actions #2

Updated by Shivani Bhardwaj 3 months ago

  • Status changed from Assigned to In Review

Seems like using Option for `*const i8` is not FFI safe and Rust warns about it.
Using a null string in place of default port does seem to do the work though. Changed it in DCERPC only for now where there is not a fixed port of communication.
PR for review: https://github.com/OISF/suricata/pull/6234

Actions

Also available in: Atom PDF