Project

General

Profile

Actions

Optimization #6111

closed

defrag: avoid passing null pointers to functions

Added by Juliana Fajardini Reichow over 1 year ago. Updated 6 months ago.

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

Description

DecodeThreadVars, ThreadVars, and possibly some other variables are sometimes passed as null parameters
in our unit test functions.

This leads to the necessity of checking if they're not null pointers in several other places, such as
seen in https://github.com/OISF/suricata/blob/master/src/defrag.c#L1033, before using those variables, to prevent
warnings code analyzers.

This may be the case for other files, too.

Expected result: clean up calls of functions with null pointers, so we can clean up those checks for NULL pointers.

Actions #1

Updated by Victor Julien 6 months ago

  • Status changed from New to Closed
  • Assignee changed from OISF Dev to Victor Julien
Actions

Also available in: Atom PDF