diff options
author | Guy Harris <guy@alum.mit.edu> | 2017-01-24 01:15:24 -0800 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2017-01-24 09:19:46 +0000 |
commit | 7b69c8702b30e76189a652b7853643fdbc506a22 (patch) | |
tree | c745e8ddada8cee08ceb672821d049d7e5f1e7a3 | |
parent | c3013565b14df28251bd2f51c1af9418833996e4 (diff) | |
download | wireshark-7b69c8702b30e76189a652b7853643fdbc506a22.tar.gz |
Add a comment talking about the dictionary file format.
Point to the FreeRADIUS man page for the dictionary file format, and
mention features we don't support - including features *not* documented
in the man page.
Change-Id: I08085062a6abcafb9dd0abe5b1edee53187ea23f
Reviewed-on: https://code.wireshark.org/review/19762
Reviewed-by: Guy Harris <guy@alum.mit.edu>
-rw-r--r-- | epan/radius_dict.l | 27 |
1 files changed, 27 insertions, 0 deletions
diff --git a/epan/radius_dict.l b/epan/radius_dict.l index 2d446b1a8e..6ca0008dfb 100644 --- a/epan/radius_dict.l +++ b/epan/radius_dict.l @@ -92,6 +92,33 @@ #include <epan/dissectors/packet-radius.h> #include <wsutil/file_util.h> +/* + * See + * + * http://freeradius.org/radiusd/man/dictionary.html + * + * for the format of RADIUS dictionary files. + * + * XXX - features not currently supported: + * + * dotted number values for the oid field; + * + * integer64, ipv4prefix, combo-prefix, bool, size, decimal, + * timeval, struct, extended, long-extended, vsa, evs, vendor, + * cidr, uint{8,16,32,64}, int{8,16,32,64} as attribute types + * (some of these aren't documented); + * + * octets[N], where N is an integer, as an attribute type + * (not documented in the man page); + * + * internal, array, concat, and virtual as attribute flags; + * + * format= for BEGIN-VENDOR. + * + * We should, perhaps, adopt FreeRADIUS's dictionary-parsing code in + * src/lib/dict.c and use that, rather than writing our own parser. + * See bug 13176. + */ #define YY_USER_INIT BEGIN WS_OUT; #ifdef _WIN32 |