diff options
| author | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2016-02-16 21:37:38 +0100 |
|---|---|---|
| committer | Michael Mann <mmann78@netscape.net> | 2016-02-19 03:20:28 +0000 |
| commit | 323659d85e4708a5556baec4355552fd07255fcc (patch) | |
| tree | 5e1cd7ba6dad447dcfc7f00bca57b88ebb06b39a /epan/dissectors/packet-dcerpc.c | |
| parent | 6cf6cdb56fbca962db6685ed8525deffe07c37bd (diff) | |
dcerpc: fix Called function pointer is null (null dereference) found by Clang Analyzer
Change-Id: Idf98bcf617d4d6343aa233e42898cf5f26b08e33
Reviewed-on: https://code.wireshark.org/review/13974
Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan/dissectors/packet-dcerpc.c')
| -rw-r--r-- | epan/dissectors/packet-dcerpc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-dcerpc.c b/epan/dissectors/packet-dcerpc.c index d36c344a0c..ea69264ae4 100644 --- a/epan/dissectors/packet-dcerpc.c +++ b/epan/dissectors/packet-dcerpc.c @@ -2414,7 +2414,7 @@ dissect_ndr_ucvarray_core(tvbuff_t *tvb, gint offset, packet_info *pinfo, pinfo, tree, di, drep); if (offset <= old_offset) THROW(ReportedBoundsError); - } else { + } else if (fnct_bytes) { for (i=0 ;i<di->array_actual_count; i++) { old_offset = offset; offset = (*fnct_bytes)(tvb, offset, pinfo, tree, di, drep); |