diff options
| author | guy <guy@f5534014-38df-0310-8fa8-9805f1628bb7> | 2010-02-10 08:20:55 +0000 |
|---|---|---|
| committer | guy <guy@f5534014-38df-0310-8fa8-9805f1628bb7> | 2010-02-10 08:20:55 +0000 |
| commit | bfd12c9b4e14d3d3283984e9ac1209320f63cc41 (patch) | |
| tree | 3b76b60a22f36bbf7970711ddbc35ef82d82c635 /epan/dissectors/packet-mip.c | |
| parent | 9ab397f1292e020c1a75f97fb8ca3028e7f8a31d (diff) | |
No need for cvse_reserved as a variable; getting rid of it squelches a
compiler warning.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@31854 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'epan/dissectors/packet-mip.c')
| -rw-r--r-- | epan/dissectors/packet-mip.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/epan/dissectors/packet-mip.c b/epan/dissectors/packet-mip.c index 2940816331..85f6a35a7b 100644 --- a/epan/dissectors/packet-mip.c +++ b/epan/dissectors/packet-mip.c @@ -418,9 +418,8 @@ dissect_mip_extensions( tvbuff_t *tvb, int offset, proto_tree *tree) guint8 pmipv4skipext_type; guint16 flags; gint hdrLen; - guint8 cvse_reserved; - guint32 cvse_vendor_id; - guint16 cvse_vendor_type; + guint32 cvse_vendor_id; + guint16 cvse_vendor_type; int cvse_local_offset= 0; /* None of this really matters if we don't have a tree */ @@ -449,7 +448,6 @@ dissect_mip_extensions( tvbuff_t *tvb, int offset, proto_tree *tree) * CVSE also breaks since it added reserved field before * the length field */ - cvse_reserved = tvb_get_guint8(tvb, offset + 1); ext_len = tvb_get_ntohs(tvb, offset + 2); hdrLen = 4; } @@ -614,7 +612,7 @@ dissect_mip_extensions( tvbuff_t *tvb, int offset, proto_tree *tree) * Very nasty . . breaks normal extensions, since the length is * in the wrong place :( */ - proto_tree_add_uint(ext_tree, hf_mip_cvse_reserved, tvb, offset, 1, cvse_reserved); + proto_tree_add_item(ext_tree, hf_mip_cvse_reserved, tvb, offset, 1, FALSE); offset++; proto_tree_add_uint(ext_tree, hf_mip_ext_len, tvb, offset, 2, ext_len); offset+=2; |