aboutsummaryrefslogtreecommitdiffstats
path: root/epan/dissectors/packet-radius.c
AgeCommit message (Collapse)AuthorFilesLines
2010-10-10Define some fcns & vars as static ...wmeier1-23/+23
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@34458 f5534014-38df-0310-8fa8-9805f1628bb7
2010-09-17Use val_to_str_ext() & friends to access sminmpec_values[];wmeier1-2/+4
Also: packet-nhrp.c: #include sminmpec.h not req'd; git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@34143 f5534014-38df-0310-8fa8-9805f1628bb7
2010-05-21Have abs_time_to_str() and abs_time_to_str_secs() take an additionalguy1-1/+1
argument indicating whether to include the time zone in the string. If we're constructing a display filter, don't include the time zone, otherwise do. Fixes bug 4756. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@32913 f5534014-38df-0310-8fa8-9805f1628bb7
2010-02-28Added svn:ignore targets.stig1-1/+1
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@32055 f5534014-38df-0310-8fa8-9805f1628bb7
2009-12-19For fields of type FT_ABSOLUTE_TIME, have the "display" value be one ofguy1-1/+1
ABSOLUTE_TIME_LOCAL or ABSOLUTE_TIME_UTC, indicating whether to display the date/time in local time or UTC. (int)ABSOLUTE_TIME_LOCAL == (int)BASE_NONE, so there's no source or binary compatiblity issue, although we might want to eliminate BASE_NONE at some point and have the BASE_ values used with integral types start at 0, so that you can't specify BASE_NONE for an integral field. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@31319 f5534014-38df-0310-8fa8-9805f1628bb7
2009-12-10Add an argument to abs_time_to_str() and abs_time_secs_to_str()guy1-1/+1
indicating whether the time should be shown as local time or UTC. For now, always pass FALSE, meaning "show as local time". Clean up some stuff in the SNMP dissector, use abs_time_secs_to_str() for times with one-second resolution, and update a comment in various macros in the WSP dissector, while we're at it. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@31227 f5534014-38df-0310-8fa8-9805f1628bb7
2009-12-10Fix Bug 4316: Crash when using custom RADIUS dictionary.wmeier1-0/+1
(The bug was caused by a variable not being initialized). See: https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4316 git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@31221 f5534014-38df-0310-8fa8-9805f1628bb7
2009-10-12Initialize radius_call, so it's null if we don't pass through the codeguy1-1/+1
path that sets it. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@30524 f5534014-38df-0310-8fa8-9805f1628bb7
2009-10-12Replace use of GMemChunks by se_alloc();wmeier1-575/+554
@include <stdlib.h> not req'd; Use consistent indentation. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@30522 f5534014-38df-0310-8fa8-9805f1628bb7
2009-08-26(FWIW) One step towards including stdio.h & stdlib.h only when req'd.wmeier1-0/+1
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@29568 f5534014-38df-0310-8fa8-9805f1628bb7
2009-08-26From Thierry Andry via bug 3941:stig1-1/+4
Added preference to set radius request TTL. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@29561 f5534014-38df-0310-8fa8-9805f1628bb7
2009-08-09Don't guard col_clear with col_checkkrj1-2/+1
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@29344 f5534014-38df-0310-8fa8-9805f1628bb7
2009-08-09Don't guard col_set_str (COL_PROTOCOL) with col_checkkrj1-2/+1
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@29340 f5534014-38df-0310-8fa8-9805f1628bb7
2009-07-30Remove a couple of unused variables.gerald1-6/+0
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@29245 f5534014-38df-0310-8fa8-9805f1628bb7
2009-07-07From Kovarththanan Rajaratnam via bug 3548:stig1-4/+4
Minor hf cleanup. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@28968 f5534014-38df-0310-8fa8-9805f1628bb7
2009-07-06Radius: Additional attribute handling:wmeier1-0/+10
- implement "ether" attribute; - parse "abinary" attribute [uses existing radius_abinary()] Also: add a comment in packet-radius.h about the meaning of the encrypt flag. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@28952 f5534014-38df-0310-8fa8-9805f1628bb7
2009-07-03From: Tobias Witek: Fix processing of radius 'vendor specific attributes'wmeier1-0/+5
In the RADIUS dissector, the function radius_register_avp_dissector() registers vendors that are not already present in the dictionary hash-table. As far as I can see, there are two problems with this: 1. The function does not set the number of type/length octets and the has_flags variable for that AVP, which is required to correctly decode AVP/VSA values 2. In some situations, the function is called _before_ radius_load_dictionary() is called (for example for the vendor 3GPP (ID: 10415)) Therefore, all vendor entries that are created by calling radius_register_avp_dissector() leave their type_octets and length_octets un-initialized, which causes incorrect decoding. [Result: Radius dissector displays messages such as: "Malformed Packet: RADIUS" and "Error/Malformed: Malformed Packet(Exception occurred)"] The attached patch fixes this problem by assuming that the dictionary knows the 'ground truth' about the type/length octet and the has_flags information and allows it to overwrite these values even for vendors that have already been loaded. Also: (from Bill Meier): set the type/length octet and the has_flags variables to default "standard" values (1,1,FALSE) in radius_register_avp_dissector(). Fixes Bug #3651 (and Bug #3635). https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3651 https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3635 git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@28937 f5534014-38df-0310-8fa8-9805f1628bb7
2009-06-30packet-radius: Fix exception CLEANUP handling when malformed attribute ↵wmeier1-12/+15
list/pair seen; Fixes crash reported in Bug #3578. [https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3578] Essentially: CLEANUP_CALL_AND_POP wasn't being executed for certain error exits from dissect_attribute_pairs() thus leaving a CLEANUP entry on the exception stack. Also: vsa_buffer_table wasn't being destroyed if an exception occurred in dissect_attribute_pairs. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@28891 f5534014-38df-0310-8fa8-9805f1628bb7
2009-06-18From Kovarththanan Rajaratnam via bug 3548:stig1-20/+20
(1) Trailing/leading spaces are removed from 'name's/'blurb's (2) Duplicate 'blurb's are replaced with NULL (3) Empty ("") 'blurb's are replaced with NULL (4) BASE_NONE, NULL, 0x0 are used for 'display', 'strings' and 'bitmask' fields for FT_NONE, FT_BYTES, FT_IPv4, FT_IPv6, FT_ABSOLUTE_TIME, FT_RELATIVE_TIME, FT_PROTOCOL, FT_STRING and FT_STRINGZ field types (5) Only allow non-zero value for 'display' if 'bitmask' is non-zero git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@28770 f5534014-38df-0310-8fa8-9805f1628bb7
2009-06-14Prevent duplicate header field registrations.jake1-5/+2
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@28720 f5534014-38df-0310-8fa8-9805f1628bb7
2009-06-09RFC3576 states, that some messages will use port 3799 insteadjmayer1-0/+2
of the "standard" radius ports. Register radius to that port as well. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@28672 f5534014-38df-0310-8fa8-9805f1628bb7
2009-05-13Apply some of the patches from:etxrab1-4/+4
http://wiki.wireshark.org/Development/Optimization git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@28356 f5534014-38df-0310-8fa8-9805f1628bb7
2009-04-02From Naoyoshi Ueda:etxrab1-9/+324
Radius dissector enhancement to support WiMAX vendor specific attributes. https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3176 git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@27937 f5534014-38df-0310-8fa8-9805f1628bb7
2009-03-29More size_t casts.etxrab1-1/+1
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@27887 f5534014-38df-0310-8fa8-9805f1628bb7
2009-03-22P64 changes: Fix some cases for which size_t is not requiredwmeier1-1/+1
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@27822 f5534014-38df-0310-8fa8-9805f1628bb7
2009-03-18From Bjørn Mork:etxrab1-0/+36
Decode ipv6prefix attributes in packet-radius. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@27769 f5534014-38df-0310-8fa8-9805f1628bb7
2008-12-29From Florian Lohoff:etxrab1-2/+77
Decode binary "Ascend-Data-Filter" vendor specific attribute in packet-radius https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=2975 git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@27127 f5534014-38df-0310-8fa8-9805f1628bb7
2008-12-29From Florian Lohoff:etxrab1-4/+5
packet-radius.c - avp specific dissectors broken https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=2974 git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@27126 f5534014-38df-0310-8fa8-9805f1628bb7
2008-09-30Minor cleanup related to proto_register, proto_reg_handoffwmeier1-30/+29
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@26307 f5534014-38df-0310-8fa8-9805f1628bb7
2008-09-11Work around a problem with custom columns.etxrab1-2/+1
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@26178 f5534014-38df-0310-8fa8-9805f1628bb7
2008-08-05luis.ontanon@gmail.com => luis@ontanon.orglego1-1/+1
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@25937 f5534014-38df-0310-8fa8-9805f1628bb7
2008-05-14Fix some of the Errors/warnings detected by checkapi.wmeier1-5/+9
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@25290 f5534014-38df-0310-8fa8-9805f1628bb7
2008-04-03http://www.wireshark.org/lists/wireshark-dev/200803/msg00308.htmllego1-150/+132
proto.[hc] define new APIs to allow delayed registration of protocol fields, so that dissectors with "flexible" fields like xml, radius, diameter, snmp do not have to load their files at startup but can do so as late as possible. gtk/dfilter_expr_dlg.c : have the expression dialog registering all prefixes so that all fileds appear in the dialog tshark.c register all prefixes when called with -G epan/radius_dict.l epan/dissectors/packet-radius.c epan/dissectors/packet-radius.h refactor registration to delay dictionary loading as long as possible git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@24762 f5534014-38df-0310-8fa8-9805f1628bb7
2008-03-03Fix for http://bugs.wireshark.org/bugzilla/show_bug.cgi?id=2257jmayer1-1/+1
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@24533 f5534014-38df-0310-8fa8-9805f1628bb7
2008-01-14This patch adds support for configuration profiles, which can be used tostig1-1/+1
configure and use more than one set of preferences and configuration files. This can be found in the "Configuration Profiles..." menu item from the Edit menu, or by pressing Shift-Ctrl-A. It's also possible to start wireshark and tshark with a named profile by using the "-C ProfileName" option. A new status pane in the main window will show the current profile. The configuration files currently stored in the Profiles are: - Preferences - Capture Filters - Display Filters - Coloring Rules - Disabled Protocols - User Accessible Tables The recent data are by design not added to the profile. Planned future enhancements: - make a more convenient function to switch between profiles - add a "clone profile" button to copy an existing profile - make the profiles list active and accept return as OK - save users "Decode as" in the profile - make new, clone and deletion of profiles more secure - make some of the recent values available in the profile This patch also fixes: - setting default status pane sizes - a bug setting status pane for packets when not having main lower pane. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@24089 f5534014-38df-0310-8fa8-9805f1628bb7
2007-11-30use g_strconcat() which should be faster instead of g_strdup_printf()kukosa1-5/+5
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@23681 f5534014-38df-0310-8fa8-9805f1628bb7
2007-09-30change dissect_radius() to become a new-style dissector that can reject sahlberg1-69/+123
packets that doesnt look like valid radius. verify that a packet is radius by checking that the command code is known and also that the length is between 20 and 4096 bytes move the tap data to be ep_allocated instead of a static global variable dont use tvb_memcpy() to read a structure off the wiredata use tvb_get_... to unmarshall the fields explicitely this fixes bug 1634 git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@23039 f5534014-38df-0310-8fa8-9805f1628bb7
2007-07-13Fix bugetxrab1-26/+47
http://bugs.wireshark.org/bugzilla/show_bug.cgi?id=1640 change "Change-Filter-Request (43)" to "CoA-Request (43)" git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@22301 f5534014-38df-0310-8fa8-9805f1628bb7
2007-05-29Use G_GINT64_MODIFIER, rather than the PRI[douxX]64 macros, for GLibguy1-1/+1
routines and routines using those routines. GLib might use different modifiers for 64-bit quantities than the platform's C library does. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@21990 f5534014-38df-0310-8fa8-9805f1628bb7
2007-04-30Add a header that redefines g_array_index() to throw in a "void *" castguy1-3/+4
to work around the "data" field of a GArray being a guint8 *, and defines a g_array_data() macro to extract that field and cast it to void *. Use that header where needed. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@21627 f5534014-38df-0310-8fa8-9805f1628bb7
2007-04-28The data in a GArray is guaranteed to be properly aligned; add a cast toguy1-3/+3
squelch compiler warnings. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@21617 f5534014-38df-0310-8fa8-9805f1628bb7
2007-04-10Get rid of some more g_assert*()'s in the dissectors. There are a fewmorriss1-1/+1
remaining that I'm not sure exactly what to do with at the moment: the one in packet-frame probably should be there, the others probably shouldn't but they also should never fail unless there's a compile or build problem (AFAICS). git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@21367 f5534014-38df-0310-8fa8-9805f1628bb7
2007-03-23fix a lot more warningsulfl1-1/+1
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@21142 f5534014-38df-0310-8fa8-9805f1628bb7
2007-01-29Fix potential buffer overruns in address_to_str_buf() found by Andrej gerald1-2/+2
Mikus. Add a buf_len parameter to ip_to_str_buf(), and make sure it's enforced. Copy the release notes over from the 0.99.5 trunk and add a note about the ISUP dissector (which is affected by the overrun). git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@20607 f5534014-38df-0310-8fa8-9805f1628bb7
2007-01-02Move epan/crypt-md5.[ch] to epan/crypt. Removegerald1-137/+137
epan/crypt/airpdcap_md5.[ch]. Fix up whitespace. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@20277 f5534014-38df-0310-8fa8-9805f1628bb7
2006-12-07As ponted out by Stig Bjørlykke change a whole bunch of dissector_handle_t ↵etxrab1-2/+2
to static. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@20062 f5534014-38df-0310-8fa8-9805f1628bb7
2006-11-14nikai@nikai.net:jmayer1-1/+0
Remaining fixes from: http://bugs.wireshark.org/bugzilla/show_bug.cgi?id=1225 git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@19895 f5534014-38df-0310-8fa8-9805f1628bb7
2006-10-24From Alejandro Vaquero:etxrab1-39/+389
For the Radius dissector to add statistics in the "Service Response Time" menu and the tshark stats. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@19668 f5534014-38df-0310-8fa8-9805f1628bb7
2006-09-05Make smrse & radius dissectors findable by namemartinm1-0/+1
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@19157 f5534014-38df-0310-8fa8-9805f1628bb7
2006-08-27squelch some compiler warningsulfl1-1/+1
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@19052 f5534014-38df-0310-8fa8-9805f1628bb7