diff options
author | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2019-01-06 14:42:35 +0000 |
---|---|---|
committer | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2019-01-07 19:35:41 +0000 |
commit | 30e677593559c5c328e770d0e0bfb50c8e6dcd62 (patch) | |
tree | e2705e33b6050b8dac38e07e3c33f62cf3e5c13f | |
parent | 4b42a45ebc35b5d2c7ff00cb5653d25497c4a848 (diff) |
f5ethtrailer: Fix Dead Store (Dead assignement/Dead increment) Warning found by Clang
Change-Id: Ie8f1a4a4a11370ff5d5d85f8110aec568f88877e
Reviewed-on: https://code.wireshark.org/review/31410
Reviewed-by: Michael Mann <mmann78@netscape.net>
Petri-Dish: Michael Mann <mmann78@netscape.net>
Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
-rw-r--r-- | epan/dissectors/packet-f5ethtrailer.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/epan/dissectors/packet-f5ethtrailer.c b/epan/dissectors/packet-f5ethtrailer.c index 0736eb66cb..775dfdd07c 100644 --- a/epan/dissectors/packet-f5ethtrailer.c +++ b/epan/dissectors/packet-f5ethtrailer.c @@ -2177,7 +2177,6 @@ dissect_dpt_trailer_noise_high( proto_tree_add_item(tree, hf_peer_local_port, tvb, o, 2, ENC_BIG_ENDIAN); pi = proto_tree_add_item(tree, hf_peer_port, tvb, o, 2, ENC_BIG_ENDIAN); PROTO_ITEM_SET_HIDDEN(pi); - o += 2; return(len); } /* dissect_dpt_trailer_noise_high() */ @@ -2324,10 +2323,9 @@ dissect_dpt_trailer_noise_med( ENC_ASCII)); proto_tree_add_item(rc_tree, hf_rstcause_txt, tvb, o, rstcauselen-(o-startcause), ENC_ASCII|ENC_NA); - o = startcause + rstcauselen; + /*o = startcause + rstcauselen;*/ break; default: - o += rstcauselen; break; } } @@ -2437,7 +2435,6 @@ dissect_dpt_trailer_noise_low( } o += 1; proto_tree_add_item(tree, hf_vip, tvb, o, vipnamelen, ENC_ASCII|ENC_NA); - o += vipnamelen; return(len); } /* dissect_dpt_trailer_noise_low() */ |