diff options
author | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2022-07-10 16:21:34 +0000 |
---|---|---|
committer | A Wireshark GitLab Utility <gerald+gitlab-utility@wireshark.org> | 2022-07-13 20:55:49 +0000 |
commit | 32a4a08683323bc38fb0ef87cb3b3342f73a08e4 (patch) | |
tree | bd6657795580b67864081004a889cc9bac26a25b /epan/dissectors | |
parent | 69ea1930a71d6ce57276714a560241d1f36bf3db (diff) |
lapd: Fix Clang Analyzer warning
Access to field 'state' results in a dereference of a null pointer (loaded from variable 'ptr')
Diffstat (limited to 'epan/dissectors')
-rw-r--r-- | epan/dissectors/packet-lapd.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/epan/dissectors/packet-lapd.c b/epan/dissectors/packet-lapd.c index 3f1f9350b1..5101d1b030 100644 --- a/epan/dissectors/packet-lapd.c +++ b/epan/dissectors/packet-lapd.c @@ -383,9 +383,11 @@ dissect_lapd_bitstream(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void if (conversation) { if (convo_data) { /* already have lapd convo data */ - if (forward_stream) + if (forward_stream) { + if (!convo_data->byte_state_a) + convo_data->byte_state_a = wmem_new(wmem_file_scope(), lapd_byte_state_t); fill_lapd_byte_state(convo_data->byte_state_a, state, full_byte, bit_offset, ones, data, data_len); - else { + } else { if (!convo_data->byte_state_b) convo_data->byte_state_b = wmem_new(wmem_file_scope(), lapd_byte_state_t); fill_lapd_byte_state(convo_data->byte_state_b, state, full_byte, bit_offset, ones, data, data_len); |