diff options
| author | Michael Mann <mmann78@netscape.net> | 2016-07-12 13:23:29 -0400 |
|---|---|---|
| committer | Michael Mann <mmann78@netscape.net> | 2016-07-12 18:29:41 +0000 |
| commit | 2f3b25c181ac93a541ea77b08bfb93944d04840b (patch) | |
| tree | 4f4922ecde93f39f4551766ca7565f4a9f72d2d4 /epan | |
| parent | e063924a02a06b1d0084b6aa584a3785d6e026b8 (diff) | |
packet-btrfcomm.c: Add NULL check to pacify VS Code Analysis
Some if checks are confusing the analysis, but it does add some
future-proofing.
Change-Id: Ic4fe16a87646a7055f7639af050d53ff004eda5f
Reviewed-on: https://code.wireshark.org/review/16396
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan')
| -rw-r--r-- | epan/dissectors/packet-btrfcomm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-btrfcomm.c b/epan/dissectors/packet-btrfcomm.c index a36403769f..01abbd9cd9 100644 --- a/epan/dissectors/packet-btrfcomm.c +++ b/epan/dissectors/packet-btrfcomm.c @@ -746,7 +746,7 @@ dissect_btrfcomm(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *data col_append_fstr(pinfo->cinfo, COL_INFO, "%s Channel=%u ", val_to_str_const(frame_type, vs_frame_type_short, "Unknown"), dlci >> 1); - if (dlci && (frame_type == FRAME_TYPE_SABM)) { + if (dlci && (frame_type == FRAME_TYPE_SABM) && service_info) { if (service_info->uuid.size==16) col_append_fstr(pinfo->cinfo, COL_INFO, "(UUID128: %s) ", print_uuid(&service_info->uuid)); else |