diff options
| author | Mark Vitale <mvitale@sinenomine.net> | 2016-06-06 13:53:14 -0400 |
|---|---|---|
| committer | Michael Mann <mmann78@netscape.net> | 2016-06-06 21:59:29 +0000 |
| commit | 528edba87ae6a6ffd951bdd137421910ea88d964 (patch) | |
| tree | 1952840c56247e71accf28a0ea6f607ec391a349 /epan/dissectors/packet-afs.c | |
| parent | 75731e68e57f6a7f4b2702a19c1d8271369bf67d (diff) | |
afs: eliminate "malformed packet" false alarms
Many AFS packets have empty payloads, and Wireshark marks these as
"Malformed Packet" even though they are normal.
Eliminate these false alarms by using tvb_reported_length_remaining when
adding items to the tree.
Change-Id: I4f134fef36b8bb7f99224f02fea9cf29117fb36a
Reviewed-on: https://code.wireshark.org/review/15754
Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan/dissectors/packet-afs.c')
| -rw-r--r-- | epan/dissectors/packet-afs.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/epan/dissectors/packet-afs.c b/epan/dissectors/packet-afs.c index 87455227d7..585ba63538 100644 --- a/epan/dissectors/packet-afs.c +++ b/epan/dissectors/packet-afs.c @@ -2903,7 +2903,8 @@ dissect_afs(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void* data) ); } - ti = proto_tree_add_item(tree, proto_afs, tvb, offset, -1, + ti = proto_tree_add_item(tree, proto_afs, tvb, offset, + tvb_reported_length_remaining(tvb, offset), ENC_NA); afs_tree = proto_item_add_subtree(ti, ett_afs); |