diff options
author | John A. Thacker <johnthacker@gmail.com> | 2016-01-06 09:05:24 -0500 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2016-01-06 18:24:05 +0000 |
commit | be0a7273f74ef53edfc599c47c4249e385bc6af1 (patch) | |
tree | 029e153a069dcbecaf72ce75ec2fdf42e46a6d09 /epan/dissectors/packet-gfp.c | |
parent | db28e237e95f1b6988d4cfc2042724b77dd47827 (diff) |
Correct fix for "GFP: Fix Dead Store (Dead assignement/Dead increment) Warning found by Clang"
This is the correct fix commit 0e7b7dc8839899e7cbfa4ff65788d84a1af1e4b5.
(As committed, deleting these two lines makes the dissector return the wrong
number of bytes processed.)
Change-Id: I2e88bbc6942f6af40199d5b03a71e182c1896871
Reviewed-on: https://code.wireshark.org/review/13074
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'epan/dissectors/packet-gfp.c')
-rw-r--r-- | epan/dissectors/packet-gfp.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/epan/dissectors/packet-gfp.c b/epan/dissectors/packet-gfp.c index 3959ee1d8d..84f10ed4cf 100644 --- a/epan/dissectors/packet-gfp.c +++ b/epan/dissectors/packet-gfp.c @@ -379,6 +379,8 @@ dissect_gfp_payload(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, proto_t default: break; } + *offset += payload_len; + *offset += fcs_len; } static int |