diff options
author | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2015-04-20 13:34:22 +0200 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2015-04-20 14:46:25 +0000 |
commit | 262625b7ba566950bc4f80a97ec7b6f35c6e4718 (patch) | |
tree | 10190e694958e5b1fd68d1c1243a068a76a9cbb0 /epan | |
parent | 263564f175851b578179c85fdcf1f8c6f213176f (diff) |
OpenSafety: Fix Dead Store (Dead assignement/Dead increment) warning found by Clang
Change-Id: Iccd80450835946abbe68d66fb4ccbc1c799e6065
Reviewed-on: https://code.wireshark.org/review/8137
Reviewed-by: Roland Knall <rknall@gmail.com>
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'epan')
-rw-r--r-- | epan/dissectors/packet-opensafety.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/epan/dissectors/packet-opensafety.c b/epan/dissectors/packet-opensafety.c index c3110032a9..3fc86782b2 100644 --- a/epan/dissectors/packet-opensafety.c +++ b/epan/dissectors/packet-opensafety.c @@ -728,9 +728,6 @@ dissect_opensafety_spdo_message(tvbuff_t *message_tvb, packet_info *pinfo, proto /* allow only valid SPDO flags */ spdoFlags = ( ( tr >> 2 ) & OPENSAFETY_SPDO_FEATURE_FLAGS ); - /* determine the ct value. if complete it can be used for analysis of the package */ - ct = tvb_get_guint8(message_tvb, packet->frame.subframe1 + 3); - /* An SPDO is always sent by the producer, to everybody else . * For a 40bit connection OPENSAFETY_DEFAULT_DOMAIN is assumed as sdn value for now */ if ( (OPENSAFETY_SPDO_FEAT_40BIT_USED & spdoFlags ) == OPENSAFETY_SPDO_FEAT_40BIT_USED ) @@ -1114,7 +1111,7 @@ dissect_opensafety_ssdo_message(tvbuff_t *message_tvb, packet_info *pinfo, proto /* Toggle bit must be removed, otherwise the values cannot be displayed correctly */ if ( packet->payload.ssdo->sacmd.toggle ) db0 &= (~OPENSAFETY_SSDO_SACMD_TGL); - item = proto_tree_add_bitmask(ssdo_tree, message_tvb, db0Offset, hf_oss_ssdo_sacmd, + proto_tree_add_bitmask(ssdo_tree, message_tvb, db0Offset, hf_oss_ssdo_sacmd, ett_opensafety_ssdo_sacmd, ssdo_sacmd_flags, ENC_NA); col_append_fstr(pinfo->cinfo, COL_INFO, ", SACMD: %s", val_to_str_const(db0, opensafety_ssdo_sacmd_values, " ")); |