diff options
author | Martin Mathieson <martin.mathieson@keysight.com> | 2020-06-12 17:38:03 +0100 |
---|---|---|
committer | Martin Mathieson <martin.r.mathieson@googlemail.com> | 2020-06-12 19:01:33 +0000 |
commit | 885ac0c2790b366f848604b69770e0a1c339fd08 (patch) | |
tree | c004cd1517889809192de02abe7c9dea69cb25c1 /epan/dissectors/packet-isobus-vt.c | |
parent | 231663a0fb419eb03ed811418a483a41a113354c (diff) |
More cppcheck fixes, using top-of-tree cppcheck.
Mostly redundant tests, but for bthci dissectors,
avoid using a key that points to out-of-scope variables.
Change-Id: I2785690b8487d45c4e2daff0b6e03cd8747ed410
Reviewed-on: https://code.wireshark.org/review/37467
Petri-Dish: Martin Mathieson <martin.r.mathieson@googlemail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Martin Mathieson <martin.r.mathieson@googlemail.com>
Diffstat (limited to 'epan/dissectors/packet-isobus-vt.c')
-rw-r--r-- | epan/dissectors/packet-isobus-vt.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/epan/dissectors/packet-isobus-vt.c b/epan/dissectors/packet-isobus-vt.c index 6e71b6a96c..132080f1fe 100644 --- a/epan/dissectors/packet-isobus-vt.c +++ b/epan/dissectors/packet-isobus-vt.c @@ -1175,10 +1175,8 @@ dissect_vt(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, enum vt_directio proto_item_append_text(ti, ": "); if (termination_cause & 0x01) - proto_item_append_text(ti, "Audio was terminated "); - - if (termination_cause & 0x01) { + proto_item_append_text(ti, "Audio was terminated "); col_append_fstr(pinfo->cinfo, COL_INFO, "VT Control audio signal termination: Audio was terminated"); } else |