diff options
| author | Anders Broman <anders.broman@ericsson.com> | 2005-01-31 21:36:31 +0000 |
|---|---|---|
| committer | Anders Broman <anders.broman@ericsson.com> | 2005-01-31 21:36:31 +0000 |
| commit | f2c5299014527ec6621312c82831efed40f8c795 (patch) | |
| tree | 554482269212595e9e9ffe082099dbb89f2650c2 /epan | |
| parent | 22c81aa543bbcb0311357d91ea59c8a277378422 (diff) | |
| download | wireshark-f2c5299014527ec6621312c82831efed40f8c795.tar.gz wireshark-f2c5299014527ec6621312c82831efed40f8c795.tar.bz2 wireshark-f2c5299014527ec6621312c82831efed40f8c795.zip | |
From LEGO:
- remove duplicate fields from VSAs (every defined field in VSAs was set twice)
- dissect radius packet with avplen == 0 (did not decode authenticator when no AVPs were present)
svn path=/trunk/; revision=13216
Diffstat (limited to 'epan')
| -rw-r--r-- | epan/dissectors/packet-radius.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/epan/dissectors/packet-radius.c b/epan/dissectors/packet-radius.c index 32ee3b17c5..2c7c6bf59c 100644 --- a/epan/dissectors/packet-radius.c +++ b/epan/dissectors/packet-radius.c @@ -3757,9 +3757,6 @@ static void rd_value_to_str(gchar *dest, rd_vsa_buffer (*vsabuffer)[VSABUFFER], if ( next_attr_info ) { - rd_add_field_to_tree(tree, tvb, offset+8, - avph->avp_length-8, - next_attr_info); if ( next_attr_info->value_type == THE3GPP_QOS ) { cont = tmp_punt; @@ -4319,7 +4316,7 @@ static void dissect_radius(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) rh.rh_ident, "Packet identifier: 0x%01x (%d)", rhident,rhident); - if (avplength > 0) { + if (avplength >= 0) { proto_tree_add_uint(radius_tree, hf_radius_length, tvb, 2, 2, rhlength); } else { |