Project

General

Profile

Actions

Bug #4952

closed

scan-build: Access to field 'de_state' results in a dereference of a null pointer

Added by Victor Julien over 2 years ago. Updated 12 months ago.

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

Description

  CC       detect.o
detect.c:1238:38: warning: Access to field 'de_state' results in a dereference of a null pointer (loaded from variable 'txd') [core.NullDereference]
    DetectEngineState *tx_de_state = txd->de_state;
                                     ^~~~~~~~~~~~~
1 warning generated.

txd is almost everywhere a part of the transaction structure, where it can't be NULL if we have a tx. However in the case of HTTP we can have a tx w/o a HTPTxUserData object which means it is possible to have a NULL txd in some edge cases.

Now a HTP tx w/o a HTPTxUserData is useless from the Suricata perspective, so I wonder if we can somehow "hide" a tx like that from Suricata.

Introduced in https://github.com/OISF/suricata/commit/1ad71b96daa2b2655691cfce2a15ccd754d9b290#diff-ee484ae4b77e59eb8b6b50f628c84ab626bd6178a6d43e1219dbc9619be7e027R1238


Related issues 1 (1 open0 closed)

Related to Suricata - Bug #5739: htp: handle alloc failure for user dataNewPhilippe AntoineActions
Actions #1

Updated by Victor Julien over 2 years ago

  • Description updated (diff)
Actions #2

Updated by Victor Julien about 2 years ago

So I have a branch that makes the AppLayerTxData non-optional, but need a solution for the HTPTxUserData case still.

Actions #3

Updated by Victor Julien about 2 years ago

  • Assignee set to OISF Dev
Actions #4

Updated by Victor Julien over 1 year ago

  • Target version changed from 7.0.0-beta1 to 7.0.0-rc1
Actions #5

Updated by Victor Julien over 1 year ago

  • Status changed from New to Assigned
  • Assignee changed from OISF Dev to Victor Julien
Actions #6

Updated by Victor Julien over 1 year ago

  • Subtask #5739 added
Actions #7

Updated by Victor Julien about 1 year ago

  • Target version changed from 7.0.0-rc1 to 7.0.0-rc2
Actions #9

Updated by Victor Julien 12 months ago

  • Subtask deleted (#5739)
Actions #10

Updated by Victor Julien 12 months ago

  • Related to Bug #5739: htp: handle alloc failure for user data added
Actions #11

Updated by Victor Julien 12 months ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF