From d0520f81d11e6ee409139c50771661b50e358674 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jo=C3=A3o=20Valverde?= Date: Wed, 14 Jun 2017 01:32:37 +0100 Subject: [PATCH] RADIUS: Add comment explaining WiMAX non-standard VSA format MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Change-Id: I5b3417c94ab10d4ed22258bdb2ef0f670dd2b995 Reviewed-on: https://code.wireshark.org/review/22118 Reviewed-by: João Valverde --- epan/dissectors/packet-radius.c | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/epan/dissectors/packet-radius.c b/epan/dissectors/packet-radius.c index ffeb80b810..d325f2aa53 100644 --- a/epan/dissectors/packet-radius.c +++ b/epan/dissectors/packet-radius.c @@ -1607,6 +1607,20 @@ dissect_attribute_value_pairs(proto_tree *tree, packet_info *pinfo, tvbuff_t *tv } if (vendor->has_flags) { + /* + * WiMAX VSA's have a non-standard format: + * + * type 1 octet + * length 1 octet + * continuation 1 octet 0bcrrrrrrr + * value 1+ octets + * + * If the high bit of the "continuation" field is set, then + * the next attribute of the same WiMAX type should have it's + * value concatenated to this one. + * + * See "dictionary.wimax" from FreeRADIUS for details and references. + */ radius_vsa_buffer_key key; radius_vsa_buffer *vsa_buffer = NULL; key.vendor_id = vendor_id;